replace PHPUnit annotations with attributes

Author: xabbuh

Tests/ClockMockTest.php

@@ -11,6 +11,7 @@
 
 namespace Symfony\Bridge\PhpUnit\Tests;
 
+use PHPUnit\Framework\Attributes\CoversClass;
 use PHPUnit\Framework\TestCase;
 use Symfony\Bridge\PhpUnit\ClockMock;
 
@@ -19,6 +20,7 @@
  *
  * @covers \Symfony\Bridge\PhpUnit\ClockMock
  */
+#[CoversClass(ClockMock::class)]
 class ClockMockTest extends TestCase
 {
     public static function setUpBeforeClass(): void

Tests/DeprecationErrorHandler/ConfigurationTest.php

@@ -11,6 +11,7 @@
 
 namespace Symfony\Bridge\PhpUnit\Tests\DeprecationErrorHandler;
 
+use PHPUnit\Framework\Attributes\DataProvider;
 use PHPUnit\Framework\Attributes\RequiresPhpunit;
 use PHPUnit\Framework\TestCase;
 use Symfony\Bridge\PhpUnit\DeprecationErrorHandler\Configuration;
@@ -194,6 +195,7 @@ public static function provideItCanBeDisabled(): array
     /**
      * @dataProvider provideItCanBeDisabled
      */
+    #[DataProvider('provideItCanBeDisabled')]
     public function testItCanBeDisabled(string $encodedString, bool $expectedEnabled)
     {
         $configuration = Configuration::fromUrlEncodedString($encodedString);
@@ -240,6 +242,7 @@ public function testOutputIsNotVerboseInWeakMode()
     /**
      * @dataProvider provideDataForToleratesForGroup
      */
+    #[DataProvider('provideDataForToleratesForGroup')]
     public function testToleratesForIndividualGroups(string $deprecationsHelper, array $deprecationsPerType, array $expected)
     {
         $configuration = Configuration::fromUrlEncodedString($deprecationsHelper);

Tests/DeprecationErrorHandler/DeprecationTest.php

@@ -11,6 +11,7 @@
 
 namespace Symfony\Bridge\PhpUnit\Tests\DeprecationErrorHandler;
 
+use PHPUnit\Framework\Attributes\DataProvider;
 use PHPUnit\Framework\TestCase;
 use Symfony\Bridge\PhpUnit\DeprecationErrorHandler;
 use Symfony\Bridge\PhpUnit\DeprecationErrorHandler\Deprecation;
@@ -88,6 +89,7 @@ public function testItRulesOutFilesOutsideVendorsAsIndirect()
     /**
      * @dataProvider mutedProvider
      */
+    #[DataProvider('mutedProvider')]
     public function testItMutesOnlySpecificErrorMessagesWhenTheCallingCodeIsInPhpunit($muted, $callingClass, $message)
     {
         $trace = $this->debugBacktrace();
@@ -170,6 +172,7 @@ public static function providerGetTypeDetectsSelf(): array
     /**
      * @dataProvider providerGetTypeDetectsSelf
      */
+    #[DataProvider('providerGetTypeDetectsSelf')]
     public function testGetTypeDetectsSelf(string $expectedType, string $message, string $traceClass, string $file)
     {
         $trace = [
@@ -233,6 +236,7 @@ public static function providerGetTypeUsesRightTrace(): array
     /**
      * @dataProvider providerGetTypeUsesRightTrace
      */
+    #[DataProvider('providerGetTypeUsesRightTrace')]
     public function testGetTypeUsesRightTrace(string $expectedType, string $message, array $trace)
     {
         $deprecation = new Deprecation(

Tests/ExpectDeprecationTraitTest.php

@@ -11,7 +11,9 @@
 
 namespace Symfony\Bridge\PhpUnit\Tests;
 
+use PHPUnit\Framework\Attributes\Group;
 use PHPUnit\Framework\Attributes\RequiresPhpunit;
+use PHPUnit\Framework\Attributes\RunInSeparateProcess;
 use PHPUnit\Framework\TestCase;
 use Symfony\Bridge\PhpUnit\ExpectDeprecationTrait;
 
@@ -25,6 +27,7 @@ final class ExpectDeprecationTraitTest extends TestCase
      *
      * @group legacy
      */
+    #[Group('legacy')]
     public function testOne()
     {
         $this->expectDeprecation('foo');
@@ -38,6 +41,8 @@ public function testOne()
      *
      * @runInSeparateProcess
      */
+    #[Group('legacy')]
+    #[RunInSeparateProcess]
     public function testOneInIsolation()
     {
         $this->expectDeprecation('foo');
@@ -49,6 +54,7 @@ public function testOneInIsolation()
      *
      * @group legacy
      */
+    #[Group('legacy')]
     public function testMany()
     {
         $this->expectDeprecation('foo');
@@ -64,6 +70,7 @@ public function testMany()
      *
      * @expectedDeprecation foo
      */
+    #[Group('legacy')]
     public function testOneWithAnnotation()
     {
         $this->expectDeprecation('bar');
@@ -79,6 +86,7 @@ public function testOneWithAnnotation()
      * @expectedDeprecation foo
      * @expectedDeprecation bar
      */
+    #[Group('legacy')]
     public function testManyWithAnnotation()
     {
         $this->expectDeprecation('ccc');

Tests/ExpectedDeprecationAnnotationTest.php

@@ -11,6 +11,7 @@
 
 namespace Symfony\Bridge\PhpUnit\Tests;
 
+use PHPUnit\Framework\Attributes\Group;
 use PHPUnit\Framework\Attributes\RequiresPhpunit;
 use PHPUnit\Framework\TestCase;
 
@@ -24,6 +25,7 @@ final class ExpectedDeprecationAnnotationTest extends TestCase
      *
      * @expectedDeprecation foo
      */
+    #[Group('legacy')]
     public function testOne()
     {
         @trigger_error('foo', \E_USER_DEPRECATED);
@@ -37,6 +39,7 @@ public function testOne()
      * @expectedDeprecation foo
      * @expectedDeprecation bar
      */
+    #[Group('legacy')]
     public function testMany()
     {
         @trigger_error('foo', \E_USER_DEPRECATED);

Tests/FailTests/NoAssertionsTestRisky.php

@@ -11,6 +11,7 @@
 
 namespace Symfony\Bridge\PhpUnit\Tests\FailTests;
 
+use PHPUnit\Framework\Attributes\Group;
 use PHPUnit\Framework\Attributes\RequiresPhpunit;
 use PHPUnit\Framework\TestCase;
 use Symfony\Bridge\PhpUnit\ExpectDeprecationTrait;
@@ -29,6 +30,7 @@ final class NoAssertionsTestRisky extends TestCase
      *
      * @group legacy
      */
+    #[Group('legacy')]
     public function testOne()
     {
         $this->expectNotToPerformAssertions();

Tests/Fixtures/coverage/tests/CoversDefaultClassTest.php

@@ -9,11 +9,13 @@
  * file that was distributed with this source code.
  */
 
+use PHPUnit\Framework\Attributes\CoversClass;
 use PHPUnit\Framework\TestCase;
 
 /**
  * @coversDefaultClass \DateTime
  */
+#[CoversClass(DateTime::class)]
 class CoversDefaultClassTest extends TestCase
 {
     public function test()

Tests/Fixtures/coverage/tests/CoversNothingTest.php

@@ -9,11 +9,13 @@
  * file that was distributed with this source code.
  */
 
+use PHPUnit\Framework\Attributes\CoversNothing;
 use PHPUnit\Framework\TestCase;
 
 /**
  * @coversNothing
  */
+#[CoversNothing]
 class CoversNothingTest extends TestCase
 {
     public function test()

Tests/Fixtures/coverage/tests/CoversTest.php

@@ -9,13 +9,15 @@
  * file that was distributed with this source code.
  */
 
+use PHPUnit\Framework\Attributes\CoversClass;
 use PHPUnit\Framework\TestCase;
 
+/**
+ * @covers \DateTime
+ */
+#[CoversClass(DateTime::class)]
 class CoversTest extends TestCase
 {
-    /**
-     * @covers \DateTime
-     */
     public function test()
     {
         $this->assertTrue(true);

Tests/Metadata/AttributeReaderTest.php

@@ -11,6 +11,7 @@
 
 namespace Symfony\Bridge\PhpUnit\Tests\Metadata;
 
+use PHPUnit\Framework\Attributes\DataProvider;
 use PHPUnit\Framework\TestCase;
 use Symfony\Bridge\PhpUnit\Attribute\DnsSensitive;
 use Symfony\Bridge\PhpUnit\Attribute\TimeSensitive;
@@ -22,6 +23,7 @@ class AttributeReaderTest extends TestCase
     /**
      * @dataProvider provideReadCases
      */
+    #[DataProvider('provideReadCases')]
     public function testAttributesAreRead(string $method, string $attributeClass, array $expected)
     {
         $reader = new AttributeReader();