Merge branch '12.1'

Author: sebastianbergmann

src/Framework/TestCase.php

@@ -1250,24 +1250,14 @@ protected function onNotSuccessfulTest(Throwable $t): never
      */
     private function runTest(): mixed
     {
-        $testArguments       = array_merge($this->data, array_values($this->dependencyInput));
-        $positionalArguments = [];
-        $namedArguments      = [];
-
-        foreach ($testArguments as $key => $value) {
-            if (is_int($key)) {
-                $positionalArguments[] = $value;
-            } else {
-                $namedArguments[$key] = $value;
-            }
-        }
+        $testArguments = array_merge($this->data, array_values($this->dependencyInput));
 
         $capture          = tmpfile();
         $errorLogPrevious = ini_set('error_log', stream_get_meta_data($capture)['uri']);
 
         try {
             /** @phpstan-ignore method.dynamicName */
-            $testResult = $this->{$this->methodName}(...$namedArguments, ...$positionalArguments);
+            $testResult = $this->{$this->methodName}(...$testArguments);
 
             $errorLogOutput = stream_get_contents($capture);
 

tests/end-to-end/data-provider/dependency-result.phpt

@@ -13,9 +13,14 @@ PHPUnit %s by Sebastian Bergmann and contributors.
 
 Runtime: %s
 
-....                                                                4 / 4 (100%)
+...E                                                                4 / 4 (100%)
 
 Time: %s, Memory: %s
 
-OK (4 tests, 7 assertions)
+There was 1 error:
 
+1) PHPUnit\TestFixture\DataProviderDependencyResultTest::testAdd with data set #2 (2, 0)
+Error: Cannot use positional argument after named argument during unpacking
+
+ERRORS!
+Tests: 4, Assertions: 5, Errors: 1.

tests/unit/Framework/TestCaseTest.php

@@ -11,8 +11,6 @@
 
 use function sprintf;
 use PHPUnit\Framework\Attributes\CoversClass;
-use PHPUnit\Framework\Attributes\DataProvider;
-use PHPUnit\Framework\Attributes\Depends;
 use PHPUnit\Framework\Attributes\ExcludeGlobalVariableFromBackup;
 use PHPUnit\TestFixture\TestWithDifferentNames;
 
@@ -23,18 +21,6 @@ class TestCaseTest extends TestCase
 {
     protected static int $testStatic = 456;
 
-    /**
-     * @return array<non-empty-string, array<non-empty-string, string>>
-     */
-    public static function provider(): array
-    {
-        return [
-            'case 1' => [
-                'fromProvider' => 'value from provider',
-            ],
-        ];
-    }
-
     public static function setUpBeforeClass(): void
     {
         $GLOBALS['a']  = 'a';
@@ -97,19 +83,4 @@ public function testGetNameReturnsMethodName(): void
 
         $this->assertSame($methodName, $testCase->nameWithDataSet());
     }
-
-    public function testDependedUponTest(): string
-    {
-        $this->assertTrue(true);
-
-        return 'value from depended-upon test';
-    }
-
-    #[DataProvider('provider')]
-    #[Depends('testDependedUponTest')]
-    public function testTestDataFromProviderAndDependedUponTest(string $fromProvider, string $fromDependency): void
-    {
-        $this->assertSame('value from provider', $fromProvider);
-        $this->assertSame('value from depended-upon test', $fromDependency);
-    }
 }