From: Alex Dowad <alexinbeijing@gmail.com>
Date: Wed, 22 Apr 2020 18:45:25 +0000 (+0200)
Subject: Try to make regression test for Bug #69900 consistent
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=86ed93c23085f7c2f189d7bbc78efe8f76981e8e;p=php

Try to make regression test for Bug #69900 consistent

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.
---

diff --git a/ext/standard/tests/streams/proc_open_bug69900.phpt b/ext/standard/tests/streams/proc_open_bug69900.phpt
index de13d1bfdd..28a532dea9 100644
--- a/ext/standard/tests/streams/proc_open_bug69900.phpt
+++ b/ext/standard/tests/streams/proc_open_bug69900.phpt
@@ -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";
 }