Skip to content

Commit 4ed01e3

Browse files
committed
Fix run-tests.php for explicitly given test cases
The recent improvement to list skipped extensions explicitly[1] missed to properly initialize `$ignored_by_ext` for the case where an explicit set of test cases are given; this was not a problem previously, since the undefined *global* variable was coerced to int. We fix this by initializing the variable earlier. [1] <baef47e> Closes GH-9617.
1 parent 01677ca commit 4ed01e3

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

run-tests.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -362,6 +362,7 @@ function main(): void
362362
$context_line_count = 3;
363363
$num_repeats = 1;
364364
$show_progress = true;
365+
$ignored_by_ext = [];
365366

366367
$cfgtypes = ['show', 'keep'];
367368
$cfgfiles = ['skip', 'php', 'clean', 'out', 'diff', 'exp', 'mem'];
@@ -735,7 +736,6 @@ function main(): void
735736
$test_files = [];
736737
$exts_tested = $exts_to_test;
737738
$exts_skipped = [];
738-
$ignored_by_ext = [];
739739
sort($exts_to_test);
740740
$test_dirs = [];
741741
$optionals = ['Zend', 'tests', 'ext', 'sapi'];

0 commit comments

Comments
 (0)