release: 2.34.9

[2.x] Added `toBeList` expectation

.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/)**
@@ -30,9 +32,7 @@ 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)
 - [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)
 - [Zapiet](https://www.zapiet.com/?ref=pestphp)
 
 Pest is an open-sourced software licensed under the **[MIT license](https://opensource.org/licenses/MIT)**.

composer.json

@@ -19,14 +19,14 @@
     "require": {
         "php": "^8.1.0",
         "brianium/paratest": "^7.3.1",
-        "nunomaduro/collision": "^7.10.0|^8.1.0",
+        "nunomaduro/collision": "^7.10.0|^8.1.1",
-        "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.4",
-        "symfony/process": "^6.4.0|^7.0.3"
+        "symfony/process": "^6.4.0|^7.1.1"
     },
     "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/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/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/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

@@ -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.
      *

src/Pest.php

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

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();

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/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

@@ -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 {});
-        });
     }
 }

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.6.  
+  Pest Testing Framework 2.34.8.  
 
   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.6.  
+  Pest Testing Framework 2.34.8.  
 

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/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/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/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/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/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,9 +2,7 @@
 
 use PHPUnit\Framework\ExpectationFailedException;
 
-class CustomException extends Exception
+class CustomException extends Exception {}
-{
-}
 
 test('passes', function () {
     expect(function () {
@@ -15,15 +13,13 @@ test('passes', function () {
     })->toThrow(Exception::class);
     expect(function () {
         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');
@@ -35,22 +31,18 @@ test('passes', function () {
     })->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 () {
@@ -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 () {
@@ -106,15 +97,11 @@ test('not failures', function () {
 })->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 () {

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 {}
-{
-}

tests/Fixtures/Covers/CoversClass1.php

@@ -4,7 +4,5 @@ namespace Tests\Fixtures\Covers;
 
 class CoversClass1
 {
-    public function foo()
+    public function foo() {}
-    {
-    }
 }

tests/Fixtures/Covers/CoversClass2.php

@@ -2,6 +2,4 @@
 
 namespace Tests\Fixtures\Covers;
 
-class CoversClass2
+class CoversClass2 {}
-{
-}

tests/Fixtures/Covers/CoversClass3.php

@@ -2,6 +2,4 @@
 
 namespace Tests\Fixtures\Covers;
 
-class CoversClass3
+class CoversClass3 {}
-{
-}

tests/Fixtures/Covers/CoversTrait.php

@@ -2,6 +2,4 @@
 
 namespace Tests\Fixtures\Covers;
 
-trait CoversTrait
+trait CoversTrait {}
-{
-}

tests/Unit/Support/Container.php

@@ -50,21 +50,15 @@ it('cannot resolve a parameter without type', function () {
 
 class ClassWithDependency
 {
-    public function __construct(Container $container)
+    public function __construct(Container $container) {}
-    {
-    }
 }
 
 class ClassWithSubDependency
 {
-    public function __construct(ClassWithDependency $param)
+    public function __construct(ClassWithDependency $param) {}
-    {
-    }
 }
 
 class ClassWithoutTypeParameter
 {
-    public function __construct($param)
+    public function __construct($param) {}
-    {
-    }
 }

tests/Unit/Support/Reflection.php

@@ -3,8 +3,7 @@
 use Pest\Support\Reflection;
 
 it('gets file name from closure', function () {
-    $fileName = Reflection::getFileNameFromClosure(function () {
+    $fileName = Reflection::getFileNameFromClosure(function () {});
-    });
 
     expect($fileName)->toBe(__FILE__);
 });

tests/Unit/TestSuite.php

@@ -19,8 +19,7 @@ it('does not allow to add the same test description twice', function () {
 it('alerts users about tests with arguments but no input', function () {
     $testSuite = new TestSuite(getcwd(), 'tests');
 
-    $method = new TestCaseMethodFactory('foo', 'bar', function (int $arg) {
+    $method = new TestCaseMethodFactory('foo', 'bar', function (int $arg) {});
-    });
 
     $testSuite->tests->set($method);
 })->throws(

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, 994 passed (2348 assertions)')
+        ->toContain('Tests:    1 deprecated, 4 warnings, 5 incomplete, 2 notices, 13 todos, 16 skipped, 998 passed (2358 assertions)')
         ->toContain('Parallel: 3 processes');
 })->skipOnWindows();