Skip to content

Always skip randomly failing OCI8 extauth tests #9524

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 29 commits into from
Closed

Always skip randomly failing OCI8 extauth tests #9524

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
29 commits
Select commit Hold shift + click to select a range
a19447f
unify tests before fix
mvorisek Sep 11, 2022
323e144
use real host to prevent timeouts
mvorisek Sep 11, 2022
1ed7cc9
DEBUG run 2x 20 tests
mvorisek Sep 11, 2022
b2246cb
DEBUG increase testing timeout
mvorisek Sep 11, 2022
ef6ce31
Revert "DEBUG run 2x 20 tests"
mvorisek Sep 11, 2022
f1544b0
use "mysql" host instead
mvorisek Sep 11, 2022
c131cc6
use "anything.invalid" host
mvorisek Sep 11, 2022
76af0fc
Revert "DEBUG increase testing timeout"
mvorisek Sep 11, 2022
77663d2
DEBUG run 2x 20 tests
mvorisek Sep 11, 2022
9e74767
DEBUG increase testing timeout
mvorisek Sep 11, 2022
9c2b696
DEBUG verify original issue is reproducible
mvorisek Sep 12, 2022
7904827
use "mysql" host instead
mvorisek Sep 12, 2022
015bdb7
use "sql1" host (MSSQL docker) everywhere
mvorisek Sep 12, 2022
8af58b0
use "x.cz" host (domain must be at least 2 chars) everywhere
mvorisek Sep 12, 2022
466ba80
DEBUG test only 'bcmath', 'oci8', 'pdo_oci' ext tests and 1 proc only
mvorisek May 26, 2022
e2147d4
DEBUG dump individual times, rm slow tests
mvorisek Sep 12, 2022
8ca49a6
DEBUG run all tests 4x
mvorisek Sep 12, 2022
4b93033
DEBUG fail + dump delay on 10+ secs
mvorisek Sep 12, 2022
a11bf80
use "sql1" host (MSSQL docker) everywhere
mvorisek Sep 12, 2022
d86f02b
anything?connect_timeout=60&transport_connect_timeout=30
mvorisek Sep 12, 2022
30786fe
anything?connect_timeout=6&transport_connect_timeout=3
mvorisek Sep 12, 2022
0a9dc03
Revert "DEBUG run 2x 20 tests"
mvorisek Sep 11, 2022
6e2bb7e
strace extauth test
mvorisek Sep 12, 2022
f509e38
Revert "Revert "DEBUG run 2x 20 tests""
mvorisek Sep 11, 2022
a0ab112
ltrace -fS
mvorisek Sep 12, 2022
a5fc4e9
strace -k
mvorisek Sep 12, 2022
bad6503
revert all
mvorisek Sep 12, 2022
474c6d7
skip the problematic extauth_xx.phpt tests
mvorisek Sep 12, 2022
7f5a754
the tests are not slow
mvorisek Sep 12, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 1 addition & 4 deletions ext/oci8/tests/extauth_01.phpt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,16 +4,13 @@ Test External Authentication errors with oci_connect
<?php
if (!extension_loaded('oci8')) die ("skip no oci8 extension");
if (substr(PHP_OS, 0, 3) == 'WIN') die("skip feature not available on Windows platforms");
require(__DIR__.'/details.inc');
if (getenv('SKIP_SLOW_TESTS')) die('skip slow tests excluded by request');
die("skip random CI timeouts caused by Oracle Instant Client, see https://github.com/php/php-src/pull/9524#issuecomment-1244409815");
?>
--INI--
oci8.privileged_connect=1
--FILE--
<?php

// Run Test

echo "Test 1\n";

$c = oci_connect('/', 'notemtpy', 'anything', null, OCI_CRED_EXT);
Expand Down
4 changes: 1 addition & 3 deletions ext/oci8/tests/extauth_02.phpt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,16 +3,14 @@ Test External Authentication errors with oci_new_connect
--SKIPIF--
<?php
if (!extension_loaded('oci8')) die ("skip no oci8 extension");
if (getenv('SKIP_SLOW_TESTS')) die('skip slow tests excluded by request');
if (substr(PHP_OS, 0, 3) == 'WIN') die("skip feature not available on Windows platforms");
die("skip random CI timeouts caused by Oracle Instant Client, see https://github.com/php/php-src/pull/9524#issuecomment-1244409815");
?>
--INI--
oci8.privileged_connect=1
--FILE--
<?php

// Run Test

echo "Test 1\n";

$c = oci_new_connect('/', 'notemtpy', 'anything', null, OCI_CRED_EXT);
Expand Down
4 changes: 1 addition & 3 deletions ext/oci8/tests/extauth_03.phpt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,16 +3,14 @@ Test External Authentication errors with oci_pconnect
--SKIPIF--
<?php
if (!extension_loaded('oci8')) die ("skip no oci8 extension");
if (getenv('SKIP_SLOW_TESTS')) die('skip slow tests excluded by request');
if (substr(PHP_OS, 0, 3) == 'WIN') die("skip feature not available on Windows platforms");
die("skip random CI timeouts caused by Oracle Instant Client, see https://github.com/php/php-src/pull/9524#issuecomment-1244409815");
?>
--INI--
oci8.privileged_connect=1
--FILE--
<?php

// Run Test

echo "Test 1\n";

$c = oci_pconnect('/', 'notemtpy', 'anything', null, OCI_CRED_EXT);
Expand Down
4 changes: 2 additions & 2 deletions ext/oci8/tests/extauth_04.phpt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,11 +5,11 @@ Test External Authentication errors on Windows
if (!extension_loaded('oci8')) die ("skip no oci8 extension");
if (substr(PHP_OS, 0, 3) != 'WIN') die("skip this test is for Windows platforms only");
?>
--INI--
oci8.privileged_connect=1
--FILE--
<?php

// Run Test

echo "Test 1\n";

$c = oci_connect('/', '', 'anything', null, OCI_CRED_EXT);
Expand Down