Upgrade to php 84 (#115)

* Since mac runners are missing composer we want to setup php before composer validate

* Fixes with paratest 7.3 and phpunit not being compatible from version 10.5.x, later version of paratest are not php8.1 compatible

* Upgraded to be compatible php 8.4 and phpunit 12

* Updated README to include php 8.4

Author: krchiper

.github/workflows/build.yml

@@ -20,6 +20,7 @@ jobs:
           - "8.1"
           - "8.2"
           - "8.3"
+          - "8.4"
         os:
           - ubuntu-latest
           - windows-latest
@@ -31,9 +32,6 @@ jobs:
       - name: Checkout
         uses: actions/checkout@v4
 
-      - name: Validate composer.json and composer.lock
-        run: composer validate
-
       - name: Set up PHP ${{ matrix.php-version }}
         uses: shivammathur/setup-php@v2
         with:
@@ -42,6 +40,9 @@ jobs:
           coverage: xdebug
           ini-values: error_reporting=E_ALL
 
+      - name: Validate composer.json and composer.lock
+        run: composer validate
+
       - name: Install dependencies
         run: composer update
           --prefer-dist

README.md

@@ -35,7 +35,7 @@ This adapter only generates JSON files containing information about tests. See [
 ## Installation && Usage
 **Note:** this adapter supports Allure 2.x.x only.
 
-Supported PHP versions: 8.1-8.3.
+Supported PHP versions: 8.1-8.4.
 
 In order to use this adapter you need to add a new dependency to your **composer.json** file:
 ```

composer.json

@@ -29,18 +29,27 @@
     "source": "https://github.com/allure-framework/allure-phpunit"
   },
   "require": {
-    "php": "~8.1.0 || ~8.2.0 || ~8.3.0",
-    "allure-framework/allure-php-commons": "^2",
-    "phpunit/phpunit": "^10"
+    "php": "~8.1.0 || ~8.2.0 || ~8.3.0 || ~8.4.0",
+    "allure-framework/allure-php-commons": "^2.0",
+    "phpunit/phpunit": "^10 || ^11 || ^12"
   },
   "require-dev": {
-    "brianium/paratest": "^7",
-    "psalm/plugin-phpunit": "^0.18.4",
+    "phpunit/phpunit": "~10.4.0 || ^11 || ^12",
+    "brianium/paratest": "^7.3",
+    "psalm/plugin-phpunit": "^0.19.3",
     "squizlabs/php_codesniffer": "^3.7.2",
-    "vimeo/psalm": "^5.15"
+    "vimeo/psalm": "^6.9"
   },
   "conflict": {
     "amphp/byte-stream": "<1.5.1",
+    "amphp/dns": "<2.2.0",
+    "amphp/socket": "<2.3.1",
+    "amphp/cache": "<2.0.1",
+    "amphp/process": "<2.0.3",
+    "amphp/parser": "<1.1.1",
+    "daverandom/libdns": "<2.1.0",
+    "spatie/array-to-xml": "<3.3.0",
+    "ramsey/uuid": "<4.3.0",
     "brianium/paratest": "<7.0.3"
   },
   "autoload": {

phpunit.10.0.xml

@@ -1,23 +1,23 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <phpunit
-        xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
-        xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/10.0/phpunit.xsd"
-        colors="true"
-        displayDetailsOnIncompleteTests="true"
-        displayDetailsOnSkippedTests="true"
-        displayDetailsOnTestsThatTriggerDeprecations="true"
-        displayDetailsOnTestsThatTriggerWarnings="true"
-        displayDetailsOnTestsThatTriggerNotices="true"
-        displayDetailsOnTestsThatTriggerErrors="true"
-        defaultTestSuite="unit">
+    xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+    xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/12.0/phpunit.xsd"
+    colors="true"
+    displayDetailsOnIncompleteTests="true"
+    displayDetailsOnSkippedTests="true"
+    displayDetailsOnTestsThatTriggerDeprecations="true"
+    displayDetailsOnTestsThatTriggerWarnings="true"
+    displayDetailsOnTestsThatTriggerNotices="true"
+    displayDetailsOnTestsThatTriggerErrors="true"
+    defaultTestSuite="unit">
     <testsuites>
         <testsuite name="unit">
             <directory>test/unit/</directory>
         </testsuite>
     </testsuites>
-    <coverage>
+    <source>
         <include>
             <directory suffix=".php">src/</directory>
         </include>
-    </coverage>
+    </source>
 </phpunit>

src/AllureAdapter.php

@@ -48,21 +48,25 @@ public static function reset(): void
         self::$instance = null;
     }
 
+    #[\Override]
     public function resetLastException(): void
     {
         $this->lastException = null;
     }
 
+    #[\Override]
     public function setLastException(Throwable $e): void
     {
         $this->lastException = $e;
     }
 
+    #[\Override]
     public function getLastException(): ?Throwable
     {
         return $this->lastException;
     }
 
+    #[\Override]
     public function registerStart(ContainerResult $containerResult, TestResult $testResult, TestInfo $info): string
     {
         $this->lastStarts[$info->getTest()] = new TestStartInfo(
@@ -73,6 +77,7 @@ public function registerStart(ContainerResult $containerResult, TestResult $test
         return $testResult->getUuid();
     }
 
+    #[\Override]
     public function getContainerId(TestInfo $info): string
     {
         $startInfo = $this->lastStarts[$info->getTest()] ?? null;
@@ -81,6 +86,7 @@ public function getContainerId(TestInfo $info): string
             ?? throw new LogicException("Container not registered: {$info->getTest()}");
     }
 
+    #[\Override]
     public function getTestId(TestInfo $info): string
     {
         $startInfo = $this->lastStarts[$info->getTest()] ?? null;
@@ -89,6 +95,7 @@ public function getTestId(TestInfo $info): string
             ?? throw new LogicException("Test not registered: {$info->getTest()}");
     }
 
+    #[\Override]
     public function registerRun(TestResult $testResult, TestInfo $info): TestRunInfo
     {
         $testCaseId = $this->buildTestCaseId($testResult, $info);

src/AllureExtension.php

@@ -90,6 +90,7 @@ private function loadConfigData(?string $configFile): array
         return [];
     }
 
+    #[\Override]
     public function bootstrap(Configuration $configuration, Facade $facade, ParameterCollection $parameters): void
     {
         $configSource = $parameters->has('config')

src/Event/TestConsideredRiskySubscriber.php

@@ -17,6 +17,7 @@ public function __construct(
     ) {
     }
 
+    #[\Override]
     public function notify(ConsideredRisky $event): void
     {
         $test = $event->test();

src/Event/TestErroredSubscriber.php

@@ -17,6 +17,7 @@ public function __construct(
     ) {
     }
 
+    #[\Override]
     public function notify(Errored $event): void
     {
         $test = $event->test();

src/Event/TestFailedSubscriber.php

@@ -17,6 +17,7 @@ public function __construct(
     ) {
     }
 
+    #[\Override]
     public function notify(Failed $event): void
     {
         $test = $event->test();

src/Event/TestFinishedSubscriber.php

@@ -16,6 +16,7 @@ public function __construct(
     ) {
     }
 
+    #[\Override]
     public function notify(Finished $event): void
     {
         $test = $event->test();