From 878f01f85a066f9c929ca97fe9fb0ea2860d54cf Mon Sep 17 00:00:00 2001 From: sasezaki Date: Sat, 4 Oct 2025 22:15:50 +0900 Subject: [PATCH 1/2] Apply phpcbf & fix for Line exceeds 120 characters --- tests/Test/TestLoggerTest.php | 42 ++++++++++++++++++++++------------- 1 file changed, 27 insertions(+), 15 deletions(-) diff --git a/tests/Test/TestLoggerTest.php b/tests/Test/TestLoggerTest.php index 05cc47a..4424af6 100644 --- a/tests/Test/TestLoggerTest.php +++ b/tests/Test/TestLoggerTest.php @@ -52,31 +52,43 @@ public function testThrowsOnInvalidLevel(): void */ public function testHasRecord(string $level): void { - $levelMethod = 'has'.ucfirst($level); + $levelMethod = 'has' . ucfirst($level); $logger = $this->getLogger(); - $this->assertFalse(call_user_func([$logger, $levelMethod.'Records'])); + $this->assertFalse(call_user_func([$logger, $levelMethod . 'Records'])); - $logger->log($level, $level.' Message', ['foo' => 'bar']); - $logger->log($level, $level.' Hello'); + $logger->log($level, $level . ' Message', ['foo' => 'bar']); + $logger->log($level, $level . ' Hello'); - $this->assertTrue(call_user_func([$logger, $levelMethod.'Records'])); + $this->assertTrue(call_user_func([$logger, $levelMethod . 'Records'])); - $record = $level.' Message'; + $record = $level . ' Message'; $this->assertTrue($logger->hasRecord($record, $level), 'hasRecord without context'); - $this->assertTrue(call_user_func([$logger, $levelMethod], $record), $levelMethod.' without context'); + $this->assertTrue(call_user_func([$logger, $levelMethod], $record), $levelMethod . ' without context'); - $record = ['message' => $level.' Message']; + $record = ['message' => $level . ' Message']; $this->assertTrue($logger->hasRecord($record, $level), 'hasRecord without context'); - $this->assertTrue(call_user_func([$logger, $levelMethod], $record), $levelMethod.' without context'); + $this->assertTrue(call_user_func([$logger, $levelMethod], $record), $levelMethod . ' without context'); - $record = ['message' => $level.' Message', ['foo' => 'bar']]; + $record = ['message' => $level . ' Message', ['foo' => 'bar']]; $this->assertTrue($logger->hasRecord($record, $level), 'hasRecord with context'); - $this->assertTrue(call_user_func([$logger, $levelMethod], $record), $levelMethod.' with context'); - - $this->assertTrue(call_user_func([$logger, $levelMethod.'ThatContains'], 'Message'), $levelMethod.'ThatContains'); - $this->assertTrue(call_user_func([$logger, $levelMethod.'ThatMatches'], '/^[a-z]+ Message$/i'), $levelMethod.'ThatMatches'); - $this->assertTrue(call_user_func([$logger, $levelMethod.'ThatPasses'], fn (array $record) => $record['message'] === $level.' Message'), $levelMethod.'ThatMatches'); + $this->assertTrue(call_user_func([$logger, $levelMethod], $record), $levelMethod . ' with context'); + + + $this->assertTrue( + call_user_func([$logger, $levelMethod . 'ThatContains'], 'Message'), + $levelMethod . 'ThatContains' + ); + $this->assertTrue( + call_user_func([$logger, $levelMethod . 'ThatMatches'], '/^[a-z]+ Message$/i'), + $levelMethod . 'ThatMatches' + ); + + $passesCallback = fn (array $record) => $record['message'] === $level . ' Message'; + $this->assertTrue( + call_user_func([$logger, $levelMethod . 'ThatPasses'], $passesCallback), + $levelMethod . 'ThatPasses' + ); } public static function getLogLevels(): array From f5b386aefd69ab40fd7c013bcb999c68595c3680 Mon Sep 17 00:00:00 2001 From: sasezaki Date: Sat, 4 Oct 2025 22:17:11 +0900 Subject: [PATCH 2/2] Removed useless installed_paths setting --- phpcs.xml.dist | 2 -- 1 file changed, 2 deletions(-) diff --git a/phpcs.xml.dist b/phpcs.xml.dist index 083ab35..b2d519e 100644 --- a/phpcs.xml.dist +++ b/phpcs.xml.dist @@ -5,7 +5,5 @@ tests - -