From 52e70156441e68e66a87619a27687f1ec470efc2 Mon Sep 17 00:00:00 2001 From: Matthieu Napoli Date: Wed, 14 Aug 2024 16:20:32 +0200 Subject: [PATCH] Fix cs/phpstan --- src/FpmRuntime/FpmHandler.php | 2 ++ tests/FpmRuntime/FpmHandlerTest.php | 4 ++-- tests/FpmRuntime/fixtures/large-logs.php | 2 +- 3 files changed, 5 insertions(+), 3 deletions(-) diff --git a/src/FpmRuntime/FpmHandler.php b/src/FpmRuntime/FpmHandler.php index b86e24921..caa1ef145 100644 --- a/src/FpmRuntime/FpmHandler.php +++ b/src/FpmRuntime/FpmHandler.php @@ -351,8 +351,10 @@ public function stopFpm(float $timeout): void @posix_kill($pid, 15); do { usleep(1000); + // @phpstan-ignore-next-line } while ($this->isFpmRunning() && microtime(true) < $timeoutMicro); + // @phpstan-ignore-next-line if ($this->isFpmRunning()) { // SIGKILL @posix_kill($pid, 9); diff --git a/tests/FpmRuntime/FpmHandlerTest.php b/tests/FpmRuntime/FpmHandlerTest.php index 7f298bdc2..9ae4e7ce0 100644 --- a/tests/FpmRuntime/FpmHandlerTest.php +++ b/tests/FpmRuntime/FpmHandlerTest.php @@ -1255,8 +1255,8 @@ public function test worker logs are still written in case of a timeou $this->fail('No exception was thrown'); } catch (Timeout $e) { $this->markTestSkipped('We can no longer test this since proc_open skips output buffering, we cannot capture the logs'); - $logs = ob_get_contents(); - self::assertStringContainsString('This is a log message', $logs); +// $logs = ob_get_contents(); +// self::assertStringContainsString('This is a log message', $logs); } } diff --git a/tests/FpmRuntime/fixtures/large-logs.php b/tests/FpmRuntime/fixtures/large-logs.php index d937bfaa6..154ebd984 100644 --- a/tests/FpmRuntime/fixtures/large-logs.php +++ b/tests/FpmRuntime/fixtures/large-logs.php @@ -1,7 +1,7 @@