minor #38798 Fix transient tests on branch 4.4 (jderusse)

nicolas-grekas · nicolas-grekas · commit bc94b6223613 · 2020-10-27T11:08:12.000+01:00
This PR was merged into the 4.4 branch. Discussion ---------- Fix transient tests on branch 4.4 | Q | A | ------------- | --- | Branch? | 4.4 | Bug fix? | yes | New feature? | no | Deprecations? | no | Tickets | - | License | MIT | Doc PR | - Fix transient tests on branch 4.4. I'll keep this PR open few days to watch travis/GHA for transient tests Commits ------- 676b808 Fix transient tests
diff --git a/src/Symfony/Component/Messenger/Tests/Transport/AmqpExt/AmqpExtIntegrationTest.php b/src/Symfony/Component/Messenger/Tests/Transport/AmqpExt/AmqpExtIntegrationTest.php
@@ -161,6 +161,9 @@ public function testItReceivesSignals()
         $signalTime = microtime(true);
         $timedOutTime = time() + 10;
 
+        // wait for worker started and registered the signal handler
+        usleep(100 * 1000); // 100ms
+
         // immediately after the process has started "booted", kill it
         $process->signal(15);
 
diff --git a/src/Symfony/Component/VarDumper/Tests/Dumper/ServerDumperTest.php b/src/Symfony/Component/VarDumper/Tests/Dumper/ServerDumperTest.php
@@ -89,6 +89,6 @@ private function getServerProcess(): Process
             'VAR_DUMPER_SERVER' => self::VAR_DUMPER_SERVER,
         ]);
 
-        return $process->setTimeout(9);
+        return $process->setTimeout('\\' === \DIRECTORY_SEPARATOR ? 19 : 9);
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -89,6 +89,6 @@ private function getServerProcess(): Process`
`89`	`89`	`'VAR_DUMPER_SERVER' => self::VAR_DUMPER_SERVER,`
`90`	`90`	`]);`
`91`	`91`
`92`		`- return $process->setTimeout(9);`
	`92`	`+ return $process->setTimeout('\\' === \DIRECTORY_SEPARATOR ? 19 : 9);`
`93`	`93`	`}`
`94`	`94`	`}`