Added Exception class implementing HttpExceptionInterface

Author: tjveldhuizen

README.md

@@ -6,7 +6,7 @@
 # Api Problem Bundle
 
 This package provides a [RFC7807](https://tools.ietf.org/html/rfc7807) Problem details exception listener for Symfony.
-Internal, this package uses the models provided by `phpro/api-problem`](https://www.github.com/phpro/api-problem).
+Internal, this package uses the models provided by [`phpro/api-problem`](https://www.github.com/phpro/api-problem).
 When an `ApiProblemException` is triggered, this bundle will return the correct response.
 
 
@@ -69,6 +69,9 @@ Body:
 }
 ```
 
+As an alternative, use ```ApiProblemHttpException``` instead of ```ApiProblemException```, to make it possible to
+[exclude the specific status code from the log](https://symfony.com/doc/current/logging/monolog_exclude_http_codes.html)
+
 ## Adding custom exception transformations
 
 Currently, we automatically transform exceptions from following packages to an ApiProblem instance:

src/Exception/ApiProblemHttpException.php

@@ -0,0 +1,22 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Phpro\ApiProblemBundle\Exception;
+
+use Phpro\ApiProblem\Exception\ApiProblemException;
+use Symfony\Component\HttpFoundation\Response;
+use Symfony\Component\HttpKernel\Exception\HttpExceptionInterface;
+
+class ApiProblemHttpException extends ApiProblemException implements HttpExceptionInterface
+{
+    public function getStatusCode()
+    {
+        return $this->code > 0 ? $this->code : Response::HTTP_BAD_REQUEST;
+    }
+
+    public function getHeaders()
+    {
+        return ['Content-Type' => 'application/problem+json'];
+    }
+}

test/Exception/ApiProblemHttpExceptionTest.php

@@ -0,0 +1,67 @@
+<?php
+
+declare(strict_types=1);
+
+namespace PhproTest\ApiProblemBundle\Exception;
+
+use Phpro\ApiProblem\Exception\ApiProblemException;
+use Phpro\ApiProblem\Http\HttpApiProblem;
+use Phpro\ApiProblemBundle\Exception\ApiProblemHttpException;
+use PHPUnit\Framework\TestCase;
+use Prophecy\Prophecy\ObjectProphecy;
+use Symfony\Component\HttpKernel\Exception\HttpExceptionInterface;
+
+class ApiProblemHttpExceptionTest extends TestCase
+{
+    /**
+     * @var HttpApiProblem|ObjectProphecy
+     */
+    private $apiProblem;
+
+    protected function setUp(): void/* The :void return type declaration that should be here would cause a BC issue */
+    {
+        $this->apiProblem = $this->prophesize(HttpApiProblem::class);
+        $this->apiProblem->toArray()->willReturn([]);
+    }
+
+    /** @test */
+    public function it_is_an_instance_of_ApiProblemException(): void
+    {
+        $exception = new ApiProblemHttpException($this->apiProblem->reveal());
+
+        $this->assertInstanceOf(ApiProblemException::class, $exception);
+    }
+
+    /** @test */
+    public function it_is_an_instance_of_HttpException(): void
+    {
+        $exception = new ApiProblemHttpException($this->apiProblem->reveal());
+
+        $this->assertInstanceOf(HttpExceptionInterface::class, $exception);
+    }
+
+    /** @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 */
+    public function it_returns_the_correct_default_http_statuscode(): void
+    {
+        $exception = new ApiProblemHttpException($this->apiProblem->reveal());
+
+        $this->assertEquals(400, $exception->getStatusCode());
+    }
+
+    /** @test */
+    public function it_returns_the_correct_specified_http_statuscode(): void
+    {
+        $this->apiProblem->toArray()->willReturn(['status' => 401]);
+        $exception = new ApiProblemHttpException($this->apiProblem->reveal());
+
+        $this->assertEquals(401, $exception->getStatusCode());
+    }
+}