Merge pull request #29 from phpro/php-85-upgrade

veewee · web-flow · commit bf1d71167406 · 2025-10-21T11:49:28.000+02:00
Upgrade to PHP 8.5
diff --git a/.github/workflows/grumphp.yaml b/.github/workflows/grumphp.yaml
@@ -7,7 +7,7 @@ jobs:
         strategy:
             matrix:
                 operating-system: [ubuntu-latest]
-                php-versions: ['8.1', '8.2', '8.3', '8.4']
+                php-versions: ['8.3', '8.4', '8.5']
                 composer-options: ['', '--prefer-lowest']
             fail-fast: false
         name: PHP ${{ matrix.php-versions }} @ ${{ matrix.operating-system }} with ${{ matrix.composer-options }}
@@ -21,7 +21,7 @@ jobs:
                   tools: 'composer:v2'
                   extensions: pcov, mbstring, posix
             - name: Set env vars for latest PHP version
-              if: matrix.php-versions == '8.4'
+              if: matrix.php-versions == '8.5'
               run: |
                 export COMPOSER_IGNORE_PLATFORM_REQ=php+
                 export BOX_REQUIREMENT_CHECKER=0
@@ -34,7 +34,7 @@ jobs:
               id: composercache
               run: echo "::set-output name=dir::$(composer config cache-files-dir)"
             - name: Cache dependencies
-              uses: actions/cache@v2
+              uses: actions/cache@v4
               with:
                   path: ${{ steps.composercache.outputs.dir }}
                   key: ${{ runner.os }}-composer-${{ hashFiles('**/composer.json') }}
@@ -43,5 +43,6 @@ jobs:
               run: composer update --prefer-dist --no-progress --ignore-platform-req=php+ ${{ matrix.composer-options }}
             - name: Run the tests
               run: php vendor/bin/grumphp run --no-interaction
+              continue-on-error: ${{ matrix.php-versions == '8.5' }}
               env:
                 PHP_CS_FIXER_IGNORE_ENV: 1
diff --git a/composer.json b/composer.json
@@ -10,19 +10,19 @@
         }
     ],
     "require": {
-        "php": "~8.1.0 || ~8.2.0 || ~8.3.0 || ~8.4.0",
+        "php": "~8.3.0 || ~8.4.0 || ~8.5.0",
         "phpro/api-problem": "^1.0",
         "symfony/dependency-injection": "^5.4 || ^6.0 || ^7.0",
         "symfony/event-dispatcher": "^5.4 || ^6.0 || ^7.0",
         "symfony/http-kernel": "^6.4 || ^7.0"
     },
     "require-dev": {
-        "friendsofphp/php-cs-fixer": "^3.39",
         "matthiasnoback/symfony-dependency-injection-test": "^4.3",
-        "phpro/grumphp-shim": "^2.10",
-        "phpspec/prophecy-phpunit": "^2.0",
+        "php-cs-fixer/shim": "^3.88",
+        "phpro/grumphp-shim": "^2.17",
         "phpspec/prophecy": "^1.17",
-        "phpunit/phpunit": "^9.5",
+        "phpspec/prophecy-phpunit": "^2.0",
+        "phpunit/phpunit": "^12.4",
         "symfony/security-core": "^5.4 || ^6.0 || ^7.0"
     },
     "config": {
diff --git a/phpunit.xml.dist b/phpunit.xml.dist
@@ -1,21 +1,28 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <phpunit
         xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
-        xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/9.3/phpunit.xsd"
+        xsi:noNamespaceSchemaLocation="vendor/phpunit/phpunit/phpunit.xsd"
         colors="true"
+        displayDetailsOnTestsThatTriggerWarnings="true"
+        failOnWarning="true"
+        failOnPhpunitWarning="true"
+        beStrictAboutOutputDuringTests="true"
 >
     <testsuites>
         <testsuite name="Unit">
             <directory>./test</directory>
         </testsuite>
     </testsuites>
 
-  <coverage processUncoveredFiles="true">
-      <include>
-          <directory suffix=".php">src</directory>
-      </include>
-      <report>
-          <clover outputFile="var/coverage.xml"/>
-      </report>
-  </coverage>
+    <source>
+        <include>
+            <directory suffix=".php">src</directory>
+        </include>
+    </source>
+
+    <coverage includeUncoveredFiles="true">
+        <report>
+            <clover outputFile="var/coverage.xml"/>
+        </report>
+    </coverage>
 </phpunit>
diff --git a/test/ApiProblemBundleTest.php b/test/ApiProblemBundleTest.php
@@ -5,13 +5,15 @@
 namespace PhproTest\ApiProblemBundle;
 
 use Phpro\ApiProblemBundle\ApiProblemBundle;
+use PHPUnit\Framework\Attributes\CoversClass;
+use PHPUnit\Framework\Attributes\Test;
 use PHPUnit\Framework\TestCase;
 use Symfony\Component\HttpKernel\Bundle\Bundle;
 
-/** @covers \Phpro\ApiProblemBundle\ApiProblemBundle */
+#[CoversClass(ApiProblemBundle::class)]
 class ApiProblemBundleTest extends TestCase
 {
-    /** @test */
+    #[Test]
     public function it_is_a_symfony_bundle(): void
     {
         $this->assertInstanceOf(Bundle::class, new ApiProblemBundle());
diff --git a/test/DependencyInjection/ApiProblemExtensionTest.php b/test/DependencyInjection/ApiProblemExtensionTest.php
@@ -8,11 +8,11 @@
 use Phpro\ApiProblemBundle\DependencyInjection\ApiProblemExtension;
 use Phpro\ApiProblemBundle\EventListener\JsonApiProblemExceptionListener;
 use Phpro\ApiProblemBundle\Transformer;
+use PHPUnit\Framework\Attributes\CoversClass;
+use PHPUnit\Framework\Attributes\Test;
 use Symfony\Component\DependencyInjection\Argument\TaggedIteratorArgument;
 
-/**
- * @covers \Phpro\ApiProblemBundle\DependencyInjection\ApiProblemExtension
- */
+#[CoversClass(ApiProblemExtension::class)]
 class ApiProblemExtensionTest extends AbstractExtensionTestCase
 {
     private const TRANSFORMER_TAG = 'phpro.api_problem.exception_transformer';
@@ -22,7 +22,7 @@ protected function getContainerExtensions(): array
         return [new ApiProblemExtension()];
     }
 
-    /** @test */
+    #[Test]
     public function it_registers_json_exception_listener(): void
     {
         $this->load([]);
@@ -52,7 +52,7 @@ public function it_registers_json_exception_listener(): void
         );
     }
 
-    /** @test */
+    #[Test]
     public function it_contains_exception_transformers(): void
     {
         $this->load([]);
diff --git a/test/EventListener/JsonApiProblemExceptionListenerTest.php b/test/EventListener/JsonApiProblemExceptionListenerTest.php
@@ -13,6 +13,8 @@
 use Phpro\ApiProblem\Http\HttpApiProblem;
 use Phpro\ApiProblemBundle\EventListener\JsonApiProblemExceptionListener;
 use Phpro\ApiProblemBundle\Transformer\ExceptionTransformerInterface;
+use PHPUnit\Framework\Attributes\CoversClass;
+use PHPUnit\Framework\Attributes\Test;
 use PHPUnit\Framework\TestCase;
 use Prophecy\Argument;
 use Prophecy\PhpUnit\ProphecyTrait;
@@ -22,7 +24,7 @@
 use Symfony\Component\HttpKernel\Event\ExceptionEvent;
 use Symfony\Component\HttpKernel\HttpKernelInterface;
 
-/** @covers \Phpro\ApiProblemBundle\EventListener\JsonApiProblemExceptionListener */
+#[CoversClass(JsonApiProblemExceptionListener::class)]
 class JsonApiProblemExceptionListenerTest extends TestCase
 {
     use ProphecyTrait;
@@ -38,7 +40,7 @@ protected function setUp(): void
         $this->exceptionTransformer->accepts(Argument::any())->willReturn(false);
     }
 
-    /** @test */
+    #[Test]
     public function it_does_nothing_on_non_json_requests(): void
     {
         $listener = new JsonApiProblemExceptionListener($this->exceptionTransformer->reveal(), false);
@@ -53,7 +55,7 @@ public function it_does_nothing_on_non_json_requests(): void
         $this->assertNull($event->getResponse());
     }
 
-    /** @test */
+    #[Test]
     public function it_runs_on_json_route_formats(): void
     {
         $listener = new JsonApiProblemExceptionListener($this->exceptionTransformer->reveal(), false);
@@ -69,7 +71,7 @@ public function it_runs_on_json_route_formats(): void
         $this->assertApiProblemWithResponseBody($event, 500, $this->parseDataForException($exception));
     }
 
-    /** @test */
+    #[Test]
     public function it_runs_on_json_content_types(): void
     {
         $listener = new JsonApiProblemExceptionListener($this->exceptionTransformer->reveal(), false);
@@ -84,7 +86,7 @@ public function it_runs_on_json_content_types(): void
         $this->assertApiProblemWithResponseBody($event, 500, $this->parseDataForException($exception));
     }
 
-    /** @test */
+    #[Test]
     public function it_runs_on_json_accept_header(): void
     {
         $listener = new JsonApiProblemExceptionListener($this->exceptionTransformer->reveal(), false);
@@ -99,7 +101,7 @@ public function it_runs_on_json_accept_header(): void
         $this->assertApiProblemWithResponseBody($event, 500, $this->parseDataForException($exception));
     }
 
-    /** @test */
+    #[Test]
     public function it_parses_an_api_problem_response(): void
     {
         $listener = new JsonApiProblemExceptionListener($this->exceptionTransformer->reveal(), false);
@@ -114,7 +116,7 @@ public function it_parses_an_api_problem_response(): void
         $this->assertApiProblemWithResponseBody($event, 500, $this->parseDataForException($exception));
     }
 
-    /** @test */
+    #[Test]
     public function it_uses_an_exception_transformer(): void
     {
         $listener = new JsonApiProblemExceptionListener($this->exceptionTransformer->reveal(), false);
@@ -134,7 +136,7 @@ public function it_uses_an_exception_transformer(): void
         $this->assertApiProblemWithResponseBody($event, 400, []);
     }
 
-    /** @test */
+    #[Test]
     public function it_returns_the_status_code_from_the_api_problem(): void
     {
         $listener = new JsonApiProblemExceptionListener($this->exceptionTransformer->reveal(), false);
@@ -154,7 +156,7 @@ public function it_returns_the_status_code_from_the_api_problem(): void
         $this->assertApiProblemWithResponseBody($event, 123, ['status' => 123]);
     }
 
-    /** @test */
+    #[Test]
     public function it_parses_a_debuggable_api_problem_response(): void
     {
         $listener = new JsonApiProblemExceptionListener($this->exceptionTransformer->reveal(), true);
diff --git a/test/Exception/ApiProblemHttpExceptionTest.php b/test/Exception/ApiProblemHttpExceptionTest.php
@@ -7,6 +7,7 @@
 use Phpro\ApiProblem\Http\HttpApiProblem;
 use Phpro\ApiProblemBundle\Exception\ApiProblemHttpException;
 use Phpro\ApiProblemBundle\Transformer\ApiProblemExceptionTransformer;
+use PHPUnit\Framework\Attributes\Test;
 use PHPUnit\Framework\TestCase;
 use Prophecy\PhpUnit\ProphecyTrait;
 use Prophecy\Prophecy\ObjectProphecy;
@@ -27,23 +28,23 @@ protected function setUp(): void/* The :void return type declaration that should
         $this->apiProblem->toArray()->willReturn([]);
     }
 
-    /** @test */
+    #[Test]
     public function it_is_accepted_by_the__api_problem_exception_transformer(): void
     {
         $transformer = new ApiProblemExceptionTransformer();
 
         $this->assertTrue($transformer->accepts(new ApiProblemHttpException($this->apiProblem->reveal())));
     }
 
-    /** @test */
+    #[Test]
     public function it_is_an_instance_of__http_exception(): void
     {
         $exception = new ApiProblemHttpException($this->apiProblem->reveal());
 
         $this->assertInstanceOf(HttpException::class, $exception);
     }
 
-    /** @test */
+    #[Test]
     public function it_contains_an_api_problem(): void
     {
         $apiProblem = $this->apiProblem->reveal();
@@ -52,23 +53,23 @@ public function it_contains_an_api_problem(): void
         $this->assertEquals($apiProblem, $exception->getApiProblem());
     }
 
-    /** @test */
+    #[Test]
     public function it_returns_the_correct_http_headers(): void
     {
         $exception = new ApiProblemHttpException($this->apiProblem->reveal());
 
         $this->assertEquals(['Content-Type' => 'application/problem+json'], $exception->getHeaders());
     }
 
-    /** @test */
+    #[Test]
     public function it_returns_the_correct_default_http_statuscode(): void
     {
         $exception = new ApiProblemHttpException($this->apiProblem->reveal());
 
         $this->assertEquals(400, $exception->getStatusCode());
     }
 
-    /** @test */
+    #[Test]
     public function it_returns_the_correct_specified_http_statuscode(): void
     {
         $this->apiProblem->toArray()->willReturn(['status' => 401]);
diff --git a/test/Transformer/ApiProblemExceptionTransformerTest.php b/test/Transformer/ApiProblemExceptionTransformerTest.php
@@ -9,13 +9,13 @@
 use Phpro\ApiProblem\Exception\ApiProblemException;
 use Phpro\ApiProblemBundle\Transformer\ApiProblemExceptionTransformer;
 use Phpro\ApiProblemBundle\Transformer\ExceptionTransformerInterface;
+use PHPUnit\Framework\Attributes\CoversClass;
+use PHPUnit\Framework\Attributes\Test;
 use PHPUnit\Framework\TestCase;
 use Prophecy\PhpUnit\ProphecyTrait;
 use Prophecy\Prophecy\ObjectProphecy;
 
-/**
- * @covers \Phpro\ApiProblemBundle\Transformer\ApiProblemExceptionTransformer
- */
+#[CoversClass(ApiProblemExceptionTransformer::class)]
 class ApiProblemExceptionTransformerTest extends TestCase
 {
     use ProphecyTrait;
@@ -31,14 +31,14 @@ protected function setUp(): void/* The :void return type declaration that should
         $this->apiProblem->toArray()->willReturn([]);
     }
 
-    /** @test */
+    #[Test]
     public function it_is_an_exception_transformer(): void
     {
         $transformer = new ApiProblemExceptionTransformer();
         $this->assertInstanceOf(ExceptionTransformerInterface::class, $transformer);
     }
 
-    /** @test */
+    #[Test]
     public function it_accepts_api_problem_exceptions(): void
     {
         $transformer = new ApiProblemExceptionTransformer();
@@ -47,7 +47,7 @@ public function it_accepts_api_problem_exceptions(): void
         $this->assertFalse($transformer->accepts(new Exception()));
     }
 
-    /** @test */
+    #[Test]
     public function it_transforms_exception_to_api_problem(): void
     {
         $transformer = new ApiProblemExceptionTransformer();
diff --git a/test/Transformer/ChainTest.php b/test/Transformer/ChainTest.php
@@ -9,33 +9,33 @@
 use Phpro\ApiProblem\Http\ExceptionApiProblem;
 use Phpro\ApiProblemBundle\Transformer\Chain;
 use Phpro\ApiProblemBundle\Transformer\ExceptionTransformerInterface;
+use PHPUnit\Framework\Attributes\CoversClass;
+use PHPUnit\Framework\Attributes\Test;
 use PHPUnit\Framework\TestCase;
 use Prophecy\Argument;
 use Prophecy\PhpUnit\ProphecyTrait;
 use Prophecy\Prophecy\ObjectProphecy;
 
-/**
- * @covers \Phpro\ApiProblemBundle\Transformer\Chain
- */
+#[CoversClass(Chain::class)]
 class ChainTest extends TestCase
 {
     use ProphecyTrait;
 
-    /** @test */
+    #[Test]
     public function it_is_an_exception_transformer(): void
     {
         $transformer = new Chain([]);
         $this->assertInstanceOf(ExceptionTransformerInterface::class, $transformer);
     }
 
-    /** @test */
+    #[Test]
     public function it_accepts_any_exception(): void
     {
         $transformer = new Chain([]);
         $this->assertTrue($transformer->accepts(new Exception()));
     }
 
-    /** @test */
+    #[Test]
     public function it_transforms_with_first_acceptable_transformer(): void
     {
         $transformer = new Chain([
@@ -47,7 +47,7 @@ public function it_transforms_with_first_acceptable_transformer(): void
         $this->assertEquals($apiProblem1, $transformer->transform(new Exception()));
     }
 
-    /** @test */
+    #[Test]
     public function it_transforms_to_basic_exception_problem_when_no_transformer_matches(): void
     {
         $transformer = new Chain([$this->mockTransformer(false)]);
diff --git a/test/Transformer/HttpExceptionTransformerTest.php b/test/Transformer/HttpExceptionTransformerTest.php
diff --git a/test/Transformer/SecurityExceptionTransformerTest.php b/test/Transformer/SecurityExceptionTransformerTest.php

Original file line number	Diff line number	Diff line change
`@@ -8,11 +8,11 @@`
`8`	`8`	`use Phpro\ApiProblemBundle\DependencyInjection\ApiProblemExtension;`
`9`	`9`	`use Phpro\ApiProblemBundle\EventListener\JsonApiProblemExceptionListener;`
`10`	`10`	`use Phpro\ApiProblemBundle\Transformer;`
	`11`	`+use PHPUnit\Framework\Attributes\CoversClass;`
	`12`	`+use PHPUnit\Framework\Attributes\Test;`
`11`	`13`	`use Symfony\Component\DependencyInjection\Argument\TaggedIteratorArgument;`
`12`	`14`
`13`		`-/**`
`14`		`- * @covers \Phpro\ApiProblemBundle\DependencyInjection\ApiProblemExtension`
`15`		`- */`
	`15`	`+#[CoversClass(ApiProblemExtension::class)]`
`16`	`16`	`class ApiProblemExtensionTest extends AbstractExtensionTestCase`
`17`	`17`	`{`
`18`	`18`	`private const TRANSFORMER_TAG = 'phpro.api_problem.exception_transformer';`
`@@ -22,7 +22,7 @@ protected function getContainerExtensions(): array`
`22`	`22`	`return [new ApiProblemExtension()];`
`23`	`23`	`}`
`24`	`24`
`25`		`- /** @test */`
	`25`	`+ #[Test]`
`26`	`26`	`public function it_registers_json_exception_listener(): void`
`27`	`27`	`{`
`28`	`28`	`$this->load([]);`
`@@ -52,7 +52,7 @@ public function it_registers_json_exception_listener(): void`
`52`	`52`	`);`
`53`	`53`	`}`
`54`	`54`
`55`		`- /** @test */`
	`55`	`+ #[Test]`
`56`	`56`	`public function it_contains_exception_transformers(): void`
`57`	`57`	`{`
`58`	`58`	`$this->load([]);`
Original file line number	Diff line number	Diff line change
`@@ -13,6 +13,8 @@`
`13`	`13`	`use Phpro\ApiProblem\Http\HttpApiProblem;`
`14`	`14`	`use Phpro\ApiProblemBundle\EventListener\JsonApiProblemExceptionListener;`
`15`	`15`	`use Phpro\ApiProblemBundle\Transformer\ExceptionTransformerInterface;`
	`16`	`+use PHPUnit\Framework\Attributes\CoversClass;`
	`17`	`+use PHPUnit\Framework\Attributes\Test;`
`16`	`18`	`use PHPUnit\Framework\TestCase;`
`17`	`19`	`use Prophecy\Argument;`
`18`	`20`	`use Prophecy\PhpUnit\ProphecyTrait;`
`@@ -22,7 +24,7 @@`
`22`	`24`	`use Symfony\Component\HttpKernel\Event\ExceptionEvent;`
`23`	`25`	`use Symfony\Component\HttpKernel\HttpKernelInterface;`
`24`	`26`
`25`		`-/** @covers \Phpro\ApiProblemBundle\EventListener\JsonApiProblemExceptionListener */`
	`27`	`+#[CoversClass(JsonApiProblemExceptionListener::class)]`
`26`	`28`	`class JsonApiProblemExceptionListenerTest extends TestCase`
`27`	`29`	`{`
`28`	`30`	`use ProphecyTrait;`
`@@ -38,7 +40,7 @@ protected function setUp(): void`
`38`	`40`	`$this->exceptionTransformer->accepts(Argument::any())->willReturn(false);`
`39`	`41`	`}`
`40`	`42`
`41`		`- /** @test */`
	`43`	`+ #[Test]`
`42`	`44`	`public function it_does_nothing_on_non_json_requests(): void`
`43`	`45`	`{`
`44`	`46`	`$listener = new JsonApiProblemExceptionListener($this->exceptionTransformer->reveal(), false);`
`@@ -53,7 +55,7 @@ public function it_does_nothing_on_non_json_requests(): void`
`53`	`55`	`$this->assertNull($event->getResponse());`
`54`	`56`	`}`
`55`	`57`
`56`		`- /** @test */`
	`58`	`+ #[Test]`
`57`	`59`	`public function it_runs_on_json_route_formats(): void`
`58`	`60`	`{`
`59`	`61`	`$listener = new JsonApiProblemExceptionListener($this->exceptionTransformer->reveal(), false);`
`@@ -69,7 +71,7 @@ public function it_runs_on_json_route_formats(): void`
`69`	`71`	`$this->assertApiProblemWithResponseBody($event, 500, $this->parseDataForException($exception));`
`70`	`72`	`}`
`71`	`73`
`72`		`- /** @test */`
	`74`	`+ #[Test]`
`73`	`75`	`public function it_runs_on_json_content_types(): void`
`74`	`76`	`{`
`75`	`77`	`$listener = new JsonApiProblemExceptionListener($this->exceptionTransformer->reveal(), false);`
`@@ -84,7 +86,7 @@ public function it_runs_on_json_content_types(): void`
`84`	`86`	`$this->assertApiProblemWithResponseBody($event, 500, $this->parseDataForException($exception));`
`85`	`87`	`}`
`86`	`88`
`87`		`- /** @test */`
	`89`	`+ #[Test]`
`88`	`90`	`public function it_runs_on_json_accept_header(): void`
`89`	`91`	`{`
`90`	`92`	`$listener = new JsonApiProblemExceptionListener($this->exceptionTransformer->reveal(), false);`
`@@ -99,7 +101,7 @@ public function it_runs_on_json_accept_header(): void`
`99`	`101`	`$this->assertApiProblemWithResponseBody($event, 500, $this->parseDataForException($exception));`
`100`	`102`	`}`
`101`	`103`
`102`		`- /** @test */`
	`104`	`+ #[Test]`
`103`	`105`	`public function it_parses_an_api_problem_response(): void`
`104`	`106`	`{`
`105`	`107`	`$listener = new JsonApiProblemExceptionListener($this->exceptionTransformer->reveal(), false);`
`@@ -114,7 +116,7 @@ public function it_parses_an_api_problem_response(): void`
`114`	`116`	`$this->assertApiProblemWithResponseBody($event, 500, $this->parseDataForException($exception));`
`115`	`117`	`}`
`116`	`118`
`117`		`- /** @test */`
	`119`	`+ #[Test]`
`118`	`120`	`public function it_uses_an_exception_transformer(): void`
`119`	`121`	`{`
`120`	`122`	`$listener = new JsonApiProblemExceptionListener($this->exceptionTransformer->reveal(), false);`
`@@ -134,7 +136,7 @@ public function it_uses_an_exception_transformer(): void`
`134`	`136`	`$this->assertApiProblemWithResponseBody($event, 400, []);`
`135`	`137`	`}`
`136`	`138`
`137`		`- /** @test */`
	`139`	`+ #[Test]`
`138`	`140`	`public function it_returns_the_status_code_from_the_api_problem(): void`
`139`	`141`	`{`
`140`	`142`	`$listener = new JsonApiProblemExceptionListener($this->exceptionTransformer->reveal(), false);`
`@@ -154,7 +156,7 @@ public function it_returns_the_status_code_from_the_api_problem(): void`
`154`	`156`	`$this->assertApiProblemWithResponseBody($event, 123, ['status' => 123]);`
`155`	`157`	`}`
`156`	`158`
`157`		`- /** @test */`
	`159`	`+ #[Test]`
`158`	`160`	`public function it_parses_a_debuggable_api_problem_response(): void`
`159`	`161`	`{`
`160`	`162`	`$listener = new JsonApiProblemExceptionListener($this->exceptionTransformer->reveal(), true);`
Original file line number	Diff line number	Diff line change
`@@ -7,6 +7,7 @@`
`7`	`7`	`use Phpro\ApiProblem\Http\HttpApiProblem;`
`8`	`8`	`use Phpro\ApiProblemBundle\Exception\ApiProblemHttpException;`
`9`	`9`	`use Phpro\ApiProblemBundle\Transformer\ApiProblemExceptionTransformer;`
	`10`	`+use PHPUnit\Framework\Attributes\Test;`
`10`	`11`	`use PHPUnit\Framework\TestCase;`
`11`	`12`	`use Prophecy\PhpUnit\ProphecyTrait;`
`12`	`13`	`use Prophecy\Prophecy\ObjectProphecy;`
`@@ -27,23 +28,23 @@ protected function setUp(): void/* The :void return type declaration that should`
`27`	`28`	`$this->apiProblem->toArray()->willReturn([]);`
`28`	`29`	`}`
`29`	`30`
`30`		`- /** @test */`
	`31`	`+ #[Test]`
`31`	`32`	`public function it_is_accepted_by_the__api_problem_exception_transformer(): void`
`32`	`33`	`{`
`33`	`34`	`$transformer = new ApiProblemExceptionTransformer();`
`34`	`35`
`35`	`36`	`$this->assertTrue($transformer->accepts(new ApiProblemHttpException($this->apiProblem->reveal())));`
`36`	`37`	`}`
`37`	`38`
`38`		`- /** @test */`
	`39`	`+ #[Test]`
`39`	`40`	`public function it_is_an_instance_of__http_exception(): void`
`40`	`41`	`{`
`41`	`42`	`$exception = new ApiProblemHttpException($this->apiProblem->reveal());`
`42`	`43`
`43`	`44`	`$this->assertInstanceOf(HttpException::class, $exception);`
`44`	`45`	`}`
`45`	`46`
`46`		`- /** @test */`
	`47`	`+ #[Test]`
`47`	`48`	`public function it_contains_an_api_problem(): void`
`48`	`49`	`{`
`49`	`50`	`$apiProblem = $this->apiProblem->reveal();`
`@@ -52,23 +53,23 @@ public function it_contains_an_api_problem(): void`
`52`	`53`	`$this->assertEquals($apiProblem, $exception->getApiProblem());`
`53`	`54`	`}`
`54`	`55`
`55`		`- /** @test */`
	`56`	`+ #[Test]`
`56`	`57`	`public function it_returns_the_correct_http_headers(): void`
`57`	`58`	`{`
`58`	`59`	`$exception = new ApiProblemHttpException($this->apiProblem->reveal());`
`59`	`60`
`60`	`61`	`$this->assertEquals(['Content-Type' => 'application/problem+json'], $exception->getHeaders());`
`61`	`62`	`}`
`62`	`63`
`63`		`- /** @test */`
	`64`	`+ #[Test]`
`64`	`65`	`public function it_returns_the_correct_default_http_statuscode(): void`
`65`	`66`	`{`
`66`	`67`	`$exception = new ApiProblemHttpException($this->apiProblem->reveal());`
`67`	`68`
`68`	`69`	`$this->assertEquals(400, $exception->getStatusCode());`
`69`	`70`	`}`
`70`	`71`
`71`		`- /** @test */`
	`72`	`+ #[Test]`
`72`	`73`	`public function it_returns_the_correct_specified_http_statuscode(): void`
`73`	`74`	`{`
`74`	`75`	`$this->apiProblem->toArray()->willReturn(['status' => 401]);`
Original file line number	Diff line number	Diff line change
`@@ -9,13 +9,13 @@`
`9`	`9`	`use Phpro\ApiProblem\Exception\ApiProblemException;`
`10`	`10`	`use Phpro\ApiProblemBundle\Transformer\ApiProblemExceptionTransformer;`
`11`	`11`	`use Phpro\ApiProblemBundle\Transformer\ExceptionTransformerInterface;`
	`12`	`+use PHPUnit\Framework\Attributes\CoversClass;`
	`13`	`+use PHPUnit\Framework\Attributes\Test;`
`12`	`14`	`use PHPUnit\Framework\TestCase;`
`13`	`15`	`use Prophecy\PhpUnit\ProphecyTrait;`
`14`	`16`	`use Prophecy\Prophecy\ObjectProphecy;`
`15`	`17`
`16`		`-/**`
`17`		`- * @covers \Phpro\ApiProblemBundle\Transformer\ApiProblemExceptionTransformer`
`18`		`- */`
	`18`	`+#[CoversClass(ApiProblemExceptionTransformer::class)]`
`19`	`19`	`class ApiProblemExceptionTransformerTest extends TestCase`
`20`	`20`	`{`
`21`	`21`	`use ProphecyTrait;`
`@@ -31,14 +31,14 @@ protected function setUp(): void/* The :void return type declaration that should`
`31`	`31`	`$this->apiProblem->toArray()->willReturn([]);`
`32`	`32`	`}`
`33`	`33`
`34`		`- /** @test */`
	`34`	`+ #[Test]`
`35`	`35`	`public function it_is_an_exception_transformer(): void`
`36`	`36`	`{`
`37`	`37`	`$transformer = new ApiProblemExceptionTransformer();`
`38`	`38`	`$this->assertInstanceOf(ExceptionTransformerInterface::class, $transformer);`
`39`	`39`	`}`
`40`	`40`
`41`		`- /** @test */`
	`41`	`+ #[Test]`
`42`	`42`	`public function it_accepts_api_problem_exceptions(): void`
`43`	`43`	`{`
`44`	`44`	`$transformer = new ApiProblemExceptionTransformer();`
`@@ -47,7 +47,7 @@ public function it_accepts_api_problem_exceptions(): void`
`47`	`47`	`$this->assertFalse($transformer->accepts(new Exception()));`
`48`	`48`	`}`
`49`	`49`
`50`		`- /** @test */`
	`50`	`+ #[Test]`
`51`	`51`	`public function it_transforms_exception_to_api_problem(): void`
`52`	`52`	`{`
`53`	`53`	`$transformer = new ApiProblemExceptionTransformer();`
Original file line number	Diff line number	Diff line change
`@@ -9,33 +9,33 @@`
`9`	`9`	`use Phpro\ApiProblem\Http\ExceptionApiProblem;`
`10`	`10`	`use Phpro\ApiProblemBundle\Transformer\Chain;`
`11`	`11`	`use Phpro\ApiProblemBundle\Transformer\ExceptionTransformerInterface;`
	`12`	`+use PHPUnit\Framework\Attributes\CoversClass;`
	`13`	`+use PHPUnit\Framework\Attributes\Test;`
`12`	`14`	`use PHPUnit\Framework\TestCase;`
`13`	`15`	`use Prophecy\Argument;`
`14`	`16`	`use Prophecy\PhpUnit\ProphecyTrait;`
`15`	`17`	`use Prophecy\Prophecy\ObjectProphecy;`
`16`	`18`
`17`		`-/**`
`18`		`- * @covers \Phpro\ApiProblemBundle\Transformer\Chain`
`19`		`- */`
	`19`	`+#[CoversClass(Chain::class)]`
`20`	`20`	`class ChainTest extends TestCase`
`21`	`21`	`{`
`22`	`22`	`use ProphecyTrait;`
`23`	`23`
`24`		`- /** @test */`
	`24`	`+ #[Test]`
`25`	`25`	`public function it_is_an_exception_transformer(): void`
`26`	`26`	`{`
`27`	`27`	`$transformer = new Chain([]);`
`28`	`28`	`$this->assertInstanceOf(ExceptionTransformerInterface::class, $transformer);`
`29`	`29`	`}`
`30`	`30`
`31`		`- /** @test */`
	`31`	`+ #[Test]`
`32`	`32`	`public function it_accepts_any_exception(): void`
`33`	`33`	`{`
`34`	`34`	`$transformer = new Chain([]);`
`35`	`35`	`$this->assertTrue($transformer->accepts(new Exception()));`
`36`	`36`	`}`
`37`	`37`
`38`		`- /** @test */`
	`38`	`+ #[Test]`
`39`	`39`	`public function it_transforms_with_first_acceptable_transformer(): void`
`40`	`40`	`{`
`41`	`41`	`$transformer = new Chain([`
`@@ -47,7 +47,7 @@ public function it_transforms_with_first_acceptable_transformer(): void`
`47`	`47`	`$this->assertEquals($apiProblem1, $transformer->transform(new Exception()));`
`48`	`48`	`}`
`49`	`49`
`50`		`- /** @test */`
	`50`	`+ #[Test]`
`51`	`51`	`public function it_transforms_to_basic_exception_problem_when_no_transformer_matches(): void`
`52`	`52`	`{`
`53`	`53`	`$transformer = new Chain([$this->mockTransformer(false)]);`