[4.x] Add dependsExternal (depend on tests in external test files)

overrides/Runner/TestSuiteLoader.php

@@ -179,7 +179,6 @@ public function load(string $suiteClassFile): ReflectionClass
         if (! class_exists($suiteClassName, false)) {
             return $this->exceptionFor($suiteClassName, $suiteClassFile);
         }
-
         // @codeCoverageIgnoreEnd
 
         if ($class->isSubclassOf(TestCase::class) && ! $class->isAbstract()) {

src/Factories/TestCaseMethodFactory.php

@@ -15,6 +15,7 @@
 use PHPUnit\Framework\Assert;
 use PHPUnit\Framework\Attributes\DataProvider;
 use PHPUnit\Framework\Attributes\Depends;
+use PHPUnit\Framework\Attributes\DependsExternal;
 use PHPUnit\Framework\Attributes\Test;
 use PHPUnit\Framework\Attributes\TestDox;
 use PHPUnit\Framework\TestCase;
@@ -102,6 +103,19 @@ final class TestCaseMethodFactory
      */
     public array $depends = [];
 
+    /**
+     * The test's external dependencies (tests in other Pest test files).
+     *
+     * Example:
+     * [
+     *  'testCase' => 'Tests\Features\Depends',
+     *  'test' => 'first',
+     * ]
+     *
+     * @var array<int, array{testCase: string, test: string}>
+     */
+    public array $dependsExternal = [];
+
     /**
      * The test's groups.
      *
@@ -183,7 +197,7 @@ public function getClosure(): Closure
      */
     public function receivesArguments(): bool
     {
-        return $this->datasets !== [] || $this->depends !== [] || $this->repetitions > 1;
+        return $this->datasets !== [] || $this->depends !== [] || $this->dependsExternal !== [] || $this->repetitions > 1;
     }
 
     /**
@@ -220,6 +234,20 @@ public function buildForEvaluation(): string
             );
         }
 
+        foreach ($this->dependsExternal as $externalDepend) {
+            $depend = Str::evaluable(
+                $this->describing === []
+                    ? $externalDepend['test']
+                    : Str::describe($this->describing, $externalDepend['test'])
+            );
+
+            $className = 'P\\'.$externalDepend['testCase'];
+            $this->attributes[] = new Attribute(
+                DependsExternal::class,
+                [$className, $depend],
+            );
+        }
+
         if ($this->datasets !== [] || $this->repetitions > 1) {
             $dataProviderName = $methodName.'_dataset';
             $this->attributes[] = new Attribute(

src/PendingCalls/TestCall.php

@@ -208,6 +208,24 @@ public function depends(string ...$depends): self
         return $this;
     }
 
+    /**
+     * Sets external test dependencies from another Pest test file reference.
+     *
+     * @param  string  $testCase  The Pest test reference (e.g. "Tests\Features\Depends")
+     * @param  string  ...$depends  One or more test descriptions to depend on
+     */
+    public function dependsExternal(string $testCase, string ...$depends): self
+    {
+        foreach ($depends as $depend) {
+            $this->testCaseMethod->dependsExternal[] = [
+                'testCase' => $testCase,
+                'test' => $depend,
+            ];
+        }
+
+        return $this;
+    }
+
     /**
      * Sets the test group(s).
      */

tests/.snapshots/success.txt

@@ -289,6 +289,14 @@
   ✓ describe block → nested describe → third in nested describe
   ✓ depends on test after describe block
 
+   PASS  Tests\Features\DependsExternal
+  ✓ dependsExternal
+  ✓ dependsExternal chained
+  ✓ dependsExternal chained, reversed
+  ✓ depends on tests in an external file
+  ✓ depends on tests in an external file with spread arguments
+  ✓ depends on tests in an external file with defined arguments
+
    PASS  Tests\Features\DependsInheritance
   ✓ it is a test
   ✓ it uses correct parent class
@@ -1901,4 +1909,4 @@
   ✓ pass with dataset with ('my-datas-set-value')
   ✓ within describe → pass with dataset with ('my-datas-set-value')
 
-  Tests:    2 deprecated, 4 warnings, 5 incomplete, 2 notices, 40 todos, 35 skipped, 1294 passed (2971 assertions)
+  Tests:    2 deprecated, 4 warnings, 5 incomplete, 2 notices, 40 todos, 35 skipped, 1300 passed (2980 assertions)

tests/Features/DependsExternal.php

@@ -0,0 +1,30 @@
+<?php
+
+test('dependsExternal', function () {
+    expect(func_get_args())->toBe(['first', 'second']);
+})->dependsExternal('Tests\Features\Depends', 'first', 'second');
+
+test('dependsExternal chained', function () {
+    expect(func_get_args())->toBe(['first', 'second']);
+})->dependsExternal('Tests\Features\Depends', 'first')
+    ->dependsExternal('Tests\Features\Depends', 'second');
+
+test('dependsExternal chained, reversed', function () {
+    expect(func_get_args())->toBe(['second', 'first']);
+})->dependsExternal('Tests\Features\Depends', 'second')
+    ->dependsExternal('Tests\Features\Depends', 'first');
+
+test('depends on tests in an external file', function (string $first, string $second) {
+    expect($first)->toBe('first');
+    expect($second)->toBe('second');
+})->dependsExternal('Tests\Features\Depends', 'first', 'second');
+
+test('depends on tests in an external file with spread arguments', function (string ...$params) {
+    expect(func_get_args())->toBe($params);
+    expect($params)->toBe(['first', 'second']);
+})->dependsExternal('Tests\Features\Depends', 'first', 'second');
+
+test('depends on tests in an external file with defined arguments', function (string $first, string $second) {
+    expect($first)->toBe('first');
+    expect($second)->toBe('second');
+})->dependsExternal('Tests\Features\Depends', 'first', 'second');

tests/Visual/Parallel.php

@@ -23,13 +23,13 @@
         $file = file_get_contents(__FILE__);
         $file = preg_replace(
             '/\$expected = \'.*?\';/',
-            "\$expected = '2 deprecated, 4 warnings, 5 incomplete, 3 notices, 40 todos, 27 skipped, 1278 passed (2920 assertions)';",
+            "\$expected = '2 deprecated, 4 warnings, 5 incomplete, 3 notices, 40 todos, 27 skipped, 1284 passed (2929 assertions)';",
             $file,
         );
         file_put_contents(__FILE__, $file);
     }
 
-    $expected = '2 deprecated, 4 warnings, 5 incomplete, 3 notices, 40 todos, 27 skipped, 1278 passed (2920 assertions)';
+    $expected = '2 deprecated, 4 warnings, 5 incomplete, 3 notices, 40 todos, 27 skipped, 1284 passed (2929 assertions)';
 
     expect($output)
         ->toContain("Tests:    {$expected}")