Fix incompatibility with PHP 8.4 and 8.5 in dev

.github/workflows/build.yml

@@ -27,7 +27,7 @@ jobs:
       - name: Set up PHP
         uses: shivammathur/setup-php@v2
         with:
-          php-version: "8.3"
+          php-version: "8.5"
 
       - name: Validate composer.json
         run: composer validate
@@ -52,6 +52,8 @@ jobs:
           - "8.1"
           - "8.2"
           - "8.3"
+          - "8.4"
+          - "8.5"
         os:
           - ubuntu
           - windows

composer.json

@@ -28,8 +28,6 @@
     },
     "require-dev": {
         "psalm/plugin-phpunit": "^0.19.0 || ^0.20.1",
-        "remorhaz/php-json-data": "^0.5.3",
-        "remorhaz/php-json-path": "^0.7.7",
         "squizlabs/php_codesniffer": "^4.0.1",
         "vimeo/psalm": "^5.26.1 || ^6.16.1"
     },

src/AllureCodeception.php

@@ -68,6 +68,7 @@ final class AllureCodeception extends Extension
      * @throws ConfigurationException
      * phpcs:disable PSR2.Methods.MethodDeclaration.Underscore
      */
+    #[\Override]
     public function _initialize(): void
     {
         parent::_initialize();

src/Internal/ArgumentAsString.php

@@ -88,12 +88,19 @@ private function prepareObject(object $argument): string
         return trim($argument::class, "\\");
     }
 
+    #[\Override]
     public function __toString(): string
     {
-        return json_encode(
+        $json = json_encode(
             $this->prepareArgument($this->argument),
             JSON_UNESCAPED_UNICODE | JSON_UNESCAPED_SLASHES,
         );
+
+        if ($json === false) {
+            return "";
+        }
+
+        return $json;
     }
 
     private function webDriverByAsString(object $selector): string

src/Internal/CeptInfoBuilder.php

@@ -13,6 +13,7 @@ public function __construct(
     ) {
     }
 
+    #[\Override]
     public function build(?string $host, ?string $thread): TestInfo
     {
         return new TestInfo(

src/Internal/CeptProvider.php

@@ -63,25 +63,29 @@ public static function createForChain(Cept $test, LinkTemplateCollectionInterfac
         return [new self($test, $linkTemplates)];
     }
 
+    #[\Override]
     public function getLinks(): array
     {
         $this->loadLegacyModels();
 
         return $this->legacyLinks;
     }
 
+    #[\Override]
     public function getLabels(): array
     {
         $this->loadLegacyModels();
 
         return $this->legacyLabels;
     }
 
+    #[\Override]
     public function getParameters(): array
     {
         return [];
     }
 
+    #[\Override]
     public function getDisplayName(): ?string
     {
         $this->loadLegacyModels();
@@ -98,18 +102,21 @@ public function getDisplayName(): ?string
             : null;
     }
 
+    #[\Override]
     public function getFullName(): ?string
     {
         return $this->test->getSignature();
     }
 
+    #[\Override]
     public function getDescription(): ?string
     {
         $this->loadLegacyModels();
 
         return $this->legacyDescription;
     }
 
+    #[\Override]
     public function getDescriptionHtml(): ?string
     {
         return null;

src/Internal/CestInfoBuilder.php

@@ -16,6 +16,7 @@ public function __construct(
     ) {
     }
 
+    #[\Override]
     public function build(?string $host, ?string $thread): TestInfo
     {
         return new TestInfo(

src/Internal/CestProvider.php

@@ -49,16 +49,19 @@ classOrObject: $test->getTestInstance(),
         ];
     }
 
+    #[\Override]
     public function getLinks(): array
     {
         return [];
     }
 
+    #[\Override]
     public function getLabels(): array
     {
         return [];
     }
 
+    #[\Override]
     public function getParameters(): array
     {
         /** @var mixed $currentExample */
@@ -80,6 +83,7 @@ public function getParameters(): array
         );
     }
 
+    #[\Override]
     public function getDisplayName(): ?string
     {
         /** @psalm-var mixed $displayName */
@@ -90,11 +94,13 @@ public function getDisplayName(): ?string
             : null;
     }
 
+    #[\Override]
     public function getDescription(): ?string
     {
         return null;
     }
 
+    #[\Override]
     public function getDescriptionHtml(): ?string
     {
         return null;
@@ -104,6 +110,7 @@ public function getDescriptionHtml(): ?string
      * @psalm-suppress MixedOperand
      * @psalm-suppress MixedArgument
      */
+    #[\Override]
     public function getFullName(): ?string
     {
         return $this->test->getTestInstance()::class . "::" . $this->test->getTestMethod();

src/Internal/DefaultThreadDetector.php

@@ -12,6 +12,7 @@ final class DefaultThreadDetector implements ThreadDetectorInterface
 {
     private string|false|null $host = null;
 
+    #[\Override]
     public function getHost(): ?string
     {
         $this->host ??= gethostname();
@@ -21,6 +22,7 @@ public function getHost(): ?string
             : $this->host;
     }
 
+    #[\Override]
     public function getThread(): ?string
     {
         return null;

src/Internal/GherkinInfoBuilder.php

@@ -15,6 +15,7 @@ public function __construct(
     ) {
     }
 
+    #[\Override]
     public function build(?string $host, ?string $thread): TestInfo
     {
         return new TestInfo(

src/Internal/GherkinProvider.php

@@ -26,11 +26,13 @@ public static function createForChain(Gherkin $test): array
         return [new self($test)];
     }
 
+    #[\Override]
     public function getLinks(): array
     {
         return [];
     }
 
+    #[\Override]
     public function getLabels(): array
     {
         return array_map(
@@ -42,26 +44,31 @@ public function getLabels(): array
         );
     }
 
+    #[\Override]
     public function getParameters(): array
     {
         return [];
     }
 
+    #[\Override]
     public function getDisplayName(): ?string
     {
         return $this->test->toString();
     }
 
+    #[\Override]
     public function getDescription(): ?string
     {
         return null;
     }
 
+    #[\Override]
     public function getDescriptionHtml(): ?string
     {
         return null;
     }
 
+    #[\Override]
     public function getFullName(): ?string
     {
         return null;

src/Internal/SuiteProvider.php

@@ -41,11 +41,13 @@ public static function createForChain(
             : $providers;
     }
 
+    #[\Override]
     public function getLinks(): array
     {
         return [];
     }
 
+    #[\Override]
     public function getLabels(): array
     {
         return [
@@ -56,26 +58,31 @@ public function getLabels(): array
         ];
     }
 
+    #[\Override]
     public function getParameters(): array
     {
         return [];
     }
 
+    #[\Override]
     public function getDisplayName(): ?string
     {
         return null;
     }
 
+    #[\Override]
     public function getDescription(): ?string
     {
         return null;
     }
 
+    #[\Override]
     public function getDescriptionHtml(): ?string
     {
         return null;
     }
 
+    #[\Override]
     public function getFullName(): ?string
     {
         return null;

src/Internal/TestInfoProvider.php

@@ -23,11 +23,13 @@ public static function createForChain(TestInfo $info): array
         return [new self($info)];
     }
 
+    #[\Override]
     public function getLinks(): array
     {
         return [];
     }
 
+    #[\Override]
     public function getLabels(): array
     {
         return [
@@ -38,26 +40,31 @@ public function getLabels(): array
         ];
     }
 
+    #[\Override]
     public function getParameters(): array
     {
         return [];
     }
 
+    #[\Override]
     public function getDisplayName(): ?string
     {
         return null;
     }
 
+    #[\Override]
     public function getDescription(): ?string
     {
         return null;
     }
 
+    #[\Override]
     public function getDescriptionHtml(): ?string
     {
         return null;
     }
 
+    #[\Override]
     public function getFullName(): ?string
     {
         return null;