From: =?utf-8?q?David_Pr=C3=A9vot?= <taffit@debian.org>
Date: Fri, 28 Feb 2025 13:19:29 +0100
Subject: Modernize PHPUnit syntax

---
 tests/Monolog/Formatter/GoogleCloudLoggingFormatterTest.php |  7 +++----
 tests/Monolog/Handler/DeduplicationHandlerTest.php          |  5 +++--
 tests/Monolog/Handler/FlowdockHandlerTest.php               |  5 ++---
 tests/Monolog/Handler/OverflowHandlerTest.php               |  9 +++------
 tests/Monolog/Handler/PushoverHandlerTest.php               |  5 ++---
 tests/Monolog/SignalHandlerTest.php                         | 11 ++++++-----
 6 files changed, 19 insertions(+), 23 deletions(-)

diff --git a/tests/Monolog/Formatter/GoogleCloudLoggingFormatterTest.php b/tests/Monolog/Formatter/GoogleCloudLoggingFormatterTest.php
index 29c776e..b52b2c9 100644
--- a/tests/Monolog/Formatter/GoogleCloudLoggingFormatterTest.php
+++ b/tests/Monolog/Formatter/GoogleCloudLoggingFormatterTest.php
@@ -13,16 +13,16 @@ namespace Monolog\Formatter;
 
 use DateTimeInterface;
 use Monolog\Test\MonologTestCase;
+use PHPUnit\Framework\Attributes\Test;
 use function json_decode;
 
 class GoogleCloudLoggingFormatterTest extends MonologTestCase
 {
     /**
-     * @test
-     *
      * @covers \Monolog\Formatter\JsonFormatter
      * @covers \Monolog\Formatter\GoogleCloudLoggingFormatter::normalizeRecord
      */
+    #[Test]
     public function formatProvidesRfc3339Timestamps(): void
     {
         $formatter = new GoogleCloudLoggingFormatter();
@@ -35,11 +35,10 @@ class GoogleCloudLoggingFormatterTest extends MonologTestCase
     }
 
     /**
-     * @test
-     *
      * @covers \Monolog\Formatter\JsonFormatter
      * @covers \Monolog\Formatter\GoogleCloudLoggingFormatter::normalizeRecord
      */
+    #[Test]
     public function formatIntroducesLogSeverity(): void
     {
         $formatter = new GoogleCloudLoggingFormatter();
diff --git a/tests/Monolog/Handler/DeduplicationHandlerTest.php b/tests/Monolog/Handler/DeduplicationHandlerTest.php
index 100bbd5..207f5a0 100644
--- a/tests/Monolog/Handler/DeduplicationHandlerTest.php
+++ b/tests/Monolog/Handler/DeduplicationHandlerTest.php
@@ -12,6 +12,7 @@
 namespace Monolog\Handler;
 
 use Monolog\Level;
+use PHPUnit\Framework\Attributes\Depends;
 
 class DeduplicationHandlerTest extends \Monolog\Test\MonologTestCase
 {
@@ -58,8 +59,8 @@ class DeduplicationHandlerTest extends \Monolog\Test\MonologTestCase
      * @covers Monolog\Handler\DeduplicationHandler::flush
      * @covers Monolog\Handler\DeduplicationHandler::appendRecord
      * @covers Monolog\Handler\DeduplicationHandler::isDuplicate
-     * @depends testFlushPassthruIfEmptyLog
      */
+    #[Depends('testFlushPassthruIfEmptyLog')]
     public function testFlushSkipsIfLogExists()
     {
         $test = new TestHandler();
@@ -79,8 +80,8 @@ class DeduplicationHandlerTest extends \Monolog\Test\MonologTestCase
      * @covers Monolog\Handler\DeduplicationHandler::flush
      * @covers Monolog\Handler\DeduplicationHandler::appendRecord
      * @covers Monolog\Handler\DeduplicationHandler::isDuplicate
-     * @depends testFlushPassthruIfEmptyLog
      */
+    #[Depends('testFlushPassthruIfEmptyLog')]
     public function testFlushPassthruIfLogTooOld()
     {
         $test = new TestHandler();
diff --git a/tests/Monolog/Handler/FlowdockHandlerTest.php b/tests/Monolog/Handler/FlowdockHandlerTest.php
index 2fa7927..9eed100 100644
--- a/tests/Monolog/Handler/FlowdockHandlerTest.php
+++ b/tests/Monolog/Handler/FlowdockHandlerTest.php
@@ -13,6 +13,7 @@ namespace Monolog\Handler;
 
 use Monolog\Formatter\FlowdockFormatter;
 use Monolog\Level;
+use PHPUnit\Framework\Attributes\Depends;
 
 /**
  * @author Dominik Liebler <liebler.dominik@gmail.com>
@@ -54,9 +55,7 @@ class FlowdockHandlerTest extends \Monolog\Test\MonologTestCase
         return $content;
     }
 
-    /**
-     * @depends testWriteHeader
-     */
+    #[Depends('testWriteHeader')]
     public function testWriteContent($content)
     {
         $this->assertMatchesRegularExpression('/"source":"test_source"/', $content);
diff --git a/tests/Monolog/Handler/OverflowHandlerTest.php b/tests/Monolog/Handler/OverflowHandlerTest.php
index cb62ba4..4f7fe4f 100644
--- a/tests/Monolog/Handler/OverflowHandlerTest.php
+++ b/tests/Monolog/Handler/OverflowHandlerTest.php
@@ -12,6 +12,7 @@
 namespace Monolog\Handler;
 
 use Monolog\Level;
+use PHPUnit\Framework\Attributes\Test;
 
 /**
  * @author Kris Buist <krisbuist@gmail.com>
@@ -41,9 +42,7 @@ class OverflowHandlerTest extends \Monolog\Test\MonologTestCase
         $this->assertTrue($testHandler->hasWarningThatContains('Warning 1'));
     }
 
-    /**
-     * @test
-     */
+    #[Test]
     public function testHoldingMessagesBeneathThreshold()
     {
         $testHandler = new TestHandler();
@@ -67,9 +66,7 @@ class OverflowHandlerTest extends \Monolog\Test\MonologTestCase
         }
     }
 
-    /**
-     * @test
-     */
+    #[Test]
     public function testCombinedThresholds()
     {
         $testHandler = new TestHandler();
diff --git a/tests/Monolog/Handler/PushoverHandlerTest.php b/tests/Monolog/Handler/PushoverHandlerTest.php
index e2ed1dd..a84f1da 100644
--- a/tests/Monolog/Handler/PushoverHandlerTest.php
+++ b/tests/Monolog/Handler/PushoverHandlerTest.php
@@ -12,6 +12,7 @@
 namespace Monolog\Handler;
 
 use Monolog\Level;
+use PHPUnit\Framework\Attributes\Depends;
 use PHPUnit\Framework\MockObject\MockObject;
 
 /**
@@ -47,9 +48,7 @@ class PushoverHandlerTest extends \Monolog\Test\MonologTestCase
         return $content;
     }
 
-    /**
-     * @depends testWriteHeader
-     */
+    #[Depends('testWriteHeader')]
     public function testWriteContent($content)
     {
         $this->assertMatchesRegularExpression('/token=myToken&user=myUser&message=test1&title=Monolog&timestamp=\d{10}$/', $content);
diff --git a/tests/Monolog/SignalHandlerTest.php b/tests/Monolog/SignalHandlerTest.php
index b7c6cae..d7a19bc 100644
--- a/tests/Monolog/SignalHandlerTest.php
+++ b/tests/Monolog/SignalHandlerTest.php
@@ -15,6 +15,7 @@ use Monolog\Handler\StreamHandler;
 use Monolog\Handler\TestHandler;
 use Monolog\Test\MonologTestCase;
 use PHPUnit\Framework\Attributes\DataProvider;
+use PHPUnit\Framework\Attributes\Depends;
 use Psr\Log\LogLevel;
 
 /**
@@ -84,7 +85,6 @@ class SignalHandlerTest extends MonologTestCase
     }
 
     /**
-     * @depends testHandleSignal
      * @requires extension pcntl
      * @requires extension posix
      * @requires function pcntl_signal
@@ -92,6 +92,7 @@ class SignalHandlerTest extends MonologTestCase
      * @requires function posix_getpid
      * @requires function posix_kill
      */
+    #[Depends('testHandleSignal')]
     public function testRegisterSignalHandler()
     {
         // SIGCONT and SIGURG should be ignored by default.
@@ -123,12 +124,12 @@ class SignalHandlerTest extends MonologTestCase
     }
 
     /**
-     * @depends testRegisterSignalHandler
      * @requires function pcntl_fork
      * @requires function pcntl_sigprocmask
      * @requires function pcntl_waitpid
      */
     #[DataProvider('defaultPreviousProvider')]
+    #[Depends('testRegisterSignalHandler')]
     public function testRegisterDefaultPreviousSignalHandler($signo, $callPrevious, $expected)
     {
         $this->setSignalHandler($signo, SIG_DFL);
@@ -175,10 +176,10 @@ class SignalHandlerTest extends MonologTestCase
     }
 
     /**
-     * @depends testRegisterSignalHandler
      * @requires function pcntl_signal_get_handler
      */
     #[DataProvider('callablePreviousProvider')]
+    #[Depends('testRegisterSignalHandler')]
     public function testRegisterCallablePreviousSignalHandler($callPrevious)
     {
         $this->setSignalHandler(SIGURG, SIG_IGN);
@@ -206,11 +207,11 @@ class SignalHandlerTest extends MonologTestCase
     }
 
     /**
-     * @depends testRegisterDefaultPreviousSignalHandler
      * @requires function pcntl_fork
      * @requires function pcntl_waitpid
      */
     #[DataProvider('restartSyscallsProvider')]
+    #[Depends('testRegisterDefaultPreviousSignalHandler')]
     public function testRegisterSyscallRestartingSignalHandler($restartSyscalls)
     {
         $this->setSignalHandler(SIGURG, SIG_IGN);
@@ -260,10 +261,10 @@ class SignalHandlerTest extends MonologTestCase
     }
 
     /**
-     * @depends testRegisterDefaultPreviousSignalHandler
      * @requires function pcntl_async_signals
      */
     #[DataProvider('asyncProvider')]
+    #[Depends('testRegisterDefaultPreviousSignalHandler')]
     public function testRegisterAsyncSignalHandler($initialAsync, $desiredAsync, $expectedBefore, $expectedAfter)
     {
         $this->setSignalHandler(SIGURG, SIG_IGN);