Merge pull request #13 from robiningelbrecht/ISSUE-12

ISSUE #12: Display random seed

Author: robiningelbrecht

Makefile

@@ -0,0 +1,38 @@
+compose=docker compose
+
+dc:
+	@${compose} -f docker-compose.yml $(cmd)
+
+dcr:
+	@make dc cmd="run --rm php-cli $(cmd)"
+
+stop:
+	@make dc cmd="stop"
+
+up:
+	@make dc cmd="up -d"
+
+build-containers:
+	@make dc cmd="up -d --build"
+
+down:
+	@make dc cmd="down"
+
+composer:
+	@make dcr cmd="composer $(arg)"
+
+# Code quality tools.
+phpunit:
+	@make dcr cmd="vendor/bin/phpunit -d --enable-pretty-print -d --compact $(arg)"
+
+phpunit-with-coverage-report:
+	@make phpunit arg="--coverage-clover=clover.xml -d --min-coverage=min-coverage-rules.php"
+
+phpstan:
+	@make dcr cmd="vendor/bin/phpstan --memory-limit=1G $(arg)"
+
+csfix:
+	@make dcr cmd="vendor/bin/php-cs-fixer fix --config=.php-cs-fixer.dist.php"
+
+delete-snapshots:
+	find . -name __snapshots__ -type d -prune -exec rm -rf {} \;

composer.json

@@ -36,9 +36,5 @@
   },
   "config": {
     "sort-packages": true
-  },
-  "scripts": {
-    "lint:fix": " ./vendor/bin/php-cs-fixer fix --config=.php-cs-fixer.dist.php",
-    "phpunit:test": "vendor/bin/phpunit --configuration=tests/phpunit.test.xml"
   }
 }

src/PhpUnitExtension.php

@@ -6,6 +6,7 @@
 use PHPUnit\Runner\Extension\Extension;
 use PHPUnit\Runner\Extension\Facade;
 use PHPUnit\Runner\Extension\ParameterCollection;
+use PHPUnit\Runner\TestSuiteSorter;
 use PHPUnit\TextUI\Configuration\Configuration as PHPUnitConfiguration;
 use RobinIngelbrecht\PHPUnitPrettyPrint\Subscriber\Application\ApplicationFinishedSubscriber;
 use RobinIngelbrecht\PHPUnitPrettyPrint\Subscriber\Application\ApplicationStartedSubscriber;
@@ -18,6 +19,7 @@ public function bootstrap(PHPUnitConfiguration $configuration, Facade $facade, P
             return;
         }
 
+        $_SERVER['PHPUNIT_RANDOM_ORDER_SEED'] = TestSuiteSorter::ORDER_RANDOMIZED === $configuration->executionOrder() ? $configuration->randomOrderSeed() : null;
         $configuration = Configuration::fromParameterCollection($parameters);
 
         $facade->replaceOutput();

src/Subscriber/Application/ApplicationStartedSubscriber.php

@@ -16,5 +16,11 @@ public function notify(Started $event): void
             '<div>&nbsp;&nbsp;Runtime: %s</div>',
             $event->runtime()->asString()
         ));
+        if (!empty($_SERVER['PHPUNIT_RANDOM_ORDER_SEED'])) {
+            render(sprintf(
+                '<div>&nbsp;&nbsp;Random Seed: %s</div>',
+                $_SERVER['PHPUNIT_RANDOM_ORDER_SEED']
+            ));
+        }
     }
 }

tests/OutputTest.php

@@ -18,7 +18,9 @@ public function testPrintWithoutConfig(): void
         ];
 
         exec(implode(' ', $command), $out);
-        $this->assertStringContainsString('Tests:    3 failed, 1 risky, 1 incomplete, 1 skipped, 3 passed (7 assertions)', implode(PHP_EOL, $out));
+        $output = implode(PHP_EOL, $out);
+        $this->assertStringContainsString('Tests:    3 failed, 1 risky, 1 incomplete, 1 skipped, 3 passed (7 assertions)', $output);
+        $this->assertStringNotContainsString('Random Seed:', $output);
     }
 
     public function testWithProfiling(): void
@@ -135,6 +137,20 @@ public function testPrintWithQuoteAtRuntime(): void
         }
     }
 
+    public function testPrintWithRandomSeed(): void
+    {
+        $command = [
+            'vendor/bin/phpunit',
+            '--configuration=tests/phpunit.test.xml',
+            '--order-by=random',
+        ];
+
+        exec(implode(' ', $command), $out);
+        $output = implode(PHP_EOL, $out);
+        $this->assertStringContainsString('Tests:    3 failed, 1 risky, 1 incomplete, 1 skipped, 3 passed (7 assertions)', $output);
+        $this->assertStringContainsString('Random Seed:', $output);
+    }
+
     public function testItShouldBeEnabled(): void
     {
         $command = [