]> granicus.if.org Git - php/commitdiff
Try to make regression test for Bug #69900 consistent
authorAlex Dowad <alexinbeijing@gmail.com>
Wed, 22 Apr 2020 18:45:25 +0000 (20:45 +0200)
committerMáté Kocsis <kocsismate@woohoolabs.com>
Fri, 1 May 2020 10:25:30 +0000 (12:25 +0200)
It has been observed that in rare cases, this regression test has spurious failures in CI.
Try increasing the threshold for failure a bit and see if this makes it pass consistently.

ext/standard/tests/streams/proc_open_bug69900.phpt

index de13d1bfdd37ee526ddccea68d19d69cac2f6d3b..28a532dea9e9964a8ec8a81488035077a4cb00f5 100644 (file)
@@ -6,7 +6,7 @@ Bug #69900 Commandline input/output weird behaviour with STDIO
 error_reporting(E_ALL);
 
 $fl = __DIR__ . DIRECTORY_SEPARATOR . "test69900.php";
-$max_ms = 10;
+$max_ms = 20;
 
 $test_content = '<?php
 
@@ -48,8 +48,8 @@ fclose($pipes[1]);
 proc_close($process);
 
 /* It is expected that the first call takes more than the limit.
- * Allow one more to account for a possible process switch. */
-if ($moreThanLimit > 2) {
+ * Allow two more to account for possible process switch, etc. */
+if ($moreThanLimit > 3) {
     echo "fgets() took more than $max_ms ms $moreThanLimit times\n";
 }