Server should always respond with `Content-type: application/vnd.api+…

…json` as per specification.

CHANGELOG.md

@@ -4,14 +4,20 @@ All Notable changes to `json-api-server` will be documented in this file.
 
 Updates should follow the [Keep a CHANGELOG](http://keepachangelog.com/) principles.
 
+## Unreleased
+
+### Fixed
+
+- Server should always respond with `Content-type: application/vnd.api+json`
+
 ## 0.3.3
 
 ### Fixed
 
 - The `authentication_test.stub` called `PUT` instead of `PATCH` (issue #14)
 
 ## 0.3.2
-
+self
 ### Fixed
 
 - Enable disable middleware for tests and add "Accept: application/vnd.api+json" header when running test calls.

src/Services/ResponseService.php

@@ -21,10 +21,10 @@ protected function createResponse($responseModel, $content)
         if ($responseModel instanceof RespondError) {
             $errors = $this->formatErrors($responseModel, $content);
 
-            return response($errors, $responseModel->getStatusCode());
+            return response()->json($errors, $responseModel->getStatusCode(), ['Content-Type' => 'application/vnd.api+json']);
         }
 
-        return response($content, $responseModel->getStatusCode());
+        return response()->json($content, $responseModel->getStatusCode(), ['Content-Type' => 'application/vnd.api+json']);
     }
 
     public function respondWithResourceCollection($strResponseModel, $content)
@@ -33,6 +33,7 @@ public function respondWithResourceCollection($strResponseModel, $content)
 
         return (new BaseApiCollectionResource($content))
             ->response()
+            ->header('Content-Type', 'application/vnd.api+json')
             ->setStatusCode($responseModel->getStatusCode());
     }
 
@@ -45,6 +46,7 @@ public function responseWithResource($strResponseModel, $content)
 
         return (new BaseApiResource($content))
             ->response()
+            ->header('Content-Type', 'application/vnd.api+json')
             ->setStatusCode($responseModel->getStatusCode());
     }
 

tests/Unit/HandleResponsesTest.php

@@ -9,6 +9,7 @@
 namespace Tests\Unit;
 
 use Swis\JsonApi\Server\Traits\HandleResponses;
+use Symfony\Component\HttpFoundation\ResponseHeaderBag;
 use Tests\TestCase;
 use Tests\TestClasses\TestModel;
 
@@ -29,6 +30,7 @@ public function test_respond_with_forbidden()
         $response = $this->mock->respondWithForbidden('FORBIDDEN');
 
         $this->assertEquals('FORBIDDEN', json_decode($response->getContent())->errors[0]->detail);
+        $this->assertEquals('application/vnd.api+json', $response->headers->get('Content-Type'));
         $this->assertEquals(403, $response->getStatusCode());
     }
 
@@ -38,6 +40,7 @@ public function test_respond_with_bad_request()
         $response = $this->mock->respondWithBadRequest('BAD REQUEST');
 
         $this->assertEquals('BAD REQUEST', json_decode($response->getContent())->errors[0]->detail);
+        $this->assertEquals('application/vnd.api+json', $response->headers->get('Content-Type'));
         $this->assertEquals(400, $response->getStatusCode());
     }
 
@@ -47,6 +50,7 @@ public function test_respond_with_not_found()
         $response = $this->mock->respondWithNotFound('NOT FOUND');
 
         $this->assertEquals('NOT FOUND', json_decode($response->getContent())->errors[0]->detail);
+        $this->assertEquals('application/vnd.api+json', $response->headers->get('Content-Type'));
         $this->assertEquals(404, $response->getStatusCode());
     }
 
@@ -56,9 +60,12 @@ public function test_respond_with_collection_model()
         $model = new TestModel();
         $model->body = 'TEST';
         $response = $this->mock->respondWithCollection($model);
+
+
         $responseBody = json_decode($response->getContent())->data;
 
         $this->assertEquals(200, $response->getStatusCode());
+        $this->assertEquals('application/vnd.api+json', $response->headers->get('Content-Type'));
         $this->assertEquals('TEST', $responseBody->attributes->body);
     }
 }