feat(tia): continues to work on poc

This commit is contained in:
nuno maduro
2026-04-16 06:32:24 -07:00
parent 192f289e7e
commit 30fff116fd
5 changed files with 107 additions and 22 deletions

View File

@ -6,13 +6,16 @@ namespace Pest\Concerns;
use Closure;
use Pest\Exceptions\DatasetArgumentsMismatch;
use Pest\Contracts\Plugins\AfterEachable;
use Pest\Contracts\Plugins\BeforeEachable;
use Pest\Contracts\Plugins\Runnable;
use Pest\Panic;
use Pest\Plugin\Loader;
use Pest\Preset;
use Pest\Support\ChainableClosure;
use Pest\Support\ExceptionTrace;
use Pest\Support\Reflection;
use Pest\Support\Shell;
use Pest\Plugins\Tia\State as TiaState;
use Pest\TestSuite;
use PHPUnit\Framework\Attributes\PostCondition;
use PHPUnit\Framework\IncompleteTest;
@ -228,14 +231,11 @@ trait Testable
{
TestSuite::getInstance()->test = $this;
// TIA replay fast-path. When the file is known to the dependency graph
// and none of its deps changed since recording, skip both the
// framework `setUp()` (Laravel app bootstrap, DB refresh, etc.) and
// the user `beforeEach` chain. The matching short-circuit inside
// `__runTest()` ensures the test body never executes, so no
// initialisation is needed.
if (TiaState::instance()->shouldReplayFromCache(self::$__filename, $this::class.'::'.$this->name())) {
return;
/** @var BeforeEachable $plugin */
foreach (Loader::getPlugins(BeforeEachable::class) as $plugin) {
if ($plugin->beforeEach(self::$__filename, $this::class.'::'.$this->name()) === false) {
return;
}
}
$method = TestSuite::getInstance()->tests->get(self::$__filename)->getMethod($this->name());
@ -313,13 +313,13 @@ trait Testable
*/
protected function tearDown(...$arguments): void
{
// TIA replay: setUp was skipped, the closure never ran — there is
// no matching cleanup to perform here. Keep the framework invariant
// of clearing the "current test" pointer and bail out.
if (TiaState::instance()->shouldReplayFromCache(self::$__filename, $this::class.'::'.$this->name())) {
TestSuite::getInstance()->test = null;
/** @var AfterEachable $plugin */
foreach (Loader::getPlugins(AfterEachable::class) as $plugin) {
if ($plugin->afterEach(self::$__filename, $this::class.'::'.$this->name()) === false) {
TestSuite::getInstance()->test = null;
return;
return;
}
}
$afterEach = TestSuite::getInstance()->afterEach->get(self::$__filename);
@ -347,13 +347,13 @@ trait Testable
*/
private function __runTest(Closure $closure, ...$args): mixed
{
// TIA replay: the file's deps haven't changed and it last passed.
// Bypass the closure entirely and register a synthetic assertion so
// PHPUnit does not emit a "risky: no assertions" warning.
if (TiaState::instance()->shouldReplayFromCache(self::$__filename, $this::class.'::'.$this->name())) {
$this->addToAssertionCount(1);
/** @var Runnable $plugin */
foreach (Loader::getPlugins(Runnable::class) as $plugin) {
if ($plugin->run(self::$__filename, $this::class.'::'.$this->name()) === false) {
$this->addToAssertionCount(1);
return null;
return null;
}
}
$arguments = $this->__resolveTestArguments($args);

View File

@ -0,0 +1,20 @@
<?php
declare(strict_types=1);
namespace Pest\Contracts\Plugins;
/**
* Called at the start of `tearDown`. Return `false` to skip the framework
* tearDown, afterEach chain, and method-level cleanup.
*
* @internal
*/
interface AfterEachable
{
/**
* @param string $filename Absolute path of the test file.
* @param string $testId Fully-qualified `Class::method` identifier.
*/
public function afterEach(string $filename, string $testId): bool;
}

View File

@ -0,0 +1,26 @@
<?php
declare(strict_types=1);
namespace Pest\Contracts\Plugins;
/**
* Plugins implementing this interface are called before each test's `setUp`.
*
* Return `false` to skip the test entirely — `setUp`, body and `tearDown`
* are all bypassed and the test counts as passed with one synthetic
* assertion. Any other return value lets the test proceed normally.
*
* Resolution happens once per process via `Loader::getPlugins`; the per-test
* call is a cheap iteration over the cached list.
*
* @internal
*/
interface BeforeEachable
{
/**
* @param string $filename Absolute path of the test file.
* @param string $testId Fully-qualified `Class::method` identifier.
*/
public function beforeEach(string $filename, string $testId): bool;
}

View File

@ -0,0 +1,21 @@
<?php
declare(strict_types=1);
namespace Pest\Contracts\Plugins;
/**
* Called around the test body (`__runTest`). Return `false` to skip the
* closure — a synthetic assertion is registered so PHPUnit does not flag the
* test as risky.
*
* @internal
*/
interface Runnable
{
/**
* @param string $filename Absolute path of the test file.
* @param string $testId Fully-qualified `Class::method` identifier.
*/
public function run(string $filename, string $testId): bool;
}

View File

@ -5,7 +5,10 @@ declare(strict_types=1);
namespace Pest\Plugins;
use Pest\Contracts\Plugins\AddsOutput;
use Pest\Contracts\Plugins\AfterEachable;
use Pest\Contracts\Plugins\BeforeEachable;
use Pest\Contracts\Plugins\HandlesArguments;
use Pest\Contracts\Plugins\Runnable;
use Pest\Contracts\Plugins\Terminable;
use Pest\Exceptions\NoDirtyTestsFound;
use Pest\Panic;
@ -62,7 +65,7 @@ use Throwable;
*
* @internal
*/
final class Tia implements AddsOutput, HandlesArguments, Terminable
final class Tia implements AddsOutput, AfterEachable, BeforeEachable, HandlesArguments, Runnable, Terminable
{
use Concerns\HandleArguments;
@ -130,6 +133,21 @@ final class Tia implements AddsOutput, HandlesArguments, Terminable
return $this->handleParent($arguments, $projectRoot, $forceRebuild);
}
public function beforeEach(string $filename, string $testId): bool
{
return ! State::instance()->shouldReplayFromCache($filename, $testId);
}
public function run(string $filename, string $testId): bool
{
return ! State::instance()->shouldReplayFromCache($filename, $testId);
}
public function afterEach(string $filename, string $testId): bool
{
return ! State::instance()->shouldReplayFromCache($filename, $testId);
}
public function terminate(): void
{
if ($this->graphWritten) {