release: v4.6.2

.github/workflows/static.yml

@@ -2,7 +2,7 @@ name: Static Analysis
 
 on:
   push:
-    branches: [5.x]
+    branches: [4.x]
   pull_request:
   schedule:
     - cron: '0 9 * * *'
@@ -33,7 +33,7 @@ jobs:
     - name: Setup PHP
       uses: shivammathur/setup-php@v2
       with:
-        php-version: 8.4
+        php-version: 8.3
         tools: composer:v2
         coverage: none
         extensions: sockets
@@ -47,10 +47,10 @@ jobs:
       uses: actions/cache@v5
       with:
         path: ${{ steps.composer-cache.outputs.dir }}
-        key: static-php-8.4-${{ matrix.dependency-version }}-composer-${{ hashFiles('**/composer.json', '**/composer.lock') }}
+        key: static-php-8.3-${{ matrix.dependency-version }}-composer-${{ hashFiles('**/composer.json', '**/composer.lock') }}
         restore-keys: |
-          static-php-8.4-${{ matrix.dependency-version }}-composer-
+          static-php-8.3-${{ matrix.dependency-version }}-composer-
-          static-php-8.4-composer-
+          static-php-8.3-composer-
 
     - name: Install Dependencies
       run: composer update --${{ matrix.dependency-version }} --no-interaction --no-progress --ansi

.github/workflows/tests.yml

@@ -2,7 +2,7 @@ name: Tests
 
 on:
   push:
-    branches: [5.x]
+    branches: [4.x]
   pull_request:
   schedule:
     - cron: '0 9 * * *'
@@ -24,9 +24,12 @@ jobs:
       fail-fast: true
       matrix:
         os: [ubuntu-latest, macos-latest] # windows-latest
-        symfony: ['8.0']
+        symfony: ['7.4', '8.0']
-        php: ['8.4', '8.5']
+        php: ['8.3', '8.4', '8.5']
         dependency_version: [prefer-stable]
+        exclude:
+          - php: '8.3'
+            symfony: '8.0'
 
     name: PHP ${{ matrix.php }} - Symfony ^${{ matrix.symfony }} - ${{ matrix.os }} - ${{ matrix.dependency_version }}
 

composer.json

@@ -17,20 +17,20 @@
         }
     ],
     "require": {
-        "php": "^8.4",
+        "php": "^8.3.0",
-        "brianium/paratest": "^7.22.3",
+        "brianium/paratest": "^7.20.0",
         "nunomaduro/collision": "^8.9.3",
         "nunomaduro/termwind": "^2.4.0",
-        "pestphp/pest-plugin": "^5.0.0",
+        "pestphp/pest-plugin": "^4.0.0",
-        "pestphp/pest-plugin-arch": "^5.0.0",
+        "pestphp/pest-plugin-arch": "^4.0.2",
-        "pestphp/pest-plugin-mutate": "^5.0.0",
+        "pestphp/pest-plugin-mutate": "^4.0.1",
-        "pestphp/pest-plugin-profanity": "^5.0.0",
+        "pestphp/pest-plugin-profanity": "^4.2.1",
-        "phpunit/phpunit": "^13.1.6",
+        "phpunit/phpunit": "^12.5.22",
-        "symfony/process": "^8.1.0"
+        "symfony/process": "^7.4.8|^8.0.8"
     },
     "conflict": {
         "filp/whoops": "<2.18.3",
-        "phpunit/phpunit": ">13.1.6",
+        "phpunit/phpunit": ">12.5.22",
         "sebastian/exporter": "<7.0.0",
         "webmozart/assert": "<1.11.0"
     },
@@ -59,10 +59,9 @@
     },
     "require-dev": {
         "mrpunyapal/peststan": "^0.2.5",
-        "nunomaduro/pao": "0.x-dev",
+        "pestphp/pest-dev-tools": "^4.1.0",
-        "pestphp/pest-dev-tools": "^5.0.0",
+        "pestphp/pest-plugin-browser": "^4.3.1",
-        "pestphp/pest-plugin-browser": "^5.0.0",
+        "pestphp/pest-plugin-type-coverage": "^4.0.4",
-        "pestphp/pest-plugin-type-coverage": "^5.0.0",
         "psy/psysh": "^0.12.22"
     },
     "minimum-stability": "dev",

src/Configuration.php

@@ -33,7 +33,7 @@ final readonly class Configuration
      */
     public function in(string ...$targets): UsesCall
     {
-        return new UsesCall($this->filename, [])->in(...$targets);
+        return (new UsesCall($this->filename, []))->in(...$targets);
     }
 
     /**
@@ -60,7 +60,7 @@ final readonly class Configuration
      */
     public function group(string ...$groups): UsesCall
     {
-        return new UsesCall($this->filename, [])->group(...$groups);
+        return (new UsesCall($this->filename, []))->group(...$groups);
     }
 
     /**
@@ -68,7 +68,7 @@ final readonly class Configuration
      */
     public function only(): void
     {
-        new BeforeEachCall(TestSuite::getInstance(), $this->filename)->only();
+        (new BeforeEachCall(TestSuite::getInstance(), $this->filename))->only();
     }
 
     /**

src/Expectation.php

@@ -238,7 +238,7 @@ final class Expectation
             if ($callbacks[$index] instanceof Closure) {
                 $callbacks[$index](new self($value), new self($key));
             } else {
-                new self($value)->toEqual($callbacks[$index]);
+                (new self($value))->toEqual($callbacks[$index]);
             }
 
             $index = isset($callbacks[$index + 1]) ? $index + 1 : 0;
@@ -915,7 +915,15 @@ final class Expectation
 
         return Targeted::make(
             $this,
-            fn (ObjectDescription $object): bool => array_all($interfaces, fn (string $interface): bool => isset($object->reflectionClass) && $object->reflectionClass->implementsInterface($interface)),
+            function (ObjectDescription $object) use ($interfaces): bool {
+                foreach ($interfaces as $interface) {
+                    if (! isset($object->reflectionClass) || ! $object->reflectionClass->implementsInterface($interface)) {
+                        return false;
+                    }
+                }
+
+                return true;
+            },
             "to implement '".implode("', '", $interfaces)."'",
             FileLineFinder::where(fn (string $line): bool => str_contains($line, 'class')),
         );
@@ -1130,8 +1138,8 @@ final class Expectation
             $this,
             fn (ObjectDescription $object): bool => isset($object->reflectionClass)
                 && $object->reflectionClass->isEnum()
-                && new ReflectionEnum($object->name)->isBacked() // @phpstan-ignore-line
+                && (new ReflectionEnum($object->name))->isBacked() // @phpstan-ignore-line
-                && (string) new ReflectionEnum($object->name)->getBackingType() === $backingType, // @phpstan-ignore-line
+                && (string) (new ReflectionEnum($object->name))->getBackingType() === $backingType, // @phpstan-ignore-line
             'to be '.$backingType.' backed enum',
             FileLineFinder::where(fn (string $line): bool => str_contains($line, 'class')),
         );

src/Expectations/OppositeExpectation.php

@@ -576,7 +576,15 @@ final readonly class OppositeExpectation
 
         return Targeted::make(
             $original,
-            fn (ObjectDescription $object): bool => array_all($traits, fn (string $trait): bool => ! (isset($object->reflectionClass) && in_array($trait, $object->reflectionClass->getTraitNames(), true))),
+            function (ObjectDescription $object) use ($traits): bool {
+                foreach ($traits as $trait) {
+                    if (isset($object->reflectionClass) && in_array($trait, $object->reflectionClass->getTraitNames(), true)) {
+                        return false;
+                    }
+                }
+
+                return true;
+            },
             "not to use traits '".implode("', '", $traits)."'",
             FileLineFinder::where(fn (string $line): bool => str_contains($line, 'class')),
         );
@@ -596,7 +604,15 @@ final readonly class OppositeExpectation
 
         return Targeted::make(
             $original,
-            fn (ObjectDescription $object): bool => array_all($interfaces, fn (string $interface): bool => ! (isset($object->reflectionClass) && $object->reflectionClass->implementsInterface($interface))),
+            function (ObjectDescription $object) use ($interfaces): bool {
+                foreach ($interfaces as $interface) {
+                    if (isset($object->reflectionClass) && $object->reflectionClass->implementsInterface($interface)) {
+                        return false;
+                    }
+                }
+
+                return true;
+            },
             "not to implement '".implode("', '", $interfaces)."'",
             FileLineFinder::where(fn (string $line): bool => str_contains($line, 'class')),
         );
@@ -798,11 +814,13 @@ final readonly class OppositeExpectation
 
         $exporter = Exporter::default();
 
+        $toString = fn (mixed $argument): string => $exporter->shortenedExport($argument);
+
         throw new ExpectationFailedException(sprintf(
             'Expecting %s not %s %s.',
-            $exporter->shortenedExport($this->original->value),
+            $toString($this->original->value),
             strtolower((string) preg_replace('/(?<!\ )[A-Z]/', ' $0', $name)),
-            implode(' ', array_map(fn (mixed $argument): string => $exporter->export($argument), $arguments)),
+            implode(' ', array_map(fn (mixed $argument): string => $toString($argument), $arguments)),
         ));
     }
 
@@ -834,8 +852,8 @@ final readonly class OppositeExpectation
             $original,
             fn (ObjectDescription $object): bool => isset($object->reflectionClass) === false
                 || ! $object->reflectionClass->isEnum()
-                || ! new \ReflectionEnum($object->name)->isBacked() // @phpstan-ignore-line
+                || ! (new \ReflectionEnum($object->name))->isBacked() // @phpstan-ignore-line
-                || (string) new \ReflectionEnum($object->name)->getBackingType() !== $backingType, // @phpstan-ignore-line
+                || (string) (new \ReflectionEnum($object->name))->getBackingType() !== $backingType, // @phpstan-ignore-line
             'not to be '.$backingType.' backed enum',
             FileLineFinder::where(fn (string $line): bool => str_contains($line, 'class')),
         );

src/Factories/TestCaseFactory.php

@@ -197,7 +197,7 @@ final class TestCaseFactory
 
         if (
             $method->closure instanceof \Closure &&
-            new \ReflectionFunction($method->closure)->isStatic()
+            (new \ReflectionFunction($method->closure))->isStatic()
         ) {
 
             throw new TestClosureMustNotBeStatic($method);

src/Mixins/Expectation.php

@@ -936,7 +936,7 @@ final class Expectation
 
         if ($exception instanceof Closure) {
             $callback = $exception;
-            $parameters = new ReflectionFunction($exception)->getParameters();
+            $parameters = (new ReflectionFunction($exception))->getParameters();
 
             if (count($parameters) !== 1) {
                 throw new InvalidArgumentException('The given closure must have a single parameter type-hinted as the class string.');

src/PHPStan/HigherOrderExpectationTypeExtension.php

@@ -37,7 +37,7 @@ final readonly class HigherOrderExpectationTypeExtension implements ExpressionTy
 
         $varType = $scope->getType($expr->var);
 
-        if (! new ObjectType(HigherOrderExpectation::class)->isSuperTypeOf($varType)->yes()) {
+        if (! (new ObjectType(HigherOrderExpectation::class))->isSuperTypeOf($varType)->yes()) {
             return null;
         }
 

src/PendingCalls/UsesCall.php

@@ -53,7 +53,9 @@ final class UsesCall
         $this->targets = [$filename];
     }
 
-    #[\Deprecated(message: 'Use `pest()->printer()->compact()` instead.')]
+    /**
+     * @deprecated Use `pest()->printer()->compact()` instead.
+     */
     public function compact(): self
     {
         DefaultPrinter::compact(true);

src/Pest.php

@@ -6,7 +6,7 @@ namespace Pest;
 
 function version(): string
 {
-    return '5.0.0-rc.4';
+    return '4.6.2';
 }
 
 function testDirectory(string $file = ''): string

src/Plugins/Parallel.php

@@ -178,7 +178,13 @@ final class Parallel implements HandlesArguments
     {
         $arguments = new ArgvInput;
 
-        return array_any(self::UNSUPPORTED_ARGUMENTS, fn (string|array $unsupportedArgument): bool => $arguments->hasParameterOption($unsupportedArgument));
+        foreach (self::UNSUPPORTED_ARGUMENTS as $unsupportedArgument) {
+            if ($arguments->hasParameterOption($unsupportedArgument)) {
+                return true;
+            }
+        }
+
+        return false;
     }
 
     /**

src/Plugins/Parallel/Paratest/WrapperRunner.php

@@ -7,6 +7,7 @@ namespace Pest\Plugins\Parallel\Paratest;
 use const DIRECTORY_SEPARATOR;
 
 use NunoMaduro\Collision\Adapters\Phpunit\Support\ResultReflection;
+use ParaTest\Coverage\CoverageMerger;
 use ParaTest\JUnit\LogMerger;
 use ParaTest\JUnit\Writer;
 use ParaTest\Options;
@@ -24,17 +25,11 @@ use PHPUnit\TestRunner\TestResult\Facade as TestResultFacade;
 use PHPUnit\TestRunner\TestResult\TestResult;
 use PHPUnit\TextUI\Configuration\CodeCoverageFilterRegistry;
 use PHPUnit\Util\ExcludeList;
-use ReflectionProperty;
-use SebastianBergmann\CodeCoverage\Node\Builder;
-use SebastianBergmann\CodeCoverage\Serialization\Merger;
-use SebastianBergmann\CodeCoverage\StaticAnalysis\FileAnalyser;
-use SebastianBergmann\CodeCoverage\StaticAnalysis\ParsingSourceAnalyser;
 use SebastianBergmann\Timer\Timer;
 use SplFileInfo;
 use Symfony\Component\Console\Output\OutputInterface;
 use Symfony\Component\Process\PhpExecutableFinder;
 
-use function array_filter;
 use function array_merge;
 use function array_merge_recursive;
 use function array_shift;
@@ -453,33 +448,10 @@ final class WrapperRunner implements RunnerInterface
 
             return;
         }
-        $coverageFiles = [];
+        $coverageMerger = new CoverageMerger($coverageManager->codeCoverage());
-        foreach ($this->coverageFiles as $fileInfo) {
+        foreach ($this->coverageFiles as $coverageFile) {
-            $realPath = $fileInfo->getRealPath();
+            $coverageMerger->addCoverageFromFile($coverageFile);
-            if ($realPath !== false && $realPath !== '') {
-                $coverageFiles[] = $realPath;
-            }
         }
-        $serializedCoverage = (new Merger)->merge($coverageFiles);
-
-        $report = (new Builder(new FileAnalyser(new ParsingSourceAnalyser, false, false)))->build(
-            $serializedCoverage['codeCoverage'],
-            $serializedCoverage['testResults'],
-            $serializedCoverage['basePath'],
-        );
-        $codeCoverage = $coverageManager->codeCoverage();
-        $codeCoverage->excludeUncoveredFiles();
-
-        $mergedData = $serializedCoverage['codeCoverage'];
-        $basePath = $serializedCoverage['basePath'];
-        if ($basePath !== '') {
-            foreach ($mergedData->coveredFiles() as $relativePath) {
-                $mergedData->renameFile($relativePath, $basePath.DIRECTORY_SEPARATOR.$relativePath);
-            }
-        }
-        $codeCoverage->setData($mergedData);
-        $codeCoverage->setTests($serializedCoverage['testResults']);
-        (new ReflectionProperty(\SebastianBergmann\CodeCoverage\CodeCoverage::class, 'cachedReport'))->setValue($codeCoverage, $report);
 
         $coverageManager->generateReports(
             $this->printer->printer,

src/Plugins/Shard.php

@@ -187,11 +187,11 @@ final class Shard implements AddsOutput, HandlesArguments, Terminable
      */
     private function allTests(array $arguments): array
     {
-        $output = new Process([
+        $output = (new Process([
             'php',
             ...$this->removeParallelArguments($arguments),
             '--list-tests',
-        ])->setTimeout(120)->mustRun()->getOutput();
+        ]))->setTimeout(120)->mustRun()->getOutput();
 
         preg_match_all('/ - (?:P\\\\)?(Tests\\\\[^:]+)::/', $output, $matches);
 

src/Support/Coverage.php

@@ -8,7 +8,6 @@ use Pest\Exceptions\ShouldNotHappen;
 use SebastianBergmann\CodeCoverage\CodeCoverage;
 use SebastianBergmann\CodeCoverage\Node\Directory;
 use SebastianBergmann\CodeCoverage\Node\File;
-use SebastianBergmann\CodeCoverage\Report\Facade;
 use SebastianBergmann\Environment\Runtime;
 use Symfony\Component\Console\Output\OutputInterface;
 
@@ -93,18 +92,10 @@ final class Coverage
         $codeCoverage = require $reportPath;
         unlink($reportPath);
 
-        // @phpstan-ignore-next-line
+        $totalCoverage = $codeCoverage->getReport()->percentageOfExecutedLines();
-        if (is_array($codeCoverage)) {
-            $facade = Facade::fromSerializedData($codeCoverage);
 
-            /** @var Directory<File|Directory> $report */
+        /** @var Directory<File|Directory> $report */
-            $report = (fn (): Directory => $this->report)->call($facade);
+        $report = $codeCoverage->getReport();
-        } else {
-            /** @var Directory<File|Directory> $report */
-            $report = $codeCoverage->getReport();
-        }
-
-        $totalCoverage = $report->percentageOfExecutedLines();
 
         foreach ($report->getIterator() as $file) {
             if (! $file instanceof File) {

src/Support/Exporter.php

@@ -86,17 +86,4 @@ final readonly class Exporter
 
         return (string) preg_replace(array_keys($map), array_values($map), $this->exporter->shortenedExport($value));
     }
-
-    /**
-     * Exports a value into a full single-line string without truncation.
-     */
-    public function export(mixed $value): string
-    {
-        $map = [
-            '#\\\n\s*#' => '',
-            '# Object \(\.{3}\)#' => '',
-        ];
-
-        return (string) preg_replace(array_keys($map), array_values($map), $this->exporter->export($value));
-    }
 }

src/Support/HigherOrderMessage.php

@@ -50,7 +50,7 @@ final class HigherOrderMessage
         }
 
         if ($this->hasHigherOrderCallable()) {
-            return new HigherOrderCallables($target)->{$this->name}(...$this->arguments);
+            return (new HigherOrderCallables($target))->{$this->name}(...$this->arguments);
         }
 
         try {

src/Support/HigherOrderMessageCollection.php

@@ -31,7 +31,7 @@ final class HigherOrderMessageCollection
      */
     public function addWhen(callable $condition, string $filename, int $line, string $name, ?array $arguments): void
     {
-        $this->messages[] = new HigherOrderMessage($filename, $line, $name, $arguments)->when($condition);
+        $this->messages[] = (new HigherOrderMessage($filename, $line, $name, $arguments))->when($condition);
     }
 
     /**

src/Support/HigherOrderTapProxy.php

@@ -38,7 +38,7 @@ final class HigherOrderTapProxy
             return $this->target->{$property};
         }
 
-        $className = new ReflectionClass($this->target)->getName();
+        $className = (new ReflectionClass($this->target))->getName();
 
         if (str_starts_with($className, 'P\\')) {
             $className = substr($className, 2);
@@ -60,7 +60,7 @@ final class HigherOrderTapProxy
         $filename = Backtrace::file();
         $line = Backtrace::line();
 
-        return new HigherOrderMessage($filename, $line, $methodName, $arguments)
+        return (new HigherOrderMessage($filename, $line, $methodName, $arguments))
             ->call($this->target);
     }
 }

src/Support/Reflection.php

@@ -181,7 +181,7 @@ final class Reflection
      */
     public static function getFunctionArguments(Closure $function): array
     {
-        $parameters = new ReflectionFunction($function)->getParameters();
+        $parameters = (new ReflectionFunction($function))->getParameters();
         $arguments = [];
 
         foreach ($parameters as $parameter) {
@@ -207,7 +207,7 @@ final class Reflection
 
     public static function getFunctionVariable(Closure $function, string $key): mixed
     {
-        return new ReflectionFunction($function)->getStaticVariables()[$key] ?? null;
+        return (new ReflectionFunction($function))->getStaticVariables()[$key] ?? null;
     }
 
     /**

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

@@ -1,5 +1,5 @@
 
-  Pest Testing Framework 5.0.0-rc.4.  
+  Pest Testing Framework 4.6.2.  
 
   USAGE: pest <file> [options]  
 
@@ -45,7 +45,6 @@
   --filter [pattern] ............................... Filter which tests to run  
   --exclude-filter [pattern] .. Exclude tests for the specified filter pattern  
   --test-suffix [suffixes]  Only search for test in files with specified suffix(es). Default: Test.php,.phpt  
-  --test-files-file [file]  Only run test files listed in file (one file by line)  
 
   EXECUTION OPTIONS:
   --parallel ........................................... Run tests in parallel  
@@ -126,12 +125,12 @@
   LOGGING OPTIONS:
   --log-junit [file] .......... Write test results in JUnit XML format to file  
   --log-otr [file]  Write test results in Open Test Reporting XML format to file  
+  --include-git-information  Include Git information in Open Test Reporting XML logfile  
   --log-teamcity [file] ........ Write test results in TeamCity format to file  
   --testdox-html [file] .. Write test results in TestDox format (HTML) to file  
   --testdox-text [file]  Write test results in TestDox format (plain text) to file  
   --log-events-text [file] ............... Stream events as plain text to file  
   --log-events-verbose-text [file]  Stream events as plain text with extended information to file  
-  --include-git-information ..... Include Git information in supported formats  
   --no-logging ....... Ignore logging configured in the XML configuration file  
 
   CODE COVERAGE OPTIONS:

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

@@ -1,3 +1,3 @@
 
-  Pest Testing Framework 5.0.0-rc.4.  
+  Pest Testing Framework 4.6.2.  
 

tests/.snapshots/success.txt

@@ -1697,8 +1697,6 @@
    PASS  Tests\Unit\Expectations\OppositeExpectation
   ✓ it throw expectation failed exception with string argument
   ✓ it throw expectation failed exception with array argument
-  ✓ it does not truncate long string arguments in error message
-  ✓ it does not truncate custom error message when using not()
 
    PASS  Tests\Unit\Overrides\ThrowableBuilder
   ✓ collision editor can be added to the stack trace
@@ -1903,4 +1901,4 @@
   ✓ pass with dataset with ('my-datas-set-value')
   ✓ within describe → pass with dataset with ('my-datas-set-value')
 
-  Tests:    2 deprecated, 4 warnings, 5 incomplete, 2 notices, 40 todos, 35 skipped, 1296 passed (2976 assertions)
+  Tests:    2 deprecated, 4 warnings, 5 incomplete, 2 notices, 40 todos, 35 skipped, 1294 passed (2971 assertions)

tests/Unit/Expectations/OppositeExpectation.php

@@ -14,17 +14,3 @@ it('throw expectation failed exception with array argument', function (): void {
 
     $expectation->throwExpectationFailedException('toBe', ['bar']);
 })->throws(ExpectationFailedException::class, "Expecting 'foo' not to be 'bar'.");
-
-it('does not truncate long string arguments in error message', function (): void {
-    $expectation = new OppositeExpectation(expect('foo'));
-
-    $longMessage = 'Very long error message. Very long error message. Very long error message.';
-
-    $expectation->throwExpectationFailedException('toBe', [$longMessage]);
-})->throws(ExpectationFailedException::class, 'Very long error message. Very long error message. Very long error message.');
-
-it('does not truncate custom error message when using not()', function (): void {
-    $longMessage = 'This is a very detailed custom error message that should not be truncated in the output.';
-
-    expect(true)->not()->toBeTrue($longMessage);
-})->throws(ExpectationFailedException::class, 'This is a very detailed custom error message that should not be truncated in the output.');

tests/Visual/Parallel.php

@@ -23,13 +23,13 @@ test('parallel', function () use ($run) {
         $file = file_get_contents(__FILE__);
         $file = preg_replace(
             '/\$expected = \'.*?\';/',
-            "\$expected = '2 deprecated, 4 warnings, 5 incomplete, 3 notices, 40 todos, 27 skipped, 1280 passed (2925 assertions)';",
+            "\$expected = '2 deprecated, 4 warnings, 5 incomplete, 3 notices, 40 todos, 27 skipped, 1278 passed (2920 assertions)';",
             $file,
         );
         file_put_contents(__FILE__, $file);
     }
 
-    $expected = '2 deprecated, 4 warnings, 5 incomplete, 3 notices, 40 todos, 27 skipped, 1280 passed (2925 assertions)';
+    $expected = '2 deprecated, 4 warnings, 5 incomplete, 3 notices, 40 todos, 27 skipped, 1278 passed (2920 assertions)';
 
     expect($output)
         ->toContain("Tests:    {$expected}")