release: v2.34.1

fix: updates Only plugin to check for CI environment

bin/worker.php

@@ -81,6 +81,7 @@ $bootPest = (static function (): void {
         $getopt['teamcity-file'] ?? null,
         $getopt['testdox-file'] ?? null,
         isset($getopt['testdox-color']),
+        $getopt['testdox-columns'] ?? null,
     );
 
     while (true) {

composer.json

@@ -22,11 +22,11 @@
         "nunomaduro/collision": "^7.10.0|^8.1.0",
         "nunomaduro/termwind": "^1.15.1|^2.0.0",
         "pestphp/pest-plugin": "^2.1.1",
-        "pestphp/pest-plugin-arch": "^2.6.1",
+        "pestphp/pest-plugin-arch": "^2.7.0",
-        "phpunit/phpunit": "^10.5.9"
+        "phpunit/phpunit": "^10.5.11"
     },
     "conflict": {
-        "phpunit/phpunit": ">10.5.9",
+        "phpunit/phpunit": ">10.5.11",
         "sebastian/exporter": "<5.1.0",
         "webmozart/assert": "<1.11.0"
     },
@@ -53,7 +53,7 @@
     "require-dev": {
         "pestphp/pest-dev-tools": "^2.16.0",
         "pestphp/pest-plugin-type-coverage": "^2.8.0",
-        "symfony/process": "^6.4.0|^7.0.2"
+        "symfony/process": "^6.4.0|^7.0.4"
     },
     "minimum-stability": "dev",
     "prefer-stable": true,

resources/views/components/badge.php

@@ -10,7 +10,7 @@
 ?>
 
 <div class="my-1">
-    <span class="ml-2 px-1 bg-<?php echo $bgBadgeColor ?>-600 font-bold"><?php echo htmlspecialchars($bgBadgeText) ?></span>
+    <span class="ml-2 px-1 bg-<?php echo $bgBadgeColor ?> font-bold"><?php echo htmlspecialchars($bgBadgeText) ?></span>
     <span class="ml-1">
         <?php echo htmlspecialchars($content) ?>
     </span>

src/Bootstrappers/BootOverrides.php

@@ -15,17 +15,17 @@ final class BootOverrides implements Bootstrapper
     /**
      * The list of files to be overridden.
      *
-     * @var array<int, string>
+     * @var array<string, string>
      */
-    private const FILES = [
+    public const FILES = [
-        'Runner/Filter/NameFilterIterator.php',
+        'c7b9c8a96006dea314204a8f09a8764e51ce0b9b79aadd58da52e8c328db4870' => 'Runner/Filter/NameFilterIterator.php',
-        'Runner/ResultCache/DefaultResultCache.php',
+        '52b2574e96269aca1bb2d41bbf418c3bcf23dd21d14c66f90789025c309e39df' => 'Runner/ResultCache/DefaultResultCache.php',
-        'Runner/TestSuiteLoader.php',
+        'bc8718c89264f65800beabc23e51c6d3bcff87dfc764a12179ef5dbfde272c8b' => 'Runner/TestSuiteLoader.php',
-        'TextUI/Command/WarmCodeCoverageCacheCommand.php',
+        'f41e48d6cb546772a7de4f8e66b6b7ce894a5318d063eb52e354d206e96c701c' => 'TextUI/Command/Commands/WarmCodeCoverageCacheCommand.php',
-        'TextUI/Output/Default/ProgressPrinter/TestSkippedSubscriber.php',
+        'cb7519f2d82893640b694492cf7ec9528da80773cc1d259634181b5d393528b5' => 'TextUI/Output/Default/ProgressPrinter/Subscriber/TestSkippedSubscriber.php',
-        'TextUI/TestSuiteFilterProcessor.php',
+        '6db25ee539e9b12b1fb4e044a0a93410e015bc983ecdd3909cd394fe44ae8c95' => 'TextUI/TestSuiteFilterProcessor.php',
-        'Event/Value/ThrowableBuilder.php',
+        'ef64a657ed9c0067791483784944107827bf227c7e3200f212b6751876b99e25' => 'Event/Value/ThrowableBuilder.php',
-        'Logging/JUnit/JunitXmlLogger.php',
+        'c78f96e34b98ed01dd8106539d59b8aa8d67f733274118b827c01c5c4111c033' => 'Logging/JUnit/JunitXmlLogger.php',
     ];
 
     /**

src/Kernel.php

@@ -139,6 +139,10 @@ final class Kernel
         $this->terminate();
 
         if (is_array($error = error_get_last())) {
+            if (! in_array($error['type'], [E_ERROR, E_CORE_ERROR], true)) {
+                return;
+            }
+
             $message = $error['message'];
             $file = $error['file'];
             $line = $error['line'];

src/Pest.php

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

src/Plugins/Only.php

@@ -40,6 +40,10 @@ final class Only implements Terminable
      */
     public static function enable(TestCall $testCall): void
     {
+        if (Environment::name() == Environment::CI) {
+            return;
+        }
+
         $testCall->group('__pest_only');
 
         $lockFile = self::TEMPORARY_FOLDER.DIRECTORY_SEPARATOR.'only.lock';

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

@@ -1,5 +1,5 @@
 
-  Pest Testing Framework 2.32.2.  
+  Pest Testing Framework 2.34.1.  
 
   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.32.2.  
+  Pest Testing Framework 2.34.1.  
 

tests/.snapshots/success.txt

@@ -1167,6 +1167,16 @@
    PASS  Tests\Hooks\BeforeEachTest
   ✓ global beforeEach execution order
 
+   PASS  Tests\Overrides\VersionsTest
+  ✓ versions with dataset "Runner/Filter/NameFilterIterator.php"
+  ✓ versions with dataset "Runner/ResultCache/DefaultResultCache.php"
+  ✓ versions with dataset "Runner/TestSuiteLoader.php"
+  ✓ versions with dataset "TextUI/Command/Commands/WarmCodeCoverageCacheCommand.php"
+  ✓ versions with dataset "TextUI/Output/Default/ProgressPrinter/Subscriber/TestSkippedSubscriber.php"
+  ✓ versions with dataset "TextUI/TestSuiteFilterProcessor.php"
+  ✓ versions with dataset "Event/Value/ThrowableBuilder.php"
+  ✓ versions with dataset "Logging/JUnit/JunitXmlLogger.php"
+
    PASS  Tests\PHPUnit\CustomAffixes\InvalidTestName
   ✓ it runs file names like @#$%^&()-_=+.php
 
@@ -1379,6 +1389,10 @@
    PASS  Tests\Visual\Help
   ✓ visual snapshot of help command output
 
+   PASS  Tests\Visual\JUnit
+  ✓ junit output
+  ✓ junit with parallel
+
    PASS  Tests\Visual\Parallel
   ✓ parallel
   ✓ a parallel test can extend another test with same name
@@ -1404,4 +1418,4 @@
    WARN  Tests\Visual\Version
   - visual snapshot of help command output
 
-  Tests:    2 deprecated, 4 warnings, 5 incomplete, 2 notices, 13 todos, 20 skipped, 999 passed (2359 assertions)
+  Tests:    2 deprecated, 4 warnings, 5 incomplete, 2 notices, 13 todos, 20 skipped, 1009 passed (2395 assertions)

tests/Overrides/VersionsTest.php

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

tests/Visual/JUnit.php

@@ -0,0 +1,79 @@
+<?php
+
+use Symfony\Component\Process\Process;
+
+$run = function () {
+    $junitLogFile = tempnam(sys_get_temp_dir(), 'junit');
+
+    $process = new Process(
+        array_merge(['php', 'bin/pest', '--log-junit', $junitLogFile], func_get_args()),
+        dirname(__DIR__, 2),
+        ['COLLISION_PRINTER' => 'DefaultPrinter', 'COLLISION_IGNORE_DURATION' => 'true'],
+    );
+
+    $process->run();
+
+    $rawXmlContent = file_get_contents($junitLogFile);
+    unlink($junitLogFile);
+
+    // convert xml to array
+    try {
+        $xml = new SimpleXMLElement(preg_replace("/(<\/?)(\w+):([^>]*>)/", '$1$2$3', $rawXmlContent));
+
+        return json_decode(json_encode((array) $xml), true);
+    } catch (Exception $exception) {
+        throw new XmlParseException($exception->getMessage(), $exception->getCode(), $exception->getPrevious());
+    }
+};
+
+$normalizedPath = function (string $path) {
+    return str_replace('/', DIRECTORY_SEPARATOR, $path);
+};
+
+test('junit output', function () use ($normalizedPath, $run) {
+    $result = $run('tests/.tests/SuccessOnly.php');
+
+    expect($result['testsuite']['@attributes'])
+        ->name->toBe('Tests\tests\SuccessOnly')
+        ->file->toBe($normalizedPath('tests/.tests/SuccessOnly.php'))
+        ->tests->toBe('2')
+        ->assertions->toBe('2')
+        ->errors->toBe('0')
+        ->failures->toBe('0')
+        ->skipped->toBe('0');
+
+    expect($result['testsuite']['testcase'])
+        ->toHaveCount(2);
+
+    expect($result['testsuite']['testcase'][0]['@attributes'])
+        ->name->toBe('it can pass with comparison')
+        ->file->toBe($normalizedPath('tests/.tests/SuccessOnly.php::it can pass with comparison'))
+        ->class->toBe('Tests\tests\SuccessOnly')
+        ->classname->toBe('Tests.tests.SuccessOnly')
+        ->assertions->toBe('1')
+        ->time->toStartWith('0.0');
+});
+
+test('junit with parallel', function () use ($normalizedPath, $run) {
+    $result = $run('tests/.tests/SuccessOnly.php', '--parallel', '--processes=1', '--filter', 'can pass with comparison');
+
+    expect($result['testsuite']['@attributes'])
+        ->name->toBe('Tests\tests\SuccessOnly')
+        ->file->toBe($normalizedPath('tests/.tests/SuccessOnly.php'))
+        ->tests->toBe('1')
+        ->assertions->toBe('1')
+        ->errors->toBe('0')
+        ->failures->toBe('0')
+        ->skipped->toBe('0');
+
+    expect($result['testsuite']['testcase'])
+        ->toHaveCount(1);
+
+    expect($result['testsuite']['testcase']['@attributes'])
+        ->name->toBe('it can pass with comparison')
+        ->file->toBe($normalizedPath('tests/.tests/SuccessOnly.php::it can pass with comparison'))
+        ->class->toBe('Tests\tests\SuccessOnly')
+        ->classname->toBe('Tests.tests.SuccessOnly')
+        ->assertions->toBe('1')
+        ->time->toStartWith('0.0');
+});

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