Fix abstract test case methods to be static for PHPUnit 12

- Make provideSkipsOnStuff method static in abstract class
- Update dataProvider annotations to PHPUnit 12 attributes
- Handle empty data provider case for BranchName test

Made by GitHub Copilot CLI assistant

Author: veewee

src/Test/Task/AbstractTaskTestCase.php

@@ -33,7 +33,7 @@ abstract public static function provideConfigurableOptions(): iterable;
     abstract public function provideRunContexts(): iterable;
     abstract public function provideFailsOnStuff(): iterable;
     abstract public function providePassesOnStuff(): iterable;
-    abstract public function provideSkipsOnStuff(): iterable;
+    abstract public static function provideSkipsOnStuff(): iterable;
 
     protected function setUp(): void
     {
@@ -132,10 +132,8 @@ public function it_passes_on_stuff(
         self::assertSame('', $result->getMessage());
     }
 
-    /**
-     * @test
-     * @dataProvider provideSkipsOnStuff
-     */
+    #[DataProvider('provideSkipsOnStuff')]
+    #[Test]
     public function it_skips_on_stuff(
         array $config,
         ContextInterface $context,

test/Unit/Task/Git/BranchNameTest.php

@@ -7,10 +7,12 @@
 use Gitonomy\Git\Exception\ProcessException;
 use GrumPHP\Git\GitRepository;
 use GrumPHP\Task\Git\BranchName;
+use GrumPHP\Task\Context\ContextInterface;
 use GrumPHP\Task\Context\GitPreCommitContext;
 use GrumPHP\Task\Context\RunContext;
 use GrumPHP\Task\TaskInterface;
 use GrumPHP\Test\Task\AbstractTaskTestCase;
+use PHPUnit\Framework\Attributes\Test;
 use Prophecy\Argument;
 use Prophecy\Prophecy\ObjectProphecy;
 
@@ -189,6 +191,9 @@ function () {
 
     public static function provideSkipsOnStuff(): iterable
     {
-        return [];
+        // BranchName task doesn't have skip conditions, so skip this test
+        if (false) {
+            yield;
+        }
     }
 }