chore: requires latest versions of collision and termwind

[Bug] Always call parent teardown even if an exception is thrown

.github/workflows/tests.yml

@@ -14,7 +14,7 @@ jobs:
       matrix:
         os: [ubuntu-latest, macos-latest, windows-latest]
         symfony: ['6.4', '7.0']
-        php: ['8.1', '8.2', '8.3']
+        php: ['8.1', '8.2', '8.3', '8.4']
         dependency_version: [prefer-lowest, prefer-stable]
         exclude:
           - php: '8.1'

README.md

@@ -31,8 +31,10 @@ We cannot thank our sponsors enough for their incredible support in funding Pest
 
 - [Akaunting](https://akaunting.com/?ref=pestphp)
 - [Codecourse](https://codecourse.com/?ref=pestphp)
+- [DocuWriter.ai](https://www.docuwriter.ai/?ref=pestphp)
 - [Laracasts](https://laracasts.com/?ref=pestphp)
 - [Localazy](https://localazy.com/?ref=pestphp)
+- [Route4Me](https://www.route4me.com/?ref=pestphp)
 - [Zapiet](https://www.zapiet.com/?ref=pestphp)
 
 Pest is an open-sourced software licensed under the **[MIT license](https://opensource.org/licenses/MIT)**.

bin/worker.php

@@ -12,7 +12,7 @@ use Symfony\Component\Console\Output\ConsoleOutput;
 use Symfony\Component\Console\Output\OutputInterface;
 
 $bootPest = (static function (): void {
-    $workerArgv = new ArgvInput();
+    $workerArgv = new ArgvInput;
 
     $rootPath = dirname(PHPUNIT_COMPOSER_INSTALL, 2);
     $testSuite = TestSuite::getInstance($rootPath, $workerArgv->getParameterOption(
@@ -20,7 +20,7 @@ $bootPest = (static function (): void {
         'tests'
     ));
 
-    $input = new ArgvInput();
+    $input = new ArgvInput;
 
     $output = new ConsoleOutput(OutputInterface::VERBOSITY_NORMAL, true);
 

composer.json

@@ -19,14 +19,15 @@
     "require": {
         "php": "^8.1.0",
         "brianium/paratest": "^7.3.1",
-        "nunomaduro/collision": "^7.10.0|^8.1.1",
+        "nunomaduro/collision": "^7.11.0|^8.4.0",
-        "nunomaduro/termwind": "^1.15.1|^2.0.1",
+        "nunomaduro/termwind": "^1.16.0|^2.1.0",
         "pestphp/pest-plugin": "^2.1.1",
         "pestphp/pest-plugin-arch": "^2.7.0",
-        "phpunit/phpunit": "^10.5.17"
+        "phpunit/phpunit": "^10.5.36"
     },
     "conflict": {
-        "phpunit/phpunit": ">10.5.17",
+        "filp/whoops": "<2.16.0",
+        "phpunit/phpunit": ">10.5.36",
         "sebastian/exporter": "<5.1.0",
         "webmozart/assert": "<1.11.0"
     },
@@ -51,9 +52,9 @@
         ]
     },
     "require-dev": {
-        "pestphp/pest-dev-tools": "^2.16.0",
+        "pestphp/pest-dev-tools": "^2.17.0",
-        "pestphp/pest-plugin-type-coverage": "^2.8.4",
+        "pestphp/pest-plugin-type-coverage": "^2.8.7",
-        "symfony/process": "^6.4.0|^7.1.1"
+        "symfony/process": "^6.4.0|^7.1.5"
     },
     "minimum-stability": "dev",
     "prefer-stable": true,
@@ -73,7 +74,7 @@
         "test:refacto": "rector --dry-run",
         "test:lint": "pint --test",
         "test:type:check": "phpstan analyse --ansi --memory-limit=-1 --debug",
-        "test:type:coverage": "php bin/pest --type-coverage --min=100",
+        "test:type:coverage": "php -d memory_limit=-1  bin/pest --type-coverage --min=100",
         "test:unit": "php bin/pest --colors=always --exclude-group=integration --compact",
         "test:inline": "php bin/pest --colors=always --configuration=phpunit.inline.xml",
         "test:parallel": "php bin/pest --colors=always --exclude-group=integration --parallel --processes=3",

phpstan.neon

@@ -1,6 +1,5 @@
 includes:
     - vendor/phpstan/phpstan-strict-rules/rules.neon
-    - vendor/ergebnis/phpstan-rules/rules.neon
     - vendor/thecodingmachine/phpstan-strict-rules/phpstan-strict-rules.neon
 
 parameters:
@@ -12,12 +11,4 @@ parameters:
     reportUnmatchedIgnoredErrors: true
 
     ignoreErrors:
-        - "#has a nullable return type declaration.#"
-        - "#Language construct isset\\(\\) should not be used.#"
-        - "#is not allowed to extend#"
-        - "#is concrete, but does not have a Test suffix#"
-        - "#with a nullable type declaration#"
         - "#type mixed is not subtype of native#"
-        - "# with null as default value#"
-        - "#has parameter \\$closure with default value.#"
-        - "#has parameter \\$description with default value.#"

src/Concerns/Testable.php

@@ -234,11 +234,13 @@ trait Testable
             $afterEach = ChainableClosure::bound($this->__afterEach, $afterEach);
         }
 
-        $this->__callClosure($afterEach, func_get_args());
+        try {
+            $this->__callClosure($afterEach, func_get_args());
+        } finally {
+            parent::tearDown();
 
-        parent::tearDown();
+            TestSuite::getInstance()->test = null;
-
+        }
-        TestSuite::getInstance()->test = null;
     }
 
     /**

src/Console/Thanks.php

@@ -50,7 +50,7 @@ final class Thanks
         $wantsToSupport = false;
 
         if (getenv('PEST_NO_SUPPORT') !== 'true' && $this->input->isInteractive()) {
-            $wantsToSupport = (new SymfonyQuestionHelper())->ask(
+            $wantsToSupport = (new SymfonyQuestionHelper)->ask(
                 new ArrayInput([]),
                 $this->output,
                 new ConfirmationQuestion(

src/Exceptions/TestClosureMustNotBeStatic.php

@@ -0,0 +1,31 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Pest\Exceptions;
+
+use InvalidArgumentException;
+use NunoMaduro\Collision\Contracts\RenderlessEditor;
+use NunoMaduro\Collision\Contracts\RenderlessTrace;
+use Pest\Factories\TestCaseMethodFactory;
+use Symfony\Component\Console\Exception\ExceptionInterface;
+
+/**
+ * @internal
+ */
+final class TestClosureMustNotBeStatic extends InvalidArgumentException implements ExceptionInterface, RenderlessEditor, RenderlessTrace
+{
+    /**
+     * Creates a new Exception instance.
+     */
+    public function __construct(TestCaseMethodFactory $method)
+    {
+        parent::__construct(
+            sprintf(
+                'Test closure must not be static. Please remove the `static` keyword from the `%s` method in `%s`.',
+                $method->description,
+                $method->filename
+            )
+        );
+    }
+}

src/Expectation.php

@@ -421,7 +421,7 @@ final class Expectation
      */
     public function any(): Any
     {
-        return new Any();
+        return new Any;
     }
 
     /**
@@ -441,7 +441,7 @@ final class Expectation
     {
         return Targeted::make(
             $this,
-            fn (ObjectDescription $object): bool => str_contains((string) file_get_contents($object->path), 'declare(strict_types=1);'),
+            fn (ObjectDescription $object): bool => (bool) preg_match('/^<\?php\s+declare\(.*?strict_types\s?=\s?1.*?\);/', (string) file_get_contents($object->path)),
             'to use strict types',
             FileLineFinder::where(fn (string $line): bool => str_contains($line, '<?php')),
         );
@@ -515,7 +515,7 @@ final class Expectation
         return Targeted::make(
             $this,
             fn (ObjectDescription $object): bool => $object->reflectionClass->hasMethod($method),
-            'to have method',
+            sprintf("to have method '%s'", $method),
             FileLineFinder::where(fn (string $line): bool => str_contains($line, 'class')),
         );
     }

src/Expectations/OppositeExpectation.php

@@ -82,7 +82,7 @@ final class OppositeExpectation
     {
         return Targeted::make(
             $this->original,
-            fn (ObjectDescription $object): bool => ! str_contains((string) file_get_contents($object->path), 'declare(strict_types=1);'),
+            fn (ObjectDescription $object): bool => ! (bool) preg_match('/^<\?php\s+declare\(.*?strict_types\s?=\s?1.*?\);/', (string) file_get_contents($object->path)),
             'not to use strict types',
             FileLineFinder::where(fn (string $line): bool => str_contains($line, '<?php')),
         );

src/Factories/Attributes/Attribute.php

@@ -20,7 +20,7 @@ abstract class Attribute
      * @param  array<int, string>  $attributes
      * @return array<int, string>
      */
-    public function __invoke(TestCaseMethodFactory $method, array $attributes): array  // @phpstan-ignore-line
+    public function __invoke(TestCaseMethodFactory $method, array $attributes): array
     {
         return $attributes;
     }

src/Factories/Concerns/HigherOrderable.php

@@ -28,8 +28,8 @@ trait HigherOrderable
      */
     private function bootHigherOrderable(): void
     {
-        $this->chains = new HigherOrderMessageCollection();
+        $this->chains = new HigherOrderMessageCollection;
-        $this->factoryProxies = new HigherOrderMessageCollection();
+        $this->factoryProxies = new HigherOrderMessageCollection;
-        $this->proxies = new HigherOrderMessageCollection();
+        $this->proxies = new HigherOrderMessageCollection;
     }
 }

src/Factories/TestCaseFactory.php

@@ -11,6 +11,7 @@ use Pest\Contracts\HasPrintableTestCaseName;
 use Pest\Exceptions\DatasetMissing;
 use Pest\Exceptions\ShouldNotHappen;
 use Pest\Exceptions\TestAlreadyExist;
+use Pest\Exceptions\TestClosureMustNotBeStatic;
 use Pest\Exceptions\TestDescriptionMissing;
 use Pest\Factories\Concerns\HigherOrderable;
 use Pest\Support\Reflection;
@@ -154,7 +155,7 @@ final class TestCaseFactory
         foreach ($classAvailableAttributes as $attribute) {
             $classAttributes = array_reduce(
                 $methods,
-                fn (array $carry, TestCaseMethodFactory $methodFactory): array => (new $attribute())->__invoke($methodFactory, $carry),
+                fn (array $carry, TestCaseMethodFactory $methodFactory): array => (new $attribute)->__invoke($methodFactory, $carry),
                 $classAttributes
             );
         }
@@ -193,7 +194,7 @@ final class TestCaseFactory
             }
             PHP;
 
-            eval($classCode); // @phpstan-ignore-line
+            eval($classCode);
         } catch (ParseError $caught) {
             throw new RuntimeException(sprintf(
                 "Unable to create test case for test file at %s. \n %s",
@@ -216,6 +217,14 @@ final class TestCaseFactory
             throw new TestAlreadyExist($method->filename, $method->description);
         }
 
+        if (
+            $method->closure instanceof \Closure &&
+            (new \ReflectionFunction($method->closure))->isStatic()
+        ) {
+
+            throw new TestClosureMustNotBeStatic($method);
+        }
+
         if (! $method->receivesArguments()) {
             if (! $method->closure instanceof \Closure) {
                 throw ShouldNotHappen::fromMessage('The test closure may not be empty.');

src/Factories/TestCaseMethodFactory.php

@@ -138,11 +138,11 @@ final class TestCaseMethodFactory
         $attributes = [];
 
         foreach ($annotationsToUse as $annotation) {
-            $annotations = (new $annotation())->__invoke($this, $annotations);
+            $annotations = (new $annotation)->__invoke($this, $annotations);
         }
 
         foreach ($attributesToUse as $attribute) {
-            $attributes = (new $attribute())->__invoke($this, $attributes);
+            $attributes = (new $attribute)->__invoke($this, $attributes);
         }
 
         if ($this->datasets !== [] || $this->repetitions > 1) {

src/Kernel.php

@@ -67,7 +67,7 @@ final class Kernel
             ->add(Container::class, $container);
 
         $kernel = new self(
-            new Application(),
+            new Application,
             $output,
         );
 

src/KernelDump.php

@@ -40,7 +40,7 @@ final class KernelDump
      */
     public function disable(): void
     {
-        @ob_clean(); // @phpstan-ignore-line
+        @ob_clean();
 
         if ($this->buffer !== '') {
             $this->flush();

src/Logging/Converter.php

@@ -36,7 +36,7 @@ final class Converter
     public function __construct(
         private readonly string $rootPath,
     ) {
-        $this->stateGenerator = new StateGenerator();
+        $this->stateGenerator = new StateGenerator;
     }
 
     /**

src/Mixins/Expectation.php

@@ -314,7 +314,7 @@ final class Expectation
      *
      * @return self<TValue>
      */
-    public function toHaveProperty(string $name, mixed $value = new Any(), string $message = ''): self
+    public function toHaveProperty(string $name, mixed $value = new Any, string $message = ''): self
     {
         $this->toBeObject();
 
@@ -654,7 +654,7 @@ final class Expectation
      *
      * @return self<TValue>
      */
-    public function toHaveKey(string|int $key, mixed $value = new Any(), string $message = ''): self
+    public function toHaveKey(string|int $key, mixed $value = new Any, string $message = ''): self
     {
         if (is_object($this->value) && method_exists($this->value, 'toArray')) {
             $array = $this->value->toArray();

src/Panic.php

@@ -42,7 +42,7 @@ final class Panic
         try {
             $output = Container::getInstance()->get(OutputInterface::class);
         } catch (Throwable) { // @phpstan-ignore-line
-            $output = new ConsoleOutput();
+            $output = new ConsoleOutput;
         }
 
         assert($output instanceof OutputInterface);

src/PendingCalls/AfterEachCall.php

@@ -39,7 +39,7 @@ final class AfterEachCall
     ) {
         $this->closure = $closure instanceof Closure ? $closure : NullClosure::create();
 
-        $this->proxies = new HigherOrderMessageCollection();
+        $this->proxies = new HigherOrderMessageCollection;
 
         $this->describing = DescribeCall::describing();
     }

src/PendingCalls/BeforeEachCall.php

@@ -44,8 +44,8 @@ final class BeforeEachCall
     ) {
         $this->closure = $closure instanceof Closure ? $closure : NullClosure::create();
 
-        $this->testCallProxies = new HigherOrderMessageCollection();
+        $this->testCallProxies = new HigherOrderMessageCollection;
-        $this->testCaseProxies = new HigherOrderMessageCollection();
+        $this->testCaseProxies = new HigherOrderMessageCollection;
 
         $this->describing = DescribeCall::describing();
     }

src/PendingCalls/TestCall.php

@@ -369,7 +369,7 @@ final class TestCall
      */
     public function coversNothing(): self
     {
-        $this->testCaseMethod->covers = [new CoversNothing()];
+        $this->testCaseMethod->covers = [new CoversNothing];
 
         return $this;
     }

src/Pest.php

@@ -6,7 +6,7 @@ namespace Pest;
 
 function version(): string
 {
-    return '2.34.9';
+    return '2.36.0';
 }
 
 function testDirectory(string $file = ''): string

src/Plugins/Help.php

@@ -97,7 +97,7 @@ final class Help implements HandlesArguments
      */
     private function getContent(): array
     {
-        $helpReflection = new PHPUnitHelp();
+        $helpReflection = new PHPUnitHelp;
 
         $content = (fn (): array => $this->elements())->call($helpReflection);
 

src/Plugins/Parallel.php

@@ -41,7 +41,7 @@ final class Parallel implements HandlesArguments
      */
     public static function isEnabled(): bool
     {
-        $argv = new ArgvInput();
+        $argv = new ArgvInput;
         if ($argv->hasParameterOption('--parallel')) {
             return true;
         }
@@ -126,7 +126,7 @@ final class Parallel implements HandlesArguments
             $arguments
         );
 
-        $exitCode = $this->paratestCommand()->run(new ArgvInput($filteredArguments), new CleanConsoleOutput());
+        $exitCode = $this->paratestCommand()->run(new ArgvInput($filteredArguments), new CleanConsoleOutput);
 
         return CallsAddsOutput::execute($exitCode);
     }
@@ -173,7 +173,7 @@ final class Parallel implements HandlesArguments
      */
     private function hasArgumentsThatWouldBeFasterWithoutParallel(): bool
     {
-        $arguments = new ArgvInput();
+        $arguments = new ArgvInput;
 
         foreach (self::UNSUPPORTED_ARGUMENTS as $unsupportedArgument) {
             if ($arguments->hasParameterOption($unsupportedArgument)) {

src/Plugins/Parallel/Paratest/ResultPrinter.php

@@ -169,7 +169,7 @@ final class ResultPrinter
             return;
         }
 
-        $state = (new StateGenerator())->fromPhpUnitTestResult($this->passedTests, $testResult);
+        $state = (new StateGenerator)->fromPhpUnitTestResult($this->passedTests, $testResult);
 
         $this->compactPrinter->errors($state);
         $this->compactPrinter->recap($state, $testResult, $duration, $this->options);

src/Plugins/Parallel/Paratest/WrapperRunner.php

@@ -91,13 +91,13 @@ final class WrapperRunner implements RunnerInterface
         private readonly OutputInterface $output
     ) {
         $this->printer = new ResultPrinter($output, $options);
-        $this->timer = new Timer();
+        $this->timer = new Timer;
 
         $wrapper = realpath(
             dirname(__DIR__, 4).DIRECTORY_SEPARATOR.'bin'.DIRECTORY_SEPARATOR.'worker.php',
         );
         assert($wrapper !== false);
-        $phpFinder = new PhpExecutableFinder();
+        $phpFinder = new PhpExecutableFinder;
         $phpBin = $phpFinder->find(false);
         assert($phpBin !== false);
         $parameters = [$phpBin];
@@ -110,7 +110,7 @@ final class WrapperRunner implements RunnerInterface
         $parameters[] = $wrapper;
 
         $this->parameters = $parameters;
-        $this->codeCoverageFilterRegistry = new CodeCoverageFilterRegistry();
+        $this->codeCoverageFilterRegistry = new CodeCoverageFilterRegistry;
     }
 
     public function run(): int
@@ -357,7 +357,7 @@ final class WrapperRunner implements RunnerInterface
             return;
         }
 
-        $coverageManager = new CodeCoverage();
+        $coverageManager = new CodeCoverage;
         $coverageManager->init(
             $this->options->configuration,
             $this->codeCoverageFilterRegistry,
@@ -389,8 +389,8 @@ final class WrapperRunner implements RunnerInterface
             return;
         }
 
-        $testSuite = (new LogMerger())->merge($this->junitFiles);
+        $testSuite = (new LogMerger)->merge($this->junitFiles);
-        (new Writer())->write(
+        (new Writer)->write(
             $testSuite,
             $this->options->configuration->logfileJunit(),
         );

src/Result.php

@@ -40,9 +40,20 @@ final class Result
      */
     public static function exitCode(Configuration $configuration, TestResult $result): int
     {
-        if ($result->wasSuccessfulIgnoringPhpunitWarnings()
+        if ($result->wasSuccessfulIgnoringPhpunitWarnings()) {
-            && ! $result->hasTestTriggeredPhpunitWarningEvents()) {
+            if ($configuration->failOnWarning()) {
-            return self::SUCCESS_EXIT;
+                $warnings = $result->numberOfTestsWithTestTriggeredPhpunitWarningEvents()
+                    + count($result->warnings())
+                    + count($result->phpWarnings());
+
+                if ($warnings > 0) {
+                    return self::FAILURE_EXIT;
+                }
+            }
+
+            if (! $result->hasTestTriggeredPhpunitWarningEvents()) {
+                return self::SUCCESS_EXIT;
+            }
         }
 
         if ($configuration->failOnEmptyTestSuite() && ResultReflection::numberOfTests($result) === 0) {
@@ -54,14 +65,6 @@ final class Result
                 $returnCode = self::FAILURE_EXIT;
             }
 
-            $warnings = $result->numberOfTestsWithTestTriggeredPhpunitWarningEvents()
-                + count($result->warnings())
-                + count($result->phpWarnings());
-
-            if ($configuration->failOnWarning() && $warnings > 0) {
-                $returnCode = self::FAILURE_EXIT;
-            }
-
             if ($configuration->failOnIncomplete() && $result->hasTestMarkedIncompleteEvents()) {
                 $returnCode = self::FAILURE_EXIT;
             }

src/Support/Container.php

@@ -26,7 +26,7 @@ final class Container
     public static function getInstance(): self
     {
         if (! self::$instance instanceof \Pest\Support\Container) {
-            self::$instance = new self();
+            self::$instance = new self;
         }
 
         return self::$instance;

src/Support/Coverage.php

@@ -37,7 +37,7 @@ final class Coverage
      */
     public static function isAvailable(): bool
     {
-        $runtime = new Runtime();
+        $runtime = new Runtime;
 
         if (! $runtime->canCollectCodeCoverage()) {
             return false;
@@ -67,7 +67,7 @@ final class Coverage
      */
     public static function usingXdebug(): bool
     {
-        return (new Runtime())->hasXdebug();
+        return (new Runtime)->hasXdebug();
     }
 
     /**

src/Support/Exporter.php

@@ -32,7 +32,7 @@ final class Exporter
     public static function default(): self
     {
         return new self(
-            new BaseExporter()
+            new BaseExporter
         );
     }
 
@@ -47,7 +47,7 @@ final class Exporter
         $array = $data;
         $itemsCount = 0;
         $exporter = self::default();
-        $context ??= new Context();
+        $context ??= new Context;
 
         $context->add($data);
 

src/Support/StateGenerator.php

@@ -20,7 +20,7 @@ final class StateGenerator
 {
     public function fromPhpUnitTestResult(int $passedTests, PHPUnitTestResult $testResult): State
     {
-        $state = new State();
+        $state = new State;
 
         foreach ($testResult->testErroredEvents() as $testResultEvent) {
             if ($testResultEvent instanceof Errored) {

src/TestCaseFilters/GitDirtyTestCaseFilter.php

@@ -76,7 +76,7 @@ final class GitDirtyTestCaseFilter implements TestCaseFilter
         $dirtyFiles = array_values($dirtyFiles);
 
         if ($dirtyFiles === []) {
-            Panic::with(new NoDirtyTestsFound());
+            Panic::with(new NoDirtyTestsFound);
         }
 
         $this->changedFiles = $dirtyFiles;

src/TestSuite.php

@@ -71,11 +71,11 @@ final class TestSuite
         string $rootPath,
         public string $testPath,
     ) {
-        $this->beforeAll = new BeforeAllRepository();
+        $this->beforeAll = new BeforeAllRepository;
-        $this->beforeEach = new BeforeEachRepository();
+        $this->beforeEach = new BeforeEachRepository;
-        $this->tests = new TestRepository();
+        $this->tests = new TestRepository;
-        $this->afterEach = new AfterEachRepository();
+        $this->afterEach = new AfterEachRepository;
-        $this->afterAll = new AfterAllRepository();
+        $this->afterAll = new AfterAllRepository;
         $this->rootPath = (string) realpath($rootPath);
         $this->snapshots = new SnapshotRepository(
             implode(DIRECTORY_SEPARATOR, [$this->rootPath, $this->testPath]),
@@ -101,7 +101,7 @@ final class TestSuite
         }
 
         if (! self::$instance instanceof self) {
-            Panic::with(new InvalidPestCommand());
+            Panic::with(new InvalidPestCommand);
         }
 
         return self::$instance;

tests/.pest/snapshots/Visual/Help/visual_snapshot_of_help_command_output.snap

@@ -1,5 +1,5 @@
 
-  Pest Testing Framework 2.34.8.  
+  Pest Testing Framework 2.36.0.  
 
   USAGE: pest <file> [options]  
 
@@ -59,6 +59,7 @@
   --fail-on-warning  Signal failure using shell exit code when a warning was triggered  
   --fail-on-risky  Signal failure using shell exit code when a test was considered risky  
   --fail-on-deprecation  Signal failure using shell exit code when a deprecation was triggered  
+  --fail-on-phpunit-deprecation  Signal failure using shell exit code when a PHPUnit deprecation was triggered  
   --fail-on-notice  Signal failure using shell exit code when a notice was triggered  
   --fail-on-skipped  Signal failure using shell exit code when a test was skipped  
   --fail-on-incomplete  Signal failure using shell exit code when a test was marked incomplete  
@@ -78,6 +79,7 @@
   --display-incomplete .................. Display details for incomplete tests  
   --display-skipped ........................ Display details for skipped tests  
   --display-deprecations . Display details for deprecations triggered by tests  
+  --display-phpunit-deprecations .... Display details for PHPUnit deprecations  
   --display-errors ............. Display details for errors triggered by tests  
   --display-notices ........... Display details for notices triggered by tests  
   --display-warnings ......... Display details for warnings triggered by tests  
@@ -105,6 +107,8 @@
   --coverage-html [dir]  Write code coverage report in HTML format to directory  
   --coverage-php [file] .......... Write serialized code coverage data to file  
   --coverage-text=[file]  Write code coverage report in text format to file [default: standard output]  
+  --only-summary-for-coverage-text  Option for code coverage report in text format: only show summary  
+  --show-uncovered-for-coverage-text  Option for code coverage report in text format: show uncovered files  
   --coverage-xml [dir] . Write code coverage report in XML format to directory  
   --warm-coverage-cache ........................... Warm static analysis cache  
   --coverage-filter [dir] ........... Include [dir] in code coverage reporting  

tests/.pest/snapshots/Visual/Version/visual_snapshot_of_help_command_output.snap

@@ -1,3 +1,3 @@
 
-  Pest Testing Framework 2.34.8.  
+  Pest Testing Framework 2.36.0.  
 

tests/.snapshots/success.txt

@@ -1173,16 +1173,6 @@
    PASS  Tests\Hooks\BeforeEachTest
   ✓ global beforeEach execution order
 
-   PASS  Tests\Overrides\VersionsTest
-  ✓ versions with dataset "Runner/Filter/NameFilterIterator.php"
-  ✓ versions with dataset "Runner/ResultCache/DefaultResultCache.php"
-  ✓ versions with dataset "Runner/TestSuiteLoader.php"
-  ✓ versions with dataset "TextUI/Command/Commands/WarmCodeCoverageCacheCommand.php"
-  ✓ versions with dataset "TextUI/Output/Default/ProgressPrinter/Subscriber/TestSkippedSubscriber.php"
-  ✓ versions with dataset "TextUI/TestSuiteFilterProcessor.php"
-  ✓ versions with dataset "Event/Value/ThrowableBuilder.php"
-  ✓ versions with dataset "Logging/JUnit/JunitXmlLogger.php"
-
    PASS  Tests\PHPUnit\CustomAffixes\InvalidTestName
   ✓ it runs file names like @#$%^&()-_=+.php
 
@@ -1287,26 +1277,6 @@
   ✓ it can resolve builtin value types
   ✓ it cannot resolve a parameter without type
 
-   PASS  Tests\Unit\Support\DatasetInfo
-  ✓ it can check if dataset is defined inside a Datasets directory with ('/var/www/project/tests/Datase…rs.php', true)
-  ✓ it can check if dataset is defined inside a Datasets directory with ('/var/www/project/tests/Datasets.php', false)
-  ✓ it can check if dataset is defined inside a Datasets directory with ('/var/www/project/tests/Featur…rs.php', true)
-  ✓ it can check if dataset is defined inside a Datasets directory with ('/var/www/project/tests/Featur…rs.php', false)
-  ✓ it can check if dataset is defined inside a Datasets directory with ('/var/www/project/tests/Featur…ts.php', false)
-  ✓ it can check if dataset is defined inside a Datasets.php file with ('/var/www/project/tests/Datase…rs.php', false)
-  ✓ it can check if dataset is defined inside a Datasets.php file with ('/var/www/project/tests/Datasets.php', true)
-  ✓ it can check if dataset is defined inside a Datasets.php file with ('/var/www/project/tests/Featur…rs.php', false) #1
-  ✓ it can check if dataset is defined inside a Datasets.php file with ('/var/www/project/tests/Featur…rs.php', false) #2
-  ✓ it can check if dataset is defined inside a Datasets.php file with ('/var/www/project/tests/Featur…ts.php', true)
-  ✓ it computes the dataset scope with ('/var/www/project/tests/Datase…rs.php', '/var/www/project/tests')
-  ✓ it computes the dataset scope with ('/var/www/project/tests/Datasets.php', '/var/www/project/tests')
-  ✓ it computes the dataset scope with ('/var/www/project/tests/Featur…rs.php', '/var/www/project/tests/Features')
-  ✓ it computes the dataset scope with ('/var/www/project/tests/Featur…rs.php', '/var/www/project/tests/Featur…rs.php') #1
-  ✓ it computes the dataset scope with ('/var/www/project/tests/Featur…ts.php', '/var/www/project/tests/Features')
-  ✓ it computes the dataset scope with ('/var/www/project/tests/Featur…rs.php', '/var/www/project/tests/Featur…ollers')
-  ✓ it computes the dataset scope with ('/var/www/project/tests/Featur…rs.php', '/var/www/project/tests/Featur…rs.php') #2
-  ✓ it computes the dataset scope with ('/var/www/project/tests/Featur…ts.php', '/var/www/project/tests/Featur…ollers')
-
    PASS  Tests\Unit\Support\ExceptionTrace
   ✓ it ensures the given closures reports the correct class name
   ✓ it ensures the given closures reports the correct class name and suggests the [uses()] function
@@ -1381,6 +1351,7 @@
 
    PASS  Tests\Unit\TestSuite
   ✓ it does not allow to add the same test description twice
+  ✓ it does not allow static closures
   ✓ it alerts users about tests with arguments but no input
   ✓ it can return an array of all test suite filenames
 
@@ -1424,4 +1395,4 @@
    WARN  Tests\Visual\Version
   - visual snapshot of help command output
 
-  Tests:    2 deprecated, 4 warnings, 5 incomplete, 2 notices, 13 todos, 20 skipped, 1013 passed (2405 assertions)
+  Tests:    2 deprecated, 4 warnings, 5 incomplete, 2 notices, 13 todos, 20 skipped, 988 passed (2381 assertions)

tests/Features/AfterEach.php

@@ -1,6 +1,6 @@
 <?php
 
-$state = new stdClass();
+$state = new stdClass;
 
 beforeEach(function () use ($state) {
     $this->state = $state;

tests/Features/BeforeAll.php

@@ -1,6 +1,6 @@
 <?php
 
-$foo = new \stdClass();
+$foo = new \stdClass;
 $foo->bar = 0;
 
 beforeAll(function () use ($foo) {

tests/Features/Coverage.php

@@ -7,7 +7,7 @@ use Symfony\Component\Console\Output\ConsoleOutput;
 it('has plugin')->assertTrue(class_exists(CoveragePlugin::class));
 
 it('adds coverage if --coverage exist', function () {
-    $plugin = new CoveragePlugin(new ConsoleOutput());
+    $plugin = new CoveragePlugin(new ConsoleOutput);
 
     expect($plugin->coverage)->toBeFalse();
     $arguments = $plugin->handleArguments([]);
@@ -20,7 +20,7 @@ it('adds coverage if --coverage exist', function () {
 })->skip(! \Pest\Support\Coverage::isAvailable() || ! function_exists('xdebug_info') || ! in_array('coverage', xdebug_info('mode'), true), 'Coverage is not available');
 
 it('adds coverage if --min exist', function () {
-    $plugin = new CoveragePlugin(new ConsoleOutput());
+    $plugin = new CoveragePlugin(new ConsoleOutput);
     expect($plugin->coverageMin)->toEqual(0.0)
         ->and($plugin->coverage)->toBeFalse();
 
@@ -35,7 +35,7 @@ it('adds coverage if --min exist', function () {
 });
 
 it('generates coverage based on file input', function () {
-    expect(Coverage::getMissingCoverage(new class()
+    expect(Coverage::getMissingCoverage(new class
     {
         public function lineCoverageData(): array
         {

tests/Features/DatasetsTests.php

@@ -46,7 +46,7 @@ test('it truncates the description', function () {
     // it gets tested by the integration test
 })->with([str_repeat('Fooo', 10)]);
 
-$state = new stdClass();
+$state = new stdClass;
 $state->text = '';
 
 $datasets = [[1], [2]];
@@ -119,7 +119,7 @@ class Bar
 }
 
 $namedDatasets = [
-    new Bar(),
+    new Bar,
 ];
 
 test('lazy named datasets', function ($text) {
@@ -132,12 +132,12 @@ it('creates unique test case names', function (string $name, Plugin $plugin, boo
     expect(true)->toBeTrue();
     $counter++;
 })->with([
-    ['Name 1', new Plugin(), true],
+    ['Name 1', new Plugin, true],
-    ['Name 1', new Plugin(), true],
+    ['Name 1', new Plugin, true],
-    ['Name 1', new Plugin(), false],
+    ['Name 1', new Plugin, false],
-    ['Name 2', new Plugin(), false],
+    ['Name 2', new Plugin, false],
-    ['Name 2', new Plugin(), true],
+    ['Name 2', new Plugin, true],
-    ['Name 1', new Plugin(), true],
+    ['Name 1', new Plugin, true],
 ]);
 
 it('creates unique test case names - count', function () use (&$counter) {
@@ -230,7 +230,7 @@ test('more than two datasets did the job right', function () use ($state) {
     expect($state->text)->toBe('121212121212131423241314232411122122111221221112212213142324135136145146235236245246');
 });
 
-$wrapped_generator_state = new stdClass();
+$wrapped_generator_state = new stdClass;
 $wrapped_generator_state->text = '';
 $wrapped_generator_function_datasets = [1, 2, 3, 4];
 

tests/Features/Exceptions.php

@@ -3,7 +3,7 @@
 it('gives access the the underlying expectException', function () {
     $this->expectException(InvalidArgumentException::class);
 
-    throw new InvalidArgumentException();
+    throw new InvalidArgumentException;
 });
 
 it('catch exceptions', function () {

tests/Features/Expect/HigherOrder/methods.php

@@ -1,29 +1,29 @@
 <?php
 
 it('can access methods', function () {
-    expect(new HasMethods())
+    expect(new HasMethods)
         ->name()->toBeString()->toEqual('Has Methods');
 });
 
 it('can access multiple methods', function () {
-    expect(new HasMethods())
+    expect(new HasMethods)
         ->name()->toBeString()->toEqual('Has Methods')
         ->quantity()->toBeInt()->toEqual(20);
 });
 
 it('works with not', function () {
-    expect(new HasMethods())
+    expect(new HasMethods)
         ->name()->not->toEqual('world')->toEqual('Has Methods')
         ->quantity()->toEqual(20)->not()->toEqual('bar')->not->toBeNull;
 });
 
 it('can accept arguments', function () {
-    expect(new HasMethods())
+    expect(new HasMethods)
         ->multiply(5, 4)->toBeInt->toEqual(20);
 });
 
 it('works with each', function () {
-    expect(new HasMethods())
+    expect(new HasMethods)
         ->attributes()->toBeArray->each->not()->toBeNull
         ->attributes()->each(function ($attribute) {
             $attribute->not->toBeNull();
@@ -31,14 +31,14 @@ it('works with each', function () {
 });
 
 it('works inside of each', function () {
-    expect(new HasMethods())
+    expect(new HasMethods)
         ->books()->each(function ($book) {
             $book->title->not->toBeNull->cost->toBeGreaterThan(19);
         });
 });
 
 it('works with sequence', function () {
-    expect(new HasMethods())
+    expect(new HasMethods)
         ->books()->sequence(
             function ($book) {
                 $book->title->toEqual('Foo')->cost->toEqual(20);
@@ -50,7 +50,7 @@ it('works with sequence', function () {
 });
 
 it('can compose complex expectations', function () {
-    expect(new HasMethods())
+    expect(new HasMethods)
         ->toBeObject()
         ->name()->toEqual('Has Methods')->not()->toEqual('bar')
         ->quantity()->not->toEqual('world')->toEqual(20)->toBeInt
@@ -68,7 +68,7 @@ it('can compose complex expectations', function () {
 });
 
 it('can handle nested method calls', function () {
-    expect(new HasMethods())
+    expect(new HasMethods)
         ->newInstance()->newInstance()->name()->toEqual('Has Methods')->toBeString()
         ->newInstance()->name()->toEqual('Has Methods')->not->toBeInt
         ->name()->toEqual('Has Methods')
@@ -76,14 +76,14 @@ it('can handle nested method calls', function () {
 });
 
 it('works with higher order tests')
-    ->expect(new HasMethods())
+    ->expect(new HasMethods)
     ->newInstance()->newInstance()->name()->toEqual('Has Methods')->toBeString()
     ->newInstance()->name()->toEqual('Has Methods')->not->toBeArray
     ->name()->toEqual('Has Methods')
     ->books()->each->toBeArray;
 
 it('can use the scoped method to lock into the given level for expectations', function () {
-    expect(new HasMethods())
+    expect(new HasMethods)
         ->attributes()->scoped(fn ($attributes) => $attributes
         ->name->toBe('Has Methods')
         ->quantity->toBe(20)
@@ -100,7 +100,7 @@ it('can use the scoped method to lock into the given level for expectations', fu
 });
 
 it('works consistently with the json expectation method', function () {
-    expect(new HasMethods())
+    expect(new HasMethods)
         ->jsonString()->json()->id->toBe(1)
         ->jsonString()->json()->name->toBe('Has Methods')->toBeString()
         ->jsonString()->json()->quantity->toBe(20)->toBeInt();
@@ -152,6 +152,6 @@ class HasMethods
 
     public function newInstance()
     {
-        return new static();
+        return new static;
     }
 }

tests/Features/Expect/HigherOrder/methodsAndProperties.php

@@ -1,7 +1,7 @@
 <?php
 
 it('can access methods and properties', function () {
-    expect(new HasMethodsAndProperties())
+    expect(new HasMethodsAndProperties)
         ->name->toEqual('Has Methods and Properties')->not()->toEqual('bar')
         ->multiply(3, 4)->not->toBeString->toEqual(12)
         ->posts->each(function ($post) {
@@ -19,7 +19,7 @@ it('can access methods and properties', function () {
 });
 
 it('can handle nested methods and properties', function () {
-    expect(new HasMethodsAndProperties())
+    expect(new HasMethodsAndProperties)
         ->meta->foo->bar->toBeString()->toEqual('baz')->not->toBeInt
         ->newInstance()->meta->foo->toBeArray()
         ->newInstance()->multiply(2, 2)->toEqual(4)->not->toEqual(5)
@@ -27,14 +27,14 @@ it('can handle nested methods and properties', function () {
 });
 
 it('works with higher order tests')
-    ->expect(new HasMethodsAndProperties())
+    ->expect(new HasMethodsAndProperties)
     ->meta->foo->bar->toBeString()->toEqual('baz')->not->toBeInt
     ->newInstance()->meta->foo->toBeArray
     ->newInstance()->multiply(2, 2)->toEqual(4)->not->toEqual(5)
     ->newInstance()->books()->toBeArray();
 
 it('can start a new higher order expectation using the and syntax', function () {
-    expect(new HasMethodsAndProperties())
+    expect(new HasMethodsAndProperties)
         ->toBeInstanceOf(HasMethodsAndProperties::class)
         ->meta->toBeArray
         ->and(['foo' => 'bar'])
@@ -45,7 +45,7 @@ it('can start a new higher order expectation using the and syntax', function ()
 });
 
 it('can start a new higher order expectation using the and syntax in higher order tests')
-    ->expect(new HasMethodsAndProperties())
+    ->expect(new HasMethodsAndProperties)
     ->toBeInstanceOf(HasMethodsAndProperties::class)
     ->meta->toBeArray
     ->and(['foo' => 'bar'])
@@ -53,7 +53,7 @@ it('can start a new higher order expectation using the and syntax in higher orde
     ->foo->toEqual('bar');
 
 it('can start a new higher order expectation using the and syntax without nesting expectations', function () {
-    expect(new HasMethodsAndProperties())
+    expect(new HasMethodsAndProperties)
         ->toBeInstanceOf(HasMethodsAndProperties::class)
         ->meta
         ->sequence(
@@ -101,6 +101,6 @@ class HasMethodsAndProperties
 
     public function newInstance()
     {
-        return new static();
+        return new static;
     }
 }

tests/Features/Expect/HigherOrder/properties.php

@@ -53,7 +53,7 @@ it('can compose complex expectations', function () {
 });
 
 it('works with objects', function () {
-    expect(new HasProperties())
+    expect(new HasProperties)
         ->name->toEqual('foo')->not->toEqual('world')
         ->posts->toHaveCount(2)->each(function ($post) {
             $post->is_published->toBeTrue();
@@ -69,13 +69,13 @@ it('works with objects', function () {
 });
 
 it('works with nested properties', function () {
-    expect(new HasProperties())
+    expect(new HasProperties)
         ->nested->foo->bar->toBeString()->toEqual('baz')
         ->posts->toBeArray()->toHaveCount(2);
 });
 
 it('works with higher order tests')
-    ->expect(new HasProperties())
+    ->expect(new HasProperties)
     ->nested->foo->bar->toBeString()->toEqual('baz')
     ->posts->toBeArray()->toHaveCount(2);
 

tests/Features/Expect/pipes.php

@@ -56,7 +56,7 @@ class State
     }
 }
 
-$state = new State();
+$state = new State;
 
 /*
  * Overrides toBe to assert two Characters are the same

tests/Features/Expect/toBe.php

@@ -5,8 +5,8 @@ use PHPUnit\Framework\ExpectationFailedException;
 expect(true)->toBeTrue()->and(false)->toBeFalse();
 
 test('strict comparisons', function () {
-    $nuno = new stdClass();
+    $nuno = new stdClass;
-    $dries = new stdClass();
+    $dries = new stdClass;
 
     expect($nuno)->toBe($nuno)->not->toBe($dries);
 });

tests/Features/Expect/toBeGreaterThan.php

@@ -8,8 +8,8 @@ test('passes', function () {
 });
 
 test('passes with DateTime and DateTimeImmutable', function () {
-    $now = new DateTime();
+    $now = new DateTime;
-    $past = (new DateTimeImmutable())->modify('-1 day');
+    $past = (new DateTimeImmutable)->modify('-1 day');
 
     expect($now)->toBeGreaterThan($past);
 

tests/Features/Expect/toBeGreaterThanOrEqual.php

@@ -8,8 +8,8 @@ test('passes', function () {
 });
 
 test('passes with DateTime and DateTimeImmutable', function () {
-    $now = new DateTime();
+    $now = new DateTime;
-    $past = (new DateTimeImmutable())->modify('-1 day');
+    $past = (new DateTimeImmutable)->modify('-1 day');
 
     expect($now)->toBeGreaterThanOrEqual($now);
 

tests/Features/Expect/toBeInstanceOf.php

@@ -3,18 +3,18 @@
 use PHPUnit\Framework\ExpectationFailedException;
 
 test('pass', function () {
-    expect(new Exception())->toBeInstanceOf(Exception::class);
+    expect(new Exception)->toBeInstanceOf(Exception::class);
-    expect(new Exception())->not->toBeInstanceOf(RuntimeException::class);
+    expect(new Exception)->not->toBeInstanceOf(RuntimeException::class);
 });
 
 test('failures', function () {
-    expect(new Exception())->toBeInstanceOf(RuntimeException::class);
+    expect(new Exception)->toBeInstanceOf(RuntimeException::class);
 })->throws(ExpectationFailedException::class);
 
 test('failures with custom message', function () {
-    expect(new Exception())->toBeInstanceOf(RuntimeException::class, 'oh no!');
+    expect(new Exception)->toBeInstanceOf(RuntimeException::class, 'oh no!');
 })->throws(ExpectationFailedException::class, 'oh no!');
 
 test('not failures', function () {
-    expect(new Exception())->not->toBeInstanceOf(Exception::class);
+    expect(new Exception)->not->toBeInstanceOf(Exception::class);
 })->throws(ExpectationFailedException::class);

tests/Features/Expect/toBeLessThan.php

@@ -8,8 +8,8 @@ test('passes', function () {
 });
 
 test('passes with DateTime and DateTimeImmutable', function () {
-    $now = new DateTime();
+    $now = new DateTime;
-    $past = (new DateTimeImmutable())->modify('-1 day');
+    $past = (new DateTimeImmutable)->modify('-1 day');
 
     expect($past)->toBeLessThan($now);
 

tests/Features/Expect/toBeLessThanOrEqual.php

@@ -8,8 +8,8 @@ test('passes', function () {
 });
 
 test('passes with DateTime and DateTimeImmutable', function () {
-    $now = new DateTime();
+    $now = new DateTime;
-    $past = (new DateTimeImmutable())->modify('-1 day');
+    $past = (new DateTimeImmutable)->modify('-1 day');
 
     expect($now)->toBeLessThanOrEqual($now);
 

tests/Features/Expect/toContainOnlyInstancesOf.php

@@ -3,7 +3,7 @@
 use PHPUnit\Framework\ExpectationFailedException;
 
 beforeEach(function () {
-    $this->times = [new DateTimeImmutable(), new DateTimeImmutable()];
+    $this->times = [new DateTimeImmutable, new DateTimeImmutable];
 });
 
 test('pass', function () {

tests/Features/Expect/toHaveProperties.php

@@ -3,7 +3,7 @@
 use PHPUnit\Framework\ExpectationFailedException;
 
 test('pass', function () {
-    $object = new stdClass();
+    $object = new stdClass;
     $object->name = 'John';
     $object->age = 21;
 
@@ -16,7 +16,7 @@ test('pass', function () {
 });
 
 test('failures', function () {
-    $object = new stdClass();
+    $object = new stdClass;
     $object->name = 'John';
 
     expect($object)
@@ -28,7 +28,7 @@ test('failures', function () {
 })->throws(ExpectationFailedException::class);
 
 test('failures with custom message', function () {
-    $object = new stdClass();
+    $object = new stdClass;
     $object->name = 'John';
 
     expect($object)
@@ -40,7 +40,7 @@ test('failures with custom message', function () {
 })->throws(ExpectationFailedException::class, 'oh no!');
 
 test('not failures', function () {
-    $object = new stdClass();
+    $object = new stdClass;
     $object->name = 'John';
     $object->age = 21;
 

tests/Features/Expect/toHaveProperty.php

@@ -2,7 +2,7 @@
 
 use PHPUnit\Framework\ExpectationFailedException;
 
-$obj = new stdClass();
+$obj = new stdClass;
 $obj->foo = 'bar';
 $obj->fooNull = null;
 

tests/Features/Expect/toMatchConstraint.php

@@ -4,17 +4,17 @@ use PHPUnit\Framework\Constraint\IsTrue;
 use PHPUnit\Framework\ExpectationFailedException;
 
 test('pass', function () {
-    expect(true)->toMatchConstraint(new IsTrue());
+    expect(true)->toMatchConstraint(new IsTrue);
 });
 
 test('failures', function () {
-    expect(false)->toMatchConstraint(new IsTrue());
+    expect(false)->toMatchConstraint(new IsTrue);
 })->throws(ExpectationFailedException::class);
 
 test('failures with custom message', function () {
-    expect(false)->toMatchConstraint(new IsTrue(), 'oh no!');
+    expect(false)->toMatchConstraint(new IsTrue, 'oh no!');
 })->throws(ExpectationFailedException::class, 'oh no!');
 
 test('not failures', function () {
-    expect(true)->not->toMatchConstraint(new IsTrue());
+    expect(true)->not->toMatchConstraint(new IsTrue);
 })->throws(ExpectationFailedException::class);

tests/Features/Expect/toMatchObject.php

@@ -18,7 +18,7 @@ test('pass', function () {
 });
 
 test('pass with class', function () {
-    expect(new class()
+    expect(new class
     {
         public $name = 'Nuno';
 

tests/Features/Expect/toThrow.php

@@ -6,13 +6,13 @@ class CustomException extends Exception {}
 
 test('passes', function () {
     expect(function () {
-        throw new RuntimeException();
+        throw new RuntimeException;
     })->toThrow(RuntimeException::class);
     expect(function () {
-        throw new RuntimeException();
+        throw new RuntimeException;
     })->toThrow(Exception::class);
     expect(function () {
-        throw new RuntimeException();
+        throw new RuntimeException;
     })->toThrow(function (RuntimeException $e) {});
     expect(function () {
         throw new RuntimeException('actual message');
@@ -24,7 +24,7 @@ test('passes', function () {
         throw new RuntimeException('actual message');
     })->toThrow('actual message');
     expect(function () {
-        throw new Exception();
+        throw new Exception;
     })->not->toThrow(RuntimeException::class);
     expect(function () {
         throw new RuntimeException('actual message');
@@ -47,7 +47,7 @@ test('failures 2', function () {
 
 test('failures 3', function () {
     expect(function () {
-        throw new Exception();
+        throw new Exception;
     })->toThrow(function (RuntimeException $e) {
         //
     });
@@ -92,7 +92,7 @@ test('failures with custom message', function () {
 
 test('not failures', function () {
     expect(function () {
-        throw new RuntimeException();
+        throw new RuntimeException;
     })->not->toThrow(RuntimeException::class);
 })->throws(ExpectationFailedException::class);
 
@@ -106,16 +106,16 @@ test('closure missing type-hint', function () {
 
 it('can handle a non-defined exception', function () {
     expect(function () {
-        throw new NonExistingException();
+        throw new NonExistingException;
     })->toThrow(NonExistingException::class);
 })->throws(Error::class, 'Class "NonExistingException" not found');
 
 it('can handle a class not found Error', function () {
     expect(function () {
-        throw new NonExistingException();
+        throw new NonExistingException;
     })->toThrow('Class "NonExistingException" not found');
 
     expect(function () {
-        throw new NonExistingException();
+        throw new NonExistingException;
     })->toThrow(Error::class, 'Class "NonExistingException" not found');
 });

tests/Features/Expect/unless.php

@@ -3,7 +3,7 @@
 use PHPUnit\Framework\ExpectationFailedException;
 
 beforeEach(function () {
-    $this->unlessObject = new stdClass();
+    $this->unlessObject = new stdClass;
     $this->unlessObject->trueValue = true;
     $this->unlessObject->foo = 'foo';
 });

tests/Features/Expect/when.php

@@ -3,7 +3,7 @@
 use PHPUnit\Framework\ExpectationFailedException;
 
 beforeEach(function () {
-    $this->whenObject = new stdClass();
+    $this->whenObject = new stdClass;
     $this->whenObject->trueValue = true;
     $this->whenObject->foo = 'foo';
 });

tests/Features/ScopedDatasets/Directory/NestedDirectory1/TestFileInNestedDirectoryWithDatasetsFile.php

@@ -1,6 +1,6 @@
 <?php
 
-$state = new stdClass();
+$state = new stdClass;
 $state->text = '';
 test('uses dataset', function ($value) use ($state) {
     $state->text .= $value;

tests/Features/ScopedDatasets/Directory/NestedDirectory2/TestFileInNestedDirectory.php

@@ -1,6 +1,6 @@
 <?php
 
-$state = new stdClass();
+$state = new stdClass;
 $state->text = '';
 test('uses dataset', function ($value) use ($state) {
     $state->text .= $value;

tests/Features/ScopedDatasets/Directory/TestFileWithLocallyDefinedDataset.php

@@ -4,7 +4,7 @@ dataset('numbers.array', [
     1, 2, 3, 4, 5, 'ScopedDatasets/ScopedDatasets.php',
 ]);
 
-$state = new stdClass();
+$state = new stdClass;
 $state->text = '';
 test('uses dataset', function ($value) use ($state) {
     $state->text .= $value;

tests/Features/ScopedDatasets/Directory/TestFileWithScopedDataset.php

@@ -1,6 +1,6 @@
 <?php
 
-$state = new stdClass();
+$state = new stdClass;
 $state->text = '';
 test('uses dataset', function ($value) use ($state) {
     $state->text .= $value;

tests/Features/ScopedDatasets/TestFileOutOfScope.php

@@ -1,6 +1,6 @@
 <?php
 
-$state = new stdClass();
+$state = new stdClass;
 $state->text = '';
 test('uses dataset', function ($value) use ($state) {
     $state->text .= $value;

tests/Features/TestCycle.php

@@ -2,7 +2,7 @@
 
 use function PHPUnit\Framework\assertFalse;
 
-$foo = new stdClass();
+$foo = new stdClass;
 $foo->beforeAll = false;
 $foo->beforeEach = false;
 $foo->afterEach = false;

tests/Overrides/VersionsTest.php

@@ -1,18 +0,0 @@
-<?php
-
-declare(strict_types=1);
-
-use Pest\Bootstrappers\BootOverrides;
-
-test('versions', function (string $vendorPath, string $expectedHash) {
-    expect(hash_file('sha256', $vendorPath))->toBe($expectedHash);
-})->with(function () {
-    foreach (BootOverrides::FILES as $hash => $file) {
-        $path = implode(DIRECTORY_SEPARATOR, [
-            dirname(__DIR__, 2),
-            'vendor/phpunit/phpunit/src',
-            $file,
-        ]);
-        yield $file => [$path, $hash];
-    }
-});

tests/Unit/Console/Help.php

@@ -4,7 +4,7 @@ use Pest\Console\Help;
 use Symfony\Component\Console\Output\BufferedOutput;
 
 it('outputs the help information when --help is used', function () {
-    $output = new BufferedOutput();
+    $output = new BufferedOutput;
     $plugin = new Help($output);
 
     $plugin();

tests/Unit/Plugins/Environment.php

@@ -5,7 +5,7 @@ use Pest\Plugins\Environment;
 test('environment is set to CI when --ci option is used', function () {
     $previousName = Environment::name();
 
-    $plugin = new Environment();
+    $plugin = new Environment;
 
     $plugin->handleArguments(['foo', '--ci', 'bar']);
 
@@ -15,7 +15,7 @@ test('environment is set to CI when --ci option is used', function () {
 });
 
 test('environment is set to Local when --ci option is not used', function () {
-    $plugin = new Environment();
+    $plugin = new Environment;
 
     $plugin->handleArguments(['foo', 'bar', 'baz']);
 

tests/Unit/Plugins/Retry.php

@@ -3,7 +3,7 @@
 use Pest\Plugins\Retry;
 
 it('orders by defects and stop on defects if when --retry is used ', function () {
-    $retry = new Retry();
+    $retry = new Retry;
 
     $arguments = $retry->handleArguments(['--retry']);
 

tests/Unit/Support/Container.php

@@ -7,7 +7,7 @@ use Pest\TestSuite;
 uses()->group('container');
 
 beforeEach(function () {
-    $this->container = new Container();
+    $this->container = new Container;
 });
 
 it('exists')

tests/Unit/Support/DatasetInfo.php

@@ -1,36 +0,0 @@
-<?php
-
-use Pest\Support\DatasetInfo;
-
-it('can check if dataset is defined inside a Datasets directory', function (string $file, bool $inside) {
-    expect(DatasetInfo::isInsideADatasetsDirectory($file))->toBe($inside);
-})->with([
-    ['file' => '/var/www/project/tests/Datasets/Numbers.php', 'inside' => true],
-    ['file' => '/var/www/project/tests/Datasets.php', 'inside' => false],
-    ['file' => '/var/www/project/tests/Features/Datasets/Numbers.php', 'inside' => true],
-    ['file' => '/var/www/project/tests/Features/Numbers.php', 'inside' => false],
-    ['file' => '/var/www/project/tests/Features/Datasets.php', 'inside' => false],
-]);
-
-it('can check if dataset is defined inside a Datasets.php file', function (string $path, bool $inside) {
-    expect(DatasetInfo::isADatasetsFile($path))->toBe($inside);
-})->with([
-    ['file' => '/var/www/project/tests/Datasets/Numbers.php', 'inside' => false],
-    ['file' => '/var/www/project/tests/Datasets.php', 'inside' => true],
-    ['file' => '/var/www/project/tests/Features/Datasets/Numbers.php', 'inside' => false],
-    ['file' => '/var/www/project/tests/Features/Numbers.php', 'inside' => false],
-    ['file' => '/var/www/project/tests/Features/Datasets.php', 'inside' => true],
-]);
-
-it('computes the dataset scope', function (string $file, string $scope) {
-    expect(DatasetInfo::scope($file))->toBe($scope);
-})->with([
-    ['file' => '/var/www/project/tests/Datasets/Numbers.php', 'scope' => '/var/www/project/tests'],
-    ['file' => '/var/www/project/tests/Datasets.php', 'scope' => '/var/www/project/tests'],
-    ['file' => '/var/www/project/tests/Features/Datasets/Numbers.php', 'scope' => '/var/www/project/tests/Features'],
-    ['file' => '/var/www/project/tests/Features/Numbers.php', 'scope' => '/var/www/project/tests/Features/Numbers.php'],
-    ['file' => '/var/www/project/tests/Features/Datasets.php', 'scope' => '/var/www/project/tests/Features'],
-    ['file' => '/var/www/project/tests/Features/Controllers/Datasets/Numbers.php', 'scope' => '/var/www/project/tests/Features/Controllers'],
-    ['file' => '/var/www/project/tests/Features/Controllers/Numbers.php', 'scope' => '/var/www/project/tests/Features/Controllers/Numbers.php'],
-    ['file' => '/var/www/project/tests/Features/Controllers/Datasets.php', 'scope' => '/var/www/project/tests/Features/Controllers'],
-]);

tests/Unit/Support/Reflection.php

@@ -9,7 +9,7 @@ it('gets file name from closure', function () {
 });
 
 it('gets property values', function () {
-    $class = new class()
+    $class = new class
     {
         private $foo = 'bar';
     };

tests/Unit/TestSuite.php

@@ -2,6 +2,7 @@
 
 use Pest\Exceptions\DatasetMissing;
 use Pest\Exceptions\TestAlreadyExist;
+use Pest\Exceptions\TestClosureMustNotBeStatic;
 use Pest\Factories\TestCaseMethodFactory;
 use Pest\TestSuite;
 
@@ -16,6 +17,16 @@ it('does not allow to add the same test description twice', function () {
     sprintf('A test with the description `%s` already exists in the filename `%s`.', 'bar', 'foo'),
 );
 
+it('does not allow static closures', function () {
+    $testSuite = new TestSuite(getcwd(), 'tests');
+    $method = new TestCaseMethodFactory('foo', 'bar', static function () {});
+
+    $testSuite->tests->set($method);
+})->throws(
+    TestClosureMustNotBeStatic::class,
+    'Test closure must not be static. Please remove the `static` keyword from the `bar` method in `foo`.',
+);
+
 it('alerts users about tests with arguments but no input', function () {
     $testSuite = new TestSuite(getcwd(), 'tests');
 

tests/Visual/Parallel.php

@@ -16,7 +16,7 @@ $run = function () {
 
 test('parallel', function () use ($run) {
     expect($run('--exclude-group=integration'))
-        ->toContain('Tests:    1 deprecated, 4 warnings, 5 incomplete, 2 notices, 13 todos, 16 skipped, 998 passed (2358 assertions)')
+        ->toContain('Tests:    1 deprecated, 4 warnings, 5 incomplete, 2 notices, 13 todos, 16 skipped, 973 passed (2334 assertions)')
         ->toContain('Parallel: 3 processes');
 })->skipOnWindows();