Fix PHPUnit deprecations

Author: amcsi

phpunit.xml

@@ -5,14 +5,14 @@
   colors="true"
   processIsolation="false"
   stopOnFailure="false"
-  xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/10.0/phpunit.xsd"
+  xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/11.5/phpunit.xsd"
   cacheDirectory=".phpunit.cache"
   backupStaticProperties="false">
-  <coverage>
+  <source>
     <include>
       <directory suffix=".php">./app</directory>
     </include>
-  </coverage>
+  </source>
   <testsuites>
     <testsuite name="Unit">
       <directory suffix="Test.php">./tests/Unit</directory>

rector.php

@@ -2,10 +2,13 @@
 declare(strict_types=1);
 
 use Rector\Config\RectorConfig;
+use Rector\PHPUnit\Set\PHPUnitSetList;
 use RectorLaravel\Set\LaravelLevelSetList;
 
 return RectorConfig::configure()
     // register single rule
     ->withSets([
         LaravelLevelSetList::UP_TO_LARAVEL_110,
+        PHPUnitSetList::PHPUNIT_120,
+        PHPUnitSetList::ANNOTATIONS_TO_ATTRIBUTES,
     ]);

tests/Feature/I18n/AutoTranslator/SpaceAfterPeriodFixerTest.php

@@ -4,14 +4,13 @@
 namespace Tests\Feature\I18n\AutoTranslator;
 
 use amcsi\LyceeOverture\I18n\AutoTranslator\SpaceAfterPeriodFixer;
+use PHPUnit\Framework\Attributes\DataProvider;
 use PHPUnit\Framework\TestCase;
 
 class SpaceAfterPeriodFixerTest extends TestCase
 {
 
-    /**
-     * @dataProvider provideFix
-     */
+    #[DataProvider('provideFix')]
     public function testFix($expected, $input)
     {
         self::assertSame($expected, SpaceAfterPeriodFixer::fix($input));

tests/I18n/Unit/Tools/JapaneseSentenceSplitterTest.php

@@ -4,17 +4,17 @@
 namespace Tests\I18n\Unit\Tools;
 
 use amcsi\LyceeOverture\I18n\Tools\JapaneseSentenceSplitter;
+use PHPUnit\Framework\Attributes\DataProvider;
 use PHPUnit\Framework\TestCase;
 
 class JapaneseSentenceSplitterTest extends TestCase
 {
 
-    /**
-     * @dataProvider provideReplace
-     */
+    #[DataProvider('provideReplace')]
     public function testReplaceCallback(string $expected, string $input)
     {
-        self::assertSame($expected, JapaneseSentenceSplitter::replaceCallback($input, static fn($match) => $match[0] . '¤'));
+        self::assertSame($expected,
+            JapaneseSentenceSplitter::replaceCallback($input, static fn($match) => $match[0].'¤'));
     }
 
     public static function provideReplace()
@@ -23,7 +23,7 @@ public static function provideReplace()
             'simple' => ['hey¤', 'hey'],
             'line break' => ["hey\n¤yo¤", "hey\nyo"],
             'periods' => ["る。¤yo¤", "る。yo"],
-            'period and line break' => ["る。¤\n¤",  "る。\n"],
+            'period and line break' => ["る。¤\n¤", "る。\n"],
             'periods and line breaks' => ["る。¤\n¤き。¤\n¤", "る。\nき。\n"],
             'periods in brackets' => ["[ペナルティ:[１枚ドローする。]]。¤る。¤", "[ペナルティ:[１枚ドローする。]]。る。"],
         ];

tests/Tools/SetAutoCreatorStubber.php

@@ -10,10 +10,10 @@
 class SetAutoCreatorStubber
 {
     /**
-     * @param Set|null $setModel Optional Set model override for mocking DB operations.
+     * @param  Set|null  $setModel  Optional Set model override for mocking DB operations.
      * @return SetAutoCreator
      */
-    public static function createInstanceWithSets(Set $setModel = null): SetAutoCreator
+    public static function createInstanceWithSets(?Set $setModel = null): SetAutoCreator
     {
         $set = new Set();
         $set->id = 1;

tests/Unit/Etc/LackeyHasherTest.php

@@ -4,13 +4,12 @@
 namespace Tests\Unit\Etc;
 
 use amcsi\LyceeOverture\Etc\LackeyHasher;
+use PHPUnit\Framework\Attributes\DataProvider;
 use PHPUnit\Framework\TestCase;
 
 class LackeyHasherTest extends TestCase
 {
-    /**
-     * @dataProvider provideFilesToHash
-     */
+    #[DataProvider('provideFilesToHash')]
     public function testHashesCorrectly($expected, $filename): void
     {
         self::assertSame($expected, LackeyHasher::hashFile($filename));
@@ -19,10 +18,10 @@ public function testHashesCorrectly($expected, $filename): void
     public static function provideFilesToHash()
     {
         return [
-            [350485, __DIR__ . '/350485.txt'],
-            [372237, __DIR__ . '/372237.txt'],
-            [-20265, __DIR__ . '/-20265.jpg'],
-            [-8196, __DIR__ . '/-8196.jpg'],
+            [350485, __DIR__.'/350485.txt'],
+            [372237, __DIR__.'/372237.txt'],
+            [-20265, __DIR__.'/-20265.jpg'],
+            [-8196, __DIR__.'/-8196.jpg'],
         ];
     }
 }

tests/Unit/I18n/AutoTranslator/CapitalizationFixerTest.php

@@ -4,13 +4,12 @@
 namespace Tests\Unit\I18n\AutoTranslator;
 
 use amcsi\LyceeOverture\I18n\AutoTranslator\CapitalizationFixer;
+use PHPUnit\Framework\Attributes\DataProvider;
 use PHPUnit\Framework\TestCase;
 
 class CapitalizationFixerTest extends TestCase
 {
-    /**
-     * @dataProvider provideFixCapitalization
-     */
+    #[DataProvider('provideFixCapitalization')]
     public function testFixCapitalization(string $expected, string $input)
     {
         self::assertSame($expected, CapitalizationFixer::fixCapitalization($input));

tests/Unit/I18n/AutoTranslator/DiscardFromDeckTest.php

@@ -4,13 +4,12 @@
 namespace Tests\Unit\I18n\AutoTranslator;
 
 use amcsi\LyceeOverture\I18n\AutoTranslator\DiscardFromDeck;
+use PHPUnit\Framework\Attributes\DataProvider;
 use PHPUnit\Framework\TestCase;
 
 class DiscardFromDeckTest extends TestCase
 {
-    /**
-     * @dataProvider provideAutoTranslate
-     */
+    #[DataProvider('provideAutoTranslate')]
     public function testAutoTranslate($expected, $input)
     {
         self::assertSame($expected, DiscardFromDeck::autoTranslate($input));

tests/Unit/I18n/AutoTranslator/DrawCardsTest.php

@@ -8,9 +8,7 @@
 
 class DrawCardsTest extends TestCase
 {
-    /**
-     * @dataProvider provideAutoTranslate
-     */
+    #[\PHPUnit\Framework\Attributes\DataProvider('provideAutoTranslate')]
     public function testAutoTranslate(string $expected, string $input)
     {
         self::assertSame($expected, DrawCards::autoTranslate($input));

tests/Unit/I18n/AutoTranslator/WhenAppearsTest.php

@@ -4,13 +4,12 @@
 namespace Tests\Unit\I18n\AutoTranslator;
 
 use amcsi\LyceeOverture\I18n\AutoTranslator\WhenAppears;
+use PHPUnit\Framework\Attributes\DataProvider;
 use PHPUnit\Framework\TestCase;
 
 class WhenAppearsTest extends TestCase
 {
-    /**
-     * @dataProvider provideAutoTranslate
-     */
+    #[DataProvider('provideAutoTranslate')]
     public function testAutoTranslate(string $expected, string $input)
     {
         self::assertSame($expected, WhenAppears::autoTranslate($input));