diff --git a/src/Concerns/TestCase.php b/src/Concerns/TestCase.php index 5dbac402..fecd488e 100644 --- a/src/Concerns/TestCase.php +++ b/src/Concerns/TestCase.php @@ -129,16 +129,25 @@ trait TestCase /** * Runs the test. + * + * @return mixed + * + * @throws \Throwable */ - public function __test(): void + public function __test() { - $this->__callClosure($this->__test, func_get_args()); + return $this->__callClosure($this->__test, func_get_args()); } - private function __callClosure(Closure $closure, array $arguments): void + /** + * @return mixed + * + * @throws \Throwable + */ + private function __callClosure(Closure $closure, array $arguments) { - ExceptionTrace::ensure(function () use ($closure, $arguments) { - call_user_func_array(Closure::bind($closure, $this, get_class($this)), $arguments); + return ExceptionTrace::ensure(function () use ($closure, $arguments) { + return call_user_func_array(Closure::bind($closure, $this, get_class($this)), $arguments); }); } diff --git a/src/Factories/TestCaseFactory.php b/src/Factories/TestCaseFactory.php index 49c6eaad..f3e8e282 100644 --- a/src/Factories/TestCaseFactory.php +++ b/src/Factories/TestCaseFactory.php @@ -132,10 +132,14 @@ final class TestCaseFactory $proxies = $this->proxies; $factoryTest = $this->test; - $test = function () use ($chains, $proxies, $factoryTest): void { + /** + * @return mixed + */ + $test = function () use ($chains, $proxies, $factoryTest) { $proxies->proxy($this); $chains->chain($this); - call_user_func(Closure::bind($factoryTest, $this, get_class($this)), ...func_get_args()); + + return call_user_func(Closure::bind($factoryTest, $this, get_class($this)), ...func_get_args()); }; $className = $this->makeClassFromFilename($this->filename); diff --git a/src/PendingObjects/TestCall.php b/src/PendingObjects/TestCall.php index 7b7549ae..e0b7f1c4 100644 --- a/src/PendingObjects/TestCall.php +++ b/src/PendingObjects/TestCall.php @@ -84,6 +84,18 @@ final class TestCall return $this; } + /** + * Sets the test depends. + */ + public function depends(string ...$tests): TestCall + { + $this->testCaseFactory + ->factoryProxies + ->add(Backtrace::file(), Backtrace::line(), 'setDependencies', [$tests]); + + return $this; + } + /** * Makes the test suite only this test case. */ @@ -95,7 +107,7 @@ final class TestCall } /** - * Sets the test groups(s). + * Sets the test group(s). */ public function group(string ...$groups): TestCall { diff --git a/src/Support/ExceptionTrace.php b/src/Support/ExceptionTrace.php index a7166f41..72a1d829 100644 --- a/src/Support/ExceptionTrace.php +++ b/src/Support/ExceptionTrace.php @@ -17,11 +17,15 @@ final class ExceptionTrace /** * Ensures the given closure reports * the good execution context. + * + * @return mixed + * + * @throws \Throwable */ - public static function ensure(Closure $closure): void + public static function ensure(Closure $closure) { try { - $closure(); + return $closure(); } catch (Throwable $throwable) { if (Str::startsWith($message = $throwable->getMessage(), self::UNDEFINED_METHOD)) { $message = str_replace(self::UNDEFINED_METHOD, 'Call to undefined method ', $message); diff --git a/tests/.snapshots/success.txt b/tests/.snapshots/success.txt index 1671b70b..eae1f6cc 100644 --- a/tests/.snapshots/success.txt +++ b/tests/.snapshots/success.txt @@ -49,6 +49,16 @@ ✓ it creates unique test case names with ('Name 1', Pest\Plugin Object (), true) #3 ✓ it creates unique test case names - count + WARN Tests\Features\Depends + ✓ first + ✓ second + ✓ depends + ✓ depends with ...params + ✓ depends with defined arguments + ✓ depends run test only once + s incomplete → incomplete + w depends on incomplete → This test depends on "P\Tests\Features\Depends::incomplete" which does not exist. + PASS Tests\Features\Exceptions ✓ it gives access the the underlying expectException ✓ it catch exceptions @@ -151,5 +161,6 @@ WARN Tests\Visual\Success s visual snapshot of test suite on success - Tests: 6 skipped, 86 passed - Time: 3.58s + Tests: 1 warnings, 7 skipped, 85 passed + Time: 2.65s + diff --git a/tests/Features/Depends.php b/tests/Features/Depends.php new file mode 100644 index 00000000..9a46034c --- /dev/null +++ b/tests/Features/Depends.php @@ -0,0 +1,44 @@ +depends('first', 'second'); + +test('depends with ...params', function (string ...$params) { + assertEquals( + ['first', 'second'], + $params + ); +})->depends('first', 'second'); + +test('depends with defined arguments', function (string $first, string $second) { + assertEquals('first', $first); + assertEquals('second', $second); +})->depends('first', 'second'); + +test('depends run test only once', function () use (&$runCounter) { + assertEquals(2, $runCounter); +})->depends('first', 'second'); + +test('incomplete')->skip('incomplete'); + +test('depends on incomplete')->depends('incomplete')->doesNotPerformAssertions();