release: 2.35.1

Include method name in toHaveMethod error message

.github/workflows/tests.yml

@@ -3,8 +3,6 @@ name: Tests
 on:
   push:
   pull_request:
-  schedule:
-    - cron: '0 0 * * *'
 
 jobs:
   tests:

README.md

@@ -21,6 +21,8 @@ We cannot thank our sponsors enough for their incredible support in funding Pest
 
 ### Platinum Sponsors
 
+- **[LaraJobs](https://larajobs.com)**
+- **[Brokerchooser](https://brokerchooser.com)**
 - **[Forge](https://forge.laravel.com)**
 - **[Spatie](https://spatie.be)**
 - **[Worksome](https://www.worksome.com/)**
@@ -29,10 +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)
-- [Laradir](https://laradir.com/?ref=pestphp)
 - [Localazy](https://localazy.com/?ref=pestphp)
-- [Stormlikes](https://www.stormlikes.net/?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

@@ -18,15 +18,15 @@
     ],
     "require": {
         "php": "^8.1.0",
-        "brianium/paratest": "^7.4.1",
+        "brianium/paratest": "^7.3.1",
-        "nunomaduro/collision": "^7.10.0|^8.1.0",
+        "nunomaduro/collision": "^7.10.0|^8.4.0",
-        "nunomaduro/termwind": "^1.15.1|^2.0.0",
+        "nunomaduro/termwind": "^1.15.1|^2.0.1",
         "pestphp/pest-plugin": "^2.1.1",
         "pestphp/pest-plugin-arch": "^2.7.0",
-        "phpunit/phpunit": "^10.5.10"
+        "phpunit/phpunit": "^10.5.17"
     },
     "conflict": {
-        "phpunit/phpunit": ">10.5.10",
+        "phpunit/phpunit": ">10.5.17",
         "sebastian/exporter": "<5.1.0",
         "webmozart/assert": "<1.11.0"
     },
@@ -52,8 +52,8 @@
     },
     "require-dev": {
         "pestphp/pest-dev-tools": "^2.16.0",
-        "pestphp/pest-plugin-type-coverage": "^2.8.0",
+        "pestphp/pest-plugin-type-coverage": "^2.8.5",
-        "symfony/process": "^6.4.0|^7.0.3"
+        "symfony/process": "^6.4.0|^7.1.3"
     },
     "minimum-stability": "dev",
     "prefer-stable": true,

overrides/Runner/ResultCache/DefaultResultCache.php

@@ -59,7 +59,6 @@ use function file_get_contents;
 use function file_put_contents;
 use function is_array;
 use function is_dir;
-use function is_file;
 use function json_decode;
 use function json_encode;
 use function Pest\version;
@@ -129,13 +128,15 @@ final class DefaultResultCache implements ResultCache
 
     public function load(): void
     {
-        if (! is_file($this->cacheFilename)) {
+        $contents = @file_get_contents($this->cacheFilename);
+
+        if ($contents === false) {
             return;
         }
 
         $data = json_decode(
-            file_get_contents($this->cacheFilename),
+            $contents,
-            true
+            true,
         );
 
         if ($data === null) {

overrides/TextUI/TestSuiteFilterProcessor.php

@@ -59,19 +59,14 @@ use function array_map;
  */
 final class TestSuiteFilterProcessor
 {
-    private Factory $filterFactory;
-
-    public function __construct(Factory $factory = new Factory)
-    {
-        $this->filterFactory = $factory;
-    }
-
     /**
      * @throws Event\RuntimeException
      * @throws FilterNotConfiguredException
      */
     public function process(Configuration $configuration, TestSuite $suite): void
     {
+        $factory = new Factory;
+
         if (! $configuration->hasFilter() &&
             ! $configuration->hasGroups() &&
             ! $configuration->hasExcludeGroups() &&
@@ -83,21 +78,21 @@ final class TestSuiteFilterProcessor
         }
 
         if ($configuration->hasExcludeGroups()) {
-            $this->filterFactory->addExcludeGroupFilter(
+            $factory->addExcludeGroupFilter(
                 $configuration->excludeGroups()
             );
         }
 
         if (Only::isEnabled()) {
-            $this->filterFactory->addIncludeGroupFilter(['__pest_only']);
+            $factory->addIncludeGroupFilter(['__pest_only']);
         } elseif ($configuration->hasGroups()) {
-            $this->filterFactory->addIncludeGroupFilter(
+            $factory->addIncludeGroupFilter(
                 $configuration->groups()
             );
         }
 
         if ($configuration->hasTestsCovering()) {
-            $this->filterFactory->addIncludeGroupFilter(
+            $factory->addIncludeGroupFilter(
                 array_map(
                     static fn (string $name): string => '__phpunit_covers_'.$name,
                     $configuration->testsCovering()
@@ -106,7 +101,7 @@ final class TestSuiteFilterProcessor
         }
 
         if ($configuration->hasTestsUsing()) {
-            $this->filterFactory->addIncludeGroupFilter(
+            $factory->addIncludeGroupFilter(
                 array_map(
                     static fn (string $name): string => '__phpunit_uses_'.$name,
                     $configuration->testsUsing()
@@ -115,12 +110,12 @@ final class TestSuiteFilterProcessor
         }
 
         if ($configuration->hasFilter()) {
-            $this->filterFactory->addNameFilter(
+            $factory->addNameFilter(
                 $configuration->filter()
             );
         }
 
-        $suite->injectFilter($this->filterFactory);
+        $suite->injectFilter($factory);
 
         Event\Facade::emitter()->testSuiteFiltered(
             Event\TestSuite\TestSuiteBuilder::from($suite)

src/Bootstrappers/BootOverrides.php

@@ -19,11 +19,11 @@ final class BootOverrides implements Bootstrapper
      */
     public const FILES = [
         'c7b9c8a96006dea314204a8f09a8764e51ce0b9b79aadd58da52e8c328db4870' => 'Runner/Filter/NameFilterIterator.php',
-        '52b2574e96269aca1bb2d41bbf418c3bcf23dd21d14c66f90789025c309e39df' => 'Runner/ResultCache/DefaultResultCache.php',
+        'c7c09ab7c9378710b27f761a4b2948196cbbdf2a73e4389bcdca1e7c94fa9c21' => 'Runner/ResultCache/DefaultResultCache.php',
         'bc8718c89264f65800beabc23e51c6d3bcff87dfc764a12179ef5dbfde272c8b' => 'Runner/TestSuiteLoader.php',
         'f41e48d6cb546772a7de4f8e66b6b7ce894a5318d063eb52e354d206e96c701c' => 'TextUI/Command/Commands/WarmCodeCoverageCacheCommand.php',
         'cb7519f2d82893640b694492cf7ec9528da80773cc1d259634181b5d393528b5' => 'TextUI/Output/Default/ProgressPrinter/Subscriber/TestSkippedSubscriber.php',
-        '6db25ee539e9b12b1fb4e044a0a93410e015bc983ecdd3909cd394fe44ae8c95' => 'TextUI/TestSuiteFilterProcessor.php',
+        '2f06e4b1a9f3a24145bfc7ea25df4f124117f940a2cde30a04d04d5678006bff' => 'TextUI/TestSuiteFilterProcessor.php',
         'ef64a657ed9c0067791483784944107827bf227c7e3200f212b6751876b99e25' => 'Event/Value/ThrowableBuilder.php',
         'c78f96e34b98ed01dd8106539d59b8aa8d67f733274118b827c01c5c4111c033' => 'Logging/JUnit/JunitXmlLogger.php',
     ];

src/Bootstrappers/BootSubscribers.php

@@ -32,8 +32,7 @@ final class BootSubscribers implements Bootstrapper
      */
     public function __construct(
         private readonly Container $container,
-    ) {
+    ) {}
-    }
 
     /**
      * Boots the list of Subscribers.

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/Expectation.php

@@ -421,7 +421,7 @@ final class Expectation
      */
     public function any(): Any
     {
-        return new Any();
+        return new Any;
     }
 
     /**
@@ -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/EachExpectation.php

@@ -24,9 +24,7 @@ final class EachExpectation
      *
      * @param  Expectation<TValue>  $original
      */
-    public function __construct(private readonly Expectation $original)
+    public function __construct(private readonly Expectation $original) {}
-    {
-    }
 
     /**
      * Creates a new expectation.

src/Expectations/OppositeExpectation.php

@@ -36,9 +36,7 @@ final class OppositeExpectation
      *
      * @param  Expectation<TValue>  $original
      */
-    public function __construct(private readonly Expectation $original)
+    public function __construct(private readonly Expectation $original) {}
-    {
-    }
 
     /**
      * Asserts that the value array not has the provided $keys.

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/Covers/CoversClass.php

@@ -9,7 +9,5 @@ namespace Pest\Factories\Covers;
  */
 final class CoversClass
 {
-    public function __construct(public string $class)
+    public function __construct(public string $class) {}
-    {
-    }
 }

src/Factories/Covers/CoversFunction.php

@@ -9,7 +9,5 @@ namespace Pest\Factories\Covers;
  */
 final class CoversFunction
 {
-    public function __construct(public string $function)
+    public function __construct(public string $function) {}
-    {
-    }
 }

src/Factories/Covers/CoversNothing.php

@@ -7,6 +7,4 @@ namespace Pest\Factories\Covers;
 /**
  * @internal
  */
-final class CoversNothing
+final class CoversNothing {}
-{
-}

src/Factories/TestCaseFactory.php

@@ -154,7 +154,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
             );
         }

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/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/Logging/TeamCity/ServiceMessage.php

@@ -17,8 +17,7 @@ final class ServiceMessage
     public function __construct(
         private readonly string $type,
         private readonly array $parameters,
-    ) {
+    ) {}
-    }
 
     public function toString(): string
     {

src/Logging/TeamCity/Subscriber/Subscriber.php

@@ -14,9 +14,7 @@ abstract class Subscriber
     /**
      * Creates a new Subscriber instance.
      */
-    public function __construct(private readonly TeamCityLogger $logger)
+    public function __construct(private readonly TeamCityLogger $logger) {}
-    {
-    }
 
     /**
      * Creates a new TeamCityLogger instance.

src/Matchers/Any.php

@@ -7,6 +7,4 @@ namespace Pest\Matchers;
 /**
  * @internal
  */
-final class Any
+final class Any {}
-{
-}

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();
 
@@ -467,6 +467,18 @@ final class Expectation
         return $this;
     }
 
+    /**
+     * Asserts that the value is a list.
+     *
+     * @return self<TValue>
+     */
+    public function toBeList(string $message = ''): self
+    {
+        Assert::assertIsList($this->value, $message);
+
+        return $this;
+    }
+
     /**
      * Asserts that the value is of type bool.
      *
@@ -642,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.33.5';
+    return '2.35.1';
 }
 
 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

@@ -63,8 +63,7 @@ final class ResultPrinter
         {
             public function __construct(
                 private readonly OutputInterface $output,
-            ) {
+            ) {}
-            }
 
             public function print(string $buffer): void
             {
@@ -79,9 +78,7 @@ final class ResultPrinter
                 $this->output->write(OutputFormatter::escape($buffer));
             }
 
-            public function flush(): void
+            public function flush(): void {}
-            {
-            }
         };
 
         $this->compactPrinter = CompactPrinter::default();
@@ -172,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/Plugins/Printer.php

@@ -22,6 +22,10 @@ final class Printer implements HandlesArguments
             return $arguments;
         }
 
+        if (in_array('--no-output', $arguments, true)) {
+            return $arguments;
+        }
+
         return $this->pushArgument('--no-output', $arguments);
     }
 }

src/Repositories/SnapshotRepository.php

@@ -21,8 +21,7 @@ final class SnapshotRepository
     public function __construct(
         readonly private string $testsPath,
         readonly private string $snapshotsPath,
-    ) {
+    ) {}
-    }
 
     /**
      * Checks if the snapshot exists.

src/Subscribers/EnsureTeamCityEnabled.php

@@ -24,8 +24,7 @@ final class EnsureTeamCityEnabled implements ConfiguredSubscriber
         private readonly InputInterface $input,
         private readonly OutputInterface $output,
         private readonly TestSuite $testSuite,
-    ) {
+    ) {}
-    }
 
     /**
      * Runs the subscriber.

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/ExpectationPipeline.php

@@ -30,8 +30,7 @@ final class ExpectationPipeline
      */
     public function __construct(
         private readonly Closure $closure
-    ) {
+    ) {}
-    }
 
     /**
      * Creates a new instance of Expectation Pipeline with given closure.

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);
 
@@ -64,6 +64,8 @@ final class Exporter
                 continue;
             }
 
+            assert(is_array($data));
+
             $result[] = $context->contains($data[$key]) !== false
                 ? '*RECURSION*'
                 : sprintf('[%s]', $this->shortenedRecursiveExport($data[$key], $context));

src/Support/NullClosure.php

@@ -16,7 +16,6 @@ final class NullClosure
      */
     public static function create(): Closure
     {
-        return Closure::fromCallable(function (): void {
+        return Closure::fromCallable(function (): void {});
-        });
     }
 }

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;

stubs/init-laravel/TestCase.php.stub

@@ -6,5 +6,5 @@ use Illuminate\Foundation\Testing\TestCase as BaseTestCase;
 
 abstract class TestCase extends BaseTestCase
 {
-    use CreatesApplication;
+    //
 }

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

@@ -1,5 +1,5 @@
 
-  Pest Testing Framework 2.33.5.  
+  Pest Testing Framework 2.35.0.  
 
   USAGE: pest <file> [options]  
 
@@ -55,6 +55,7 @@
   --stop-on-notice ............. Stop after first test that triggered a notice  
   --stop-on-skipped ............................ Stop after first skipped test  
   --stop-on-incomplete ...................... Stop after first incomplete test  
+  --fail-on-empty-test-suite  Signal failure using shell exit code when no tests were run  
   --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  

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

@@ -1,3 +1,3 @@
 
-  Pest Testing Framework 2.33.5.  
+  Pest Testing Framework 2.35.0.  
 

tests/.snapshots/success.txt

@@ -511,6 +511,12 @@
   ✓ passes with strings
   ✓ failures
   ✓ failures with custom message
+  ✓ not failures
+
+   PASS  Tests\Features\Expect\toBeList
+  ✓ pass
+  ✓ failures
+  ✓ failures with custom message
   ✓ not failures
 
    PASS  Tests\Features\Expect\toBeLowercase
@@ -1418,4 +1424,4 @@
    WARN  Tests\Visual\Version
   - visual snapshot of help command output
 
-  Tests:    2 deprecated, 4 warnings, 5 incomplete, 2 notices, 13 todos, 20 skipped, 1009 passed (2395 assertions)
+  Tests:    2 deprecated, 4 warnings, 5 incomplete, 2 notices, 13 todos, 20 skipped, 1013 passed (2405 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/Covers.php

@@ -10,9 +10,7 @@ use Tests\Fixtures\Covers\CoversTrait;
 
 $runCounter = 0;
 
-function testCoversFunction()
+function testCoversFunction() {}
-{
-}
 
 it('uses the correct PHPUnit attribute for class', function () {
     $attributes = (new ReflectionClass($this))->getAttributes();

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/toBeCallable.php

@@ -3,8 +3,7 @@
 use PHPUnit\Framework\ExpectationFailedException;
 
 test('pass', function () {
-    expect(function () {
+    expect(function () {})->toBeCallable();
-    })->toBeCallable();
     expect(null)->not->toBeCallable();
 });
 

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/toBeList.php

@@ -0,0 +1,21 @@
+<?php
+
+use PHPUnit\Framework\ExpectationFailedException;
+
+test('pass', function () {
+    expect([1, 2, 3])->toBeList();
+    expect(['a' => 1, 'b' => 2, 'c' => 3])->not->toBeList();
+    expect('1, 2, 3')->not->toBeList();
+});
+
+test('failures', function () {
+    expect(null)->toBeList();
+})->throws(ExpectationFailedException::class);
+
+test('failures with custom message', function () {
+    expect(null)->toBeList('oh no!');
+})->throws(ExpectationFailedException::class, 'oh no!');
+
+test('not failures', function () {
+    expect(['a', 'b', 'c'])->not->toBeList();
+})->throws(ExpectationFailedException::class);

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/toHaveMethods.php

@@ -4,13 +4,9 @@ use PHPUnit\Framework\ExpectationFailedException;
 
 $object = new class
 {
-    public function foo(): void
+    public function foo(): void {}
-    {
-    }
 
-    public function bar(): void
+    public function bar(): void {}
-    {
-    }
 };
 
 test('pass', function () use ($object) {

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/toMatchSnapshot.php

@@ -43,9 +43,7 @@ test('pass with `__toString`', function () {
 
     $object = new class($this->snapshotable)
     {
-        public function __construct(protected string $snapshotable)
+        public function __construct(protected string $snapshotable) {}
-        {
-        }
 
         public function __toString()
         {
@@ -61,9 +59,7 @@ test('pass with `toString`', function () {
 
     $object = new class($this->snapshotable)
     {
-        public function __construct(protected string $snapshotable)
+        public function __construct(protected string $snapshotable) {}
-        {
-        }
 
         public function toString()
         {
@@ -97,9 +93,7 @@ test('pass with `toArray`', function () {
 
     $object = new class($this->snapshotable)
     {
-        public function __construct(protected string $snapshotable)
+        public function __construct(protected string $snapshotable) {}
-        {
-        }
 
         public function toArray()
         {
@@ -125,9 +119,7 @@ test('pass with `toSnapshot`', function () {
 
     $object = new class($this->snapshotable)
     {
-        public function __construct(protected string $snapshotable)
+        public function __construct(protected string $snapshotable) {}
-        {
-        }
 
         public function toSnapshot()
         {

tests/Features/Expect/toThrow.php

@@ -2,60 +2,52 @@
 
 use PHPUnit\Framework\ExpectationFailedException;
 
-class CustomException extends Exception
+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) {
+    })->toThrow(function (RuntimeException $e) {});
-    });
     expect(function () {
         throw new RuntimeException('actual message');
     })->toThrow(function (Exception $e) {
         expect($e->getMessage())->toBe('actual message');
     });
-    expect(function () {
+    expect(function () {})->not->toThrow(Exception::class);
-    })->not->toThrow(Exception::class);
     expect(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');
     })->toThrow(RuntimeException::class, 'actual message');
     expect(function () {
         throw new RuntimeException('actual message');
-    })->toThrow(function (RuntimeException $e) {
+    })->toThrow(function (RuntimeException $e) {}, 'actual message');
-    }, 'actual message');
     expect(function () {
         throw new CustomException('foo');
     })->toThrow(new CustomException('foo'));
 });
 
 test('failures 1', function () {
-    expect(function () {
+    expect(function () {})->toThrow(RuntimeException::class);
-    })->toThrow(RuntimeException::class);
 })->throws(ExpectationFailedException::class, 'Exception "'.RuntimeException::class.'" not thrown.');
 
 test('failures 2', function () {
-    expect(function () {
+    expect(function () {})->toThrow(function (RuntimeException $e) {});
-    })->toThrow(function (RuntimeException $e) {
-    });
 })->throws(ExpectationFailedException::class, 'Exception "'.RuntimeException::class.'" not thrown.');
 
 test('failures 3', function () {
     expect(function () {
-        throw new Exception();
+        throw new Exception;
     })->toThrow(function (RuntimeException $e) {
         //
     });
@@ -77,8 +69,7 @@ test('failures 5', function () {
 })->throws(ExpectationFailedException::class, 'Failed asserting that \'actual message\' [ASCII](length: 14) contains "expected message" [ASCII](length: 16).');
 
 test('failures 6', function () {
-    expect(function () {
+    expect(function () {})->toThrow('actual message');
-    })->toThrow('actual message');
 })->throws(ExpectationFailedException::class, 'Exception with message "actual message" not thrown');
 
 test('failures 7', function () {
@@ -101,34 +92,30 @@ 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);
 
 test('closure missing parameter', function () {
-    expect(function () {
+    expect(function () {})->toThrow(function () {});
-    })->toThrow(function () {
-    });
 })->throws(InvalidArgumentException::class, 'The given closure must have a single parameter type-hinted as the class string.');
 
 test('closure missing type-hint', function () {
-    expect(function () {
+    expect(function () {})->toThrow(function ($e) {});
-    })->toThrow(function ($e) {
-    });
 })->throws(InvalidArgumentException::class, 'The given closure\'s parameter must be type-hinted as the class string.');
 
 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/Fixtures/Arch/ToBeInvokable/IsInvokable/InvokableClass.php

@@ -6,8 +6,5 @@ namespace Tests\Fixtures\Arch\ToBeInvokable\IsInvokable;
 
 class InvokableClass
 {
-    public function __invoke(): void
+    public function __invoke(): void {}
-    {
-
-    }
 }

tests/Fixtures/Arch/ToBeInvokable/IsInvokable/ParentInvokableClass.php

@@ -6,8 +6,5 @@ namespace Tests\Fixtures\Arch\ToBeInvokable\IsInvokable;
 
 class ParentInvokableClass
 {
-    public function __invoke(): void
+    public function __invoke(): void {}
-    {
-
-    }
 }

tests/Fixtures/Arch/ToBeInvokable/IsNotInvokable/IsNotInvokableClass.php

@@ -6,8 +6,5 @@ namespace Tests\Fixtures\Arch\ToBeInvokable\IsNotInvokable;
 
 class IsNotInvokableClass
 {
-    public function handle(): void
+    public function handle(): void {}
-    {
-
-    }
 }

tests/Fixtures/Arch/ToHaveAttribute/Attributes/AsAttribute.php

@@ -7,6 +7,4 @@ namespace Tests\Fixtures\Arch\ToHaveAttribute\Attributes;
 use Attribute;
 
 #[Attribute()]
-class AsAttribute
+class AsAttribute {}
-{
-}

tests/Fixtures/Arch/ToHaveAttribute/HaveAttribute/HaveAttributeClass.php

@@ -7,6 +7,4 @@ namespace Tests\Fixtures\Arch\ToHaveAttribute\HaveAttribute;
 use Tests\Fixtures\Arch\ToHaveAttribute\Attributes\AsAttribute;
 
 #[AsAttribute]
-class HaveAttributeClass
+class HaveAttributeClass {}
-{
-}

tests/Fixtures/Arch/ToHaveAttribute/NotHaveAttribute/NotHaveAttributeClass.php

@@ -4,6 +4,4 @@ declare(strict_types=1);
 
 namespace Tests\Fixtures\Arch\ToHaveAttribute\NotHaveAttribute;
 
-class NotHaveAttributeClass
+class NotHaveAttributeClass {}
-{
-}

tests/Fixtures/Arch/ToHaveConstructor/HasConstructor/HasConstructor.php

@@ -6,8 +6,5 @@ namespace Tests\Fixtures\Arch\ToHaveConstructor\HasConstructor;
 
 class HasConstructor
 {
-    public function __construct()
+    public function __construct() {}
-    {
-
-    }
 }

tests/Fixtures/Arch/ToHaveConstructor/HasNoConstructor/HasNoConstructor.php

@@ -4,6 +4,4 @@ declare(strict_types=1);
 
 namespace Tests\Fixtures\Arch\ToHaveConstructor\HasNoConstructor;
 
-class HasNoConstructor
+class HasNoConstructor {}
-{
-}

tests/Fixtures/Arch/ToHaveDestructor/HasDestructor/HasDestructor.php

@@ -6,8 +6,5 @@ namespace Tests\Fixtures\Arch\ToHaveDestructor\HasDestructor;
 
 class HasDestructor
 {
-    public function __destruct()
+    public function __destruct() {}
-    {
-
-    }
 }

tests/Fixtures/Arch/ToHaveDestructor/HasNoDestructor/HasNoDestructor.php

@@ -4,6 +4,4 @@ declare(strict_types=1);
 
 namespace Tests\Fixtures\Arch\ToHaveDestructor\HasNoDestructor;
 
-class HasNoDestructor
+class HasNoDestructor {}
-{
-}

tests/Fixtures/Arch/ToHaveMethod/HasMethod/HasMethod.php

@@ -6,8 +6,5 @@ namespace Tests\Fixtures\Arch\ToHaveMethod\HasMethod;
 
 class HasMethod
 {
-    public function foo(): void
+    public function foo(): void {}
-    {
-
-    }
 }

tests/Fixtures/Arch/ToHaveMethod/HasMethod/HasMethodTrait.php

@@ -6,8 +6,5 @@ namespace Tests\Fixtures\Arch\ToHaveMethod\HasMethod;
 
 trait HasMethodTrait
 {
-    public function foo(): void
+    public function foo(): void {}
-    {
-
-    }
 }

tests/Fixtures/Arch/ToHaveMethod/HasMethod/ParentHasMethodClass.php

@@ -6,8 +6,5 @@ namespace Tests\Fixtures\Arch\ToHaveMethod\HasMethod;
 
 class ParentHasMethodClass
 {
-    public function foo(): void
+    public function foo(): void {}
-    {
-
-    }
 }

tests/Fixtures/Arch/ToHaveMethod/HasNoMethod/HasNoMethodClass.php

@@ -6,8 +6,5 @@ namespace Tests\Fixtures\Arch\ToHaveMethod\HasNoMethod;
 
 class HasNoMethodClass
 {
-    public function bar(): void
+    public function bar(): void {}
-    {
-
-    }
 }

tests/Fixtures/Arch/ToHavePrefix/HasNoPrefix/ClassWithout.php

@@ -4,6 +4,4 @@ declare(strict_types=1);
 
 namespace Tests\Fixtures\Arch\ToHavePrefix\HasNoPrefix;
 
-class ClassWithout
+class ClassWithout {}
-{
-}

tests/Fixtures/Arch/ToHavePrefix/HasPrefix/PrefixClassWith.php

@@ -4,6 +4,4 @@ declare(strict_types=1);
 
 namespace Tests\Fixtures\Arch\ToHavePrefix\HasPrefix;
 
-class PrefixClassWith
+class PrefixClassWith {}
-{
-}

tests/Fixtures/Arch/ToHaveSuffix/HasNoSuffix/ClassWithout.php

@@ -4,6 +4,4 @@ declare(strict_types=1);
 
 namespace Tests\Fixtures\Arch\ToHaveSuffix\HasNoSuffix;
 
-class ClassWithout
+class ClassWithout {}
-{
-}

tests/Fixtures/Arch/ToHaveSuffix/HasSuffix/ClassWithSuffix.php

@@ -4,6 +4,4 @@ declare(strict_types=1);
 
 namespace Tests\Fixtures\Arch\ToHaveSuffix\HasSuffix;
 
-class ClassWithSuffix
+class ClassWithSuffix {}
-{
-}