refactor: change case from IgnorePHPUnitWarnings to IgnorePhpunitWarnings

nikophil · sebastianbergmann · commit 52384ee1b1ea · 2025-07-08T10:52:41.000+02:00
diff --git a/src/Event/Emitter/DispatchingEmitter.php b/src/Event/Emitter/DispatchingEmitter.php
@@ -29,7 +29,7 @@
 use PHPUnit\Event\TestSuite\Started as TestSuiteStarted;
 use PHPUnit\Event\TestSuite\TestSuite;
 use PHPUnit\Framework\TestCase;
-use PHPUnit\Metadata\IgnorePHPUnitWarnings;
+use PHPUnit\Metadata\IgnorePhpunitWarnings;
 use PHPUnit\Metadata\Parser\Registry;
 use PHPUnit\TextUI\Configuration\Configuration;
 use SebastianBergmann\Comparator\Comparator;
@@ -1014,9 +1014,9 @@ public function testTriggeredPhpunitWarning(Code\Test $test, string $message): v
         if (
             $test->isTestMethod() &&
             class_exists($test->className()) &&
-            ($metadata = Registry::parser()->forMethod($test->className(), $test->methodName())->isIgnorePHPUnitWarnings())->isNotEmpty() &&
+            ($metadata = Registry::parser()->forMethod($test->className(), $test->methodName())->isIgnorePhpunitWarnings())->isNotEmpty() &&
             ($ignoreWarnings = $metadata->asArray()[0] ?? null) !== null &&
-            $ignoreWarnings instanceof IgnorePHPUnitWarnings &&
+            $ignoreWarnings instanceof IgnorePhpunitWarnings &&
             $ignoreWarnings->shouldIgnore($message)
         ) {
             return;
diff --git a/src/Framework/Attributes/IgnorePhpunitWarnings.php b/src/Framework/Attributes/IgnorePhpunitWarnings.php
@@ -17,7 +17,7 @@
  * @no-named-arguments Parameter names are not covered by the backward compatibility promise for PHPUnit
  */
 #[Attribute(Attribute::TARGET_METHOD)]
-final readonly class IgnorePHPUnitWarnings
+final readonly class IgnorePhpunitWarnings
 {
     /** @var null|non-empty-string */
     private ?string $messagePattern;
diff --git a/src/Metadata/IgnorePhpunitWarnings.php b/src/Metadata/IgnorePhpunitWarnings.php
@@ -16,7 +16,7 @@
  *
  * @no-named-arguments Parameter names are not covered by the backward compatibility promise for PHPUnit
  */
-final readonly class IgnorePHPUnitWarnings extends Metadata
+final readonly class IgnorePhpunitWarnings extends Metadata
 {
     /** @var null|non-empty-string */
     private ?string $messagePattern;
@@ -32,7 +32,7 @@ protected function __construct(int $level, null|string $messagePattern)
         $this->messagePattern = $messagePattern;
     }
 
-    public function isIgnorePHPUnitWarnings(): true
+    public function isIgnorePhpunitWarnings(): true
     {
         return true;
     }
diff --git a/src/Metadata/Metadata.php b/src/Metadata/Metadata.php
@@ -537,9 +537,9 @@ public static function withoutErrorHandler(): WithoutErrorHandler
     /**
      * @param null|non-empty-string $messagePattern
      */
-    public static function ignorePHPUnitWarnings(?string $messagePattern): IgnorePHPUnitWarnings
+    public static function ignorePhpunitWarnings(?string $messagePattern): IgnorePhpunitWarnings
     {
-        return new IgnorePHPUnitWarnings(self::METHOD_LEVEL, $messagePattern);
+        return new IgnorePhpunitWarnings(self::METHOD_LEVEL, $messagePattern);
     }
 
     /**
@@ -979,9 +979,9 @@ public function isWithoutErrorHandler(): bool
     }
 
     /**
-     * @phpstan-assert-if-true IgnorePHPUnitWarnings $this
+     * @phpstan-assert-if-true IgnorePhpunitWarnings $this
      */
-    public function isIgnorePHPUnitWarnings(): bool
+    public function isIgnorePhpunitWarnings(): bool
     {
         return false;
     }
diff --git a/src/Metadata/MetadataCollection.php b/src/Metadata/MetadataCollection.php
@@ -641,12 +641,12 @@ public function isWithoutErrorHandler(): self
         );
     }
 
-    public function isIgnorePHPUnitWarnings(): self
+    public function isIgnorePhpunitWarnings(): self
     {
         return new self(
             ...array_filter(
                 $this->metadata,
-                static fn (Metadata $metadata): bool => $metadata->isIgnorePHPUnitWarnings(),
+                static fn (Metadata $metadata): bool => $metadata->isIgnorePhpunitWarnings(),
             ),
         );
     }
diff --git a/src/Metadata/Parser/AttributeParser.php b/src/Metadata/Parser/AttributeParser.php
@@ -52,7 +52,7 @@
 use PHPUnit\Framework\Attributes\Group;
 use PHPUnit\Framework\Attributes\IgnoreDeprecations;
 use PHPUnit\Framework\Attributes\IgnorePhpunitDeprecations;
-use PHPUnit\Framework\Attributes\IgnorePHPUnitWarnings;
+use PHPUnit\Framework\Attributes\IgnorePhpunitWarnings;
 use PHPUnit\Framework\Attributes\Large;
 use PHPUnit\Framework\Attributes\Medium;
 use PHPUnit\Framework\Attributes\PostCondition;
@@ -885,10 +885,10 @@ public function forMethod(string $className, string $methodName): MetadataCollec
 
                     break;
 
-                case IgnorePHPUnitWarnings::class:
-                    assert($attributeInstance instanceof IgnorePHPUnitWarnings);
+                case IgnorePhpunitWarnings::class:
+                    assert($attributeInstance instanceof IgnorePhpunitWarnings);
 
-                    $result[] = Metadata::ignorePHPUnitWarnings($attributeInstance->messagePattern());
+                    $result[] = Metadata::ignorePhpunitWarnings($attributeInstance->messagePattern());
 
                     break;
             }
diff --git a/tests/_files/Metadata/Attribute/tests/IgnorePhpunitWarningsTest.php b/tests/_files/Metadata/Attribute/tests/IgnorePhpunitWarningsTest.php
@@ -9,26 +9,14 @@
  */
 namespace PHPUnit\TestFixture\Metadata\Attribute;
 
-use PHPUnit\Framework\Attributes\IgnorePHPUnitWarnings;
+use PHPUnit\Framework\Attributes\IgnorePhpunitWarnings;
 use PHPUnit\Framework\TestCase;
 
-final class IgnorePHPUnitWarningsTest extends TestCase
+final class IgnorePhpunitWarningsTest extends TestCase
 {
-    #[IgnorePHPUnitWarnings]
+    #[IgnorePhpunitWarnings]
     public function testOne(): void
     {
         $this->assertTrue(true);
     }
-
-    #[IgnorePHPUnitWarnings('warning.*pattern')]
-    public function testTwo(): void
-    {
-        $this->assertTrue(true);
-    }
-
-    #[IgnorePHPUnitWarnings('exact message')]
-    public function testThree(): void
-    {
-        $this->assertTrue(true);
-    }
 }
diff --git a/tests/end-to-end/event/_files/PhpunitWarningIgnoredTest.php b/tests/end-to-end/event/_files/PhpunitWarningIgnoredTest.php
@@ -11,7 +11,7 @@
 
 use PHPUnit\Event\Facade as EventFacade;
 use PHPUnit\Framework\Attributes\DataProvider;
-use PHPUnit\Framework\Attributes\IgnorePHPUnitWarnings;
+use PHPUnit\Framework\Attributes\IgnorePhpunitWarnings;
 use PHPUnit\Framework\TestCase;
 
 final class PhpunitWarningIgnoredTest extends TestCase
@@ -21,7 +21,7 @@ public static function dataProvider(): iterable
         yield [true];
     }
 
-    #[IgnorePHPUnitWarnings]
+    #[IgnorePhpunitWarnings]
     public function testPhpunitWarning(): void
     {
         EventFacade::emitter()->testTriggeredPhpunitWarning(
@@ -32,7 +32,7 @@ public function testPhpunitWarning(): void
         $this->assertTrue(true);
     }
 
-    #[IgnorePHPUnitWarnings('warning message')]
+    #[IgnorePhpunitWarnings('warning message')]
     public function testPhpunitWarningWithExactMessage(): void
     {
         EventFacade::emitter()->testTriggeredPhpunitWarning(
@@ -43,7 +43,7 @@ public function testPhpunitWarningWithExactMessage(): void
         $this->assertTrue(true);
     }
 
-    #[IgnorePHPUnitWarnings('warn(.*)mess(.*)')]
+    #[IgnorePhpunitWarnings('warn(.*)mess(.*)')]
     public function testPhpunitWarningWithRegex(): void
     {
         EventFacade::emitter()->testTriggeredPhpunitWarning(
@@ -54,7 +54,7 @@ public function testPhpunitWarningWithRegex(): void
         $this->assertTrue(true);
     }
 
-    #[IgnorePHPUnitWarnings('warn(.*)mess(.*)')]
+    #[IgnorePhpunitWarnings('warn(.*)mess(.*)')]
     public function testPhpunitWarningWithWrongPattern(): void
     {
         EventFacade::emitter()->testTriggeredPhpunitWarning(
@@ -66,7 +66,7 @@ public function testPhpunitWarningWithWrongPattern(): void
     }
 
     #[DataProvider('dataProvider')]
-    #[IgnorePHPUnitWarnings]
+    #[IgnorePhpunitWarnings]
     public function testTooManyArgumentsInDataProvider(): void
     {
         $this->assertTrue(true);
diff --git a/tests/unit/Metadata/IgnorePhpunitWarningsTest.php b/tests/unit/Metadata/IgnorePhpunitWarningsTest.php
@@ -14,9 +14,9 @@
 use PHPUnit\Framework\Attributes\Small;
 use PHPUnit\Framework\TestCase;
 
-#[CoversClass(IgnorePHPUnitWarnings::class)]
+#[CoversClass(IgnorePhpunitWarnings::class)]
 #[Small]
-final class IgnorePHPUnitWarningsTest extends TestCase
+final class IgnorePhpunitWarningsTest extends TestCase
 {
     public static function shouldIgnoreProvider(): array
     {
@@ -32,7 +32,7 @@ public static function shouldIgnoreProvider(): array
     #[DataProvider('shouldIgnoreProvider')]
     public function testShouldIgnore(?string $messagePattern, string $message, bool $expected): void
     {
-        $metadata = Metadata::ignorePHPUnitWarnings($messagePattern);
+        $metadata = Metadata::ignorePhpunitWarnings($messagePattern);
 
         $this->assertSame($expected, $metadata->shouldIgnore($message));
     }
diff --git a/tests/unit/Metadata/MetadataTest.php b/tests/unit/Metadata/MetadataTest.php
diff --git a/tests/unit/Metadata/Parser/AttributeParserTest.php b/tests/unit/Metadata/Parser/AttributeParserTest.php
diff --git a/tests/unit/Metadata/Parser/AttributeParserTestCase.php b/tests/unit/Metadata/Parser/AttributeParserTestCase.php

Original file line number	Diff line number	Diff line change
`@@ -17,7 +17,7 @@`
`17`	`17`	`* @no-named-arguments Parameter names are not covered by the backward compatibility promise for PHPUnit`
`18`	`18`	`*/`
`19`	`19`	`#[Attribute(Attribute::TARGET_METHOD)]`
`20`		`-final readonly class IgnorePHPUnitWarnings`
	`20`	`+final readonly class IgnorePhpunitWarnings`
`21`	`21`	`{`
`22`	`22`	`/** @var null\|non-empty-string */`
`23`	`23`	`private ?string $messagePattern;`
Original file line number	Diff line number	Diff line change
`@@ -16,7 +16,7 @@`
`16`	`16`	`*`
`17`	`17`	`* @no-named-arguments Parameter names are not covered by the backward compatibility promise for PHPUnit`
`18`	`18`	`*/`
`19`		`-final readonly class IgnorePHPUnitWarnings extends Metadata`
	`19`	`+final readonly class IgnorePhpunitWarnings extends Metadata`
`20`	`20`	`{`
`21`	`21`	`/** @var null\|non-empty-string */`
`22`	`22`	`private ?string $messagePattern;`
`@@ -32,7 +32,7 @@ protected function __construct(int $level, null\|string $messagePattern)`
`32`	`32`	`$this->messagePattern = $messagePattern;`
`33`	`33`	`}`
`34`	`34`
`35`		`- public function isIgnorePHPUnitWarnings(): true`
	`35`	`+ public function isIgnorePhpunitWarnings(): true`
`36`	`36`	`{`
`37`	`37`	`return true;`
`38`	`38`	`}`
Original file line number	Diff line number	Diff line change
`@@ -537,9 +537,9 @@ public static function withoutErrorHandler(): WithoutErrorHandler`
`537`	`537`	`/**`
`538`	`538`	`* @param null\|non-empty-string $messagePattern`
`539`	`539`	`*/`
`540`		`- public static function ignorePHPUnitWarnings(?string $messagePattern): IgnorePHPUnitWarnings`
	`540`	`+ public static function ignorePhpunitWarnings(?string $messagePattern): IgnorePhpunitWarnings`
`541`	`541`	`{`
`542`		`- return new IgnorePHPUnitWarnings(self::METHOD_LEVEL, $messagePattern);`
	`542`	`+ return new IgnorePhpunitWarnings(self::METHOD_LEVEL, $messagePattern);`
`543`	`543`	`}`
`544`	`544`
`545`	`545`	`/**`
`@@ -979,9 +979,9 @@ public function isWithoutErrorHandler(): bool`
`979`	`979`	`}`
`980`	`980`
`981`	`981`	`/**`
`982`		`- * @phpstan-assert-if-true IgnorePHPUnitWarnings $this`
	`982`	`+ * @phpstan-assert-if-true IgnorePhpunitWarnings $this`
`983`	`983`	`*/`
`984`		`- public function isIgnorePHPUnitWarnings(): bool`
	`984`	`+ public function isIgnorePhpunitWarnings(): bool`
`985`	`985`	`{`
`986`	`986`	`return false;`
`987`	`987`	`}`
Original file line number	Diff line number	Diff line change
`@@ -641,12 +641,12 @@ public function isWithoutErrorHandler(): self`
`641`	`641`	`);`
`642`	`642`	`}`
`643`	`643`
`644`		`- public function isIgnorePHPUnitWarnings(): self`
	`644`	`+ public function isIgnorePhpunitWarnings(): self`
`645`	`645`	`{`
`646`	`646`	`return new self(`
`647`	`647`	`...array_filter(`
`648`	`648`	`$this->metadata,`
`649`		`- static fn (Metadata $metadata): bool => $metadata->isIgnorePHPUnitWarnings(),`
	`649`	`+ static fn (Metadata $metadata): bool => $metadata->isIgnorePhpunitWarnings(),`
`650`	`650`	`),`
`651`	`651`	`);`
`652`	`652`	`}`
Original file line number	Diff line number	Diff line change
`@@ -11,7 +11,7 @@`
`11`	`11`
`12`	`12`	`use PHPUnit\Event\Facade as EventFacade;`
`13`	`13`	`use PHPUnit\Framework\Attributes\DataProvider;`
`14`		`-use PHPUnit\Framework\Attributes\IgnorePHPUnitWarnings;`
	`14`	`+use PHPUnit\Framework\Attributes\IgnorePhpunitWarnings;`
`15`	`15`	`use PHPUnit\Framework\TestCase;`
`16`	`16`
`17`	`17`	`final class PhpunitWarningIgnoredTest extends TestCase`
`@@ -21,7 +21,7 @@ public static function dataProvider(): iterable`
`21`	`21`	`yield [true];`
`22`	`22`	`}`
`23`	`23`
`24`		`- #[IgnorePHPUnitWarnings]`
	`24`	`+ #[IgnorePhpunitWarnings]`
`25`	`25`	`public function testPhpunitWarning(): void`
`26`	`26`	`{`
`27`	`27`	`EventFacade::emitter()->testTriggeredPhpunitWarning(`
`@@ -32,7 +32,7 @@ public function testPhpunitWarning(): void`
`32`	`32`	`$this->assertTrue(true);`
`33`	`33`	`}`
`34`	`34`
`35`		`- #[IgnorePHPUnitWarnings('warning message')]`
	`35`	`+ #[IgnorePhpunitWarnings('warning message')]`
`36`	`36`	`public function testPhpunitWarningWithExactMessage(): void`
`37`	`37`	`{`
`38`	`38`	`EventFacade::emitter()->testTriggeredPhpunitWarning(`
`@@ -43,7 +43,7 @@ public function testPhpunitWarningWithExactMessage(): void`
`43`	`43`	`$this->assertTrue(true);`
`44`	`44`	`}`
`45`	`45`
`46`		`- #[IgnorePHPUnitWarnings('warn(.)mess(.)')]`
	`46`	`+ #[IgnorePhpunitWarnings('warn(.)mess(.)')]`
`47`	`47`	`public function testPhpunitWarningWithRegex(): void`
`48`	`48`	`{`
`49`	`49`	`EventFacade::emitter()->testTriggeredPhpunitWarning(`
`@@ -54,7 +54,7 @@ public function testPhpunitWarningWithRegex(): void`
`54`	`54`	`$this->assertTrue(true);`
`55`	`55`	`}`
`56`	`56`
`57`		`- #[IgnorePHPUnitWarnings('warn(.)mess(.)')]`
	`57`	`+ #[IgnorePhpunitWarnings('warn(.)mess(.)')]`
`58`	`58`	`public function testPhpunitWarningWithWrongPattern(): void`
`59`	`59`	`{`
`60`	`60`	`EventFacade::emitter()->testTriggeredPhpunitWarning(`
`@@ -66,7 +66,7 @@ public function testPhpunitWarningWithWrongPattern(): void`
`66`	`66`	`}`
`67`	`67`
`68`	`68`	`#[DataProvider('dataProvider')]`
`69`		`- #[IgnorePHPUnitWarnings]`
	`69`	`+ #[IgnorePhpunitWarnings]`
`70`	`70`	`public function testTooManyArgumentsInDataProvider(): void`
`71`	`71`	`{`
`72`	`72`	`$this->assertTrue(true);`
Original file line number	Diff line number	Diff line change
`@@ -14,9 +14,9 @@`
`14`	`14`	`use PHPUnit\Framework\Attributes\Small;`
`15`	`15`	`use PHPUnit\Framework\TestCase;`
`16`	`16`
`17`		`-#[CoversClass(IgnorePHPUnitWarnings::class)]`
	`17`	`+#[CoversClass(IgnorePhpunitWarnings::class)]`
`18`	`18`	`#[Small]`
`19`		`-final class IgnorePHPUnitWarningsTest extends TestCase`
	`19`	`+final class IgnorePhpunitWarningsTest extends TestCase`
`20`	`20`	`{`
`21`	`21`	`public static function shouldIgnoreProvider(): array`
`22`	`22`	`{`
`@@ -32,7 +32,7 @@ public static function shouldIgnoreProvider(): array`
`32`	`32`	`#[DataProvider('shouldIgnoreProvider')]`
`33`	`33`	`public function testShouldIgnore(?string $messagePattern, string $message, bool $expected): void`
`34`	`34`	`{`
`35`		`- $metadata = Metadata::ignorePHPUnitWarnings($messagePattern);`
	`35`	`+ $metadata = Metadata::ignorePhpunitWarnings($messagePattern);`
`36`	`36`
`37`	`37`	`$this->assertSame($expected, $metadata->shouldIgnore($message));`
`38`	`38`	`}`