Merge 555d30a into 5b16929

app/ErrorHandler.php

@@ -0,0 +1,28 @@
+<?php declare(strict_types=1);
+
+namespace CultuurNet\UDB3\UiTPASService;
+
+use Crell\ApiProblem\ApiProblem;
+use CultuurNet\UDB3\HttpFoundation\Response\ApiProblemJsonResponse;
+
+class ErrorHandler
+{
+
+    public function __invoke(\Exception $exception) : ApiProblem
+    {
+        $problem = new ApiProblem($this->formatMessage($exception));
+        $problem->setStatus($exception->getCode() ?: ApiProblemJsonResponse::HTTP_BAD_REQUEST);
+        return $problem;
+    }
+
+    /**
+     * @param \Exception $exception
+     * @return string
+     */
+    private function formatMessage(\Exception $exception): string
+    {
+        $message = $exception->getMessage();
+        return preg_replace('/URL CALLED.*/','',$message);
+    }
+
+}

app/ErrorHandlerProvider.php

@@ -2,8 +2,6 @@
 
 namespace CultuurNet\UDB3\UiTPASService;
 
-use Crell\ApiProblem\ApiProblem;
-use CultuurNet\UDB3\HttpFoundation\Response\ApiProblemJsonResponse;
 use Exception;
 use Silex\Application;
 use Silex\ServiceProviderInterface;
@@ -14,19 +12,11 @@ public function register(Application $app)
     {
         $app->error(
             function (Exception $e) {
-                $problem = $this->createNewApiProblem($e);
-                return new ApiProblemJsonResponse($problem);
+                return (new ErrorHandler())->__invoke($e);
             }
         );
     }
 
-    protected function createNewApiProblem(Exception $e): ApiProblem
-    {
-        $problem = new ApiProblem($e->getMessage());
-        $problem->setStatus($e->getCode() ?: ApiProblemJsonResponse::HTTP_BAD_REQUEST);
-        return $problem;
-    }
-
     public function boot(Application $app)
     {
     }

tests/ErrorHandlerTest.php

@@ -0,0 +1,48 @@
+<?php declare(strict_types=1);
+
+namespace CultuurNet\UDB3\UiTPASService;
+
+use PHPUnit\Framework\TestCase;
+
+class ErrorHandlerTest extends TestCase
+{
+    /**
+     * @test
+     */
+    public function it_creates_api_problem_from_exception()
+    {
+        $exception = new \Exception('Exception message', 500);
+
+        $errorHandler = new ErrorHandler();
+        $apiProblem = $errorHandler->__invoke($exception);
+        $this->assertEquals('Exception message', $apiProblem->getTitle());
+        $this->assertEquals(500, $apiProblem->getStatus());
+    }
+
+    /**
+     * @test
+     */
+    public function it_creates_api_problem_with_400_status_if_exception_has_no_status_code()
+    {
+        $exception = new \Exception('Exception message');
+
+        $errorHandler = new ErrorHandler();
+        $apiProblem = $errorHandler->__invoke($exception);
+        $this->assertEquals('Exception message', $apiProblem->getTitle());
+        $this->assertEquals(400, $apiProblem->getStatus());
+    }
+
+    /**
+     * @test
+     */
+    public function it_strips_URL_CALLED_from_api_problem_message_title()
+    {
+        // the resulting ApiProblem title should be stripped
+        // of everything after URL CALLED
+        $exception = new \Exception('Exception message URL CALLED: https://acc.uitid.be/uitid/rest/uitpas/cultureevent/de343e38-d656-4928-96bc-55578e0d94ec/cardsystems POST DATA: cardSystemId=3');
+
+        $errorHandler = new ErrorHandler();
+        $apiProblem = $errorHandler->__invoke($exception);
+        $this->assertEquals('Exception message ', $apiProblem->getTitle());
+    }
+}