Merge pull request #122 from tlconnor/fixAcceptHeader

Default Accept and Content-Type headers to application/vnd.api+json
jsonapi-suite · Nov 30, 2018 · d854bab · d854bab
2 parents 2196041 + 1d0af3c
commit d854bab
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 9 deletions.
diff --git a/src/model.ts b/src/model.ts
@@ -671,8 +671,8 @@ export class JSORMBase {
   static fetchOptions(): RequestInit {
     const options = {
       headers: {
-        Accept: "application/json",
-        ["Content-Type"]: "application/json"
+        Accept: "application/vnd.api+json",
+        ["Content-Type"]: "application/vnd.api+json"
       } as any
     }
 
@@ -941,4 +941,4 @@ export const isModelInstance = (arg: any): arg is JSORMBase => {
     return false
   }
   return isModelClass(arg.constructor.currentClass)
-}
+}
diff --git a/test/integration/fetch-middleware.test.ts b/test/integration/fetch-middleware.test.ts
@@ -152,8 +152,8 @@ describe("fetch middleware", () => {
           expect(before.url).to.eq("http://example.com/api/v1/authors")
           expect(before.options).to.deep.eq({
             headers: {
-              Accept: "application/json",
-              "Content-Type": "application/json",
+              Accept: "application/vnd.api+json",
+              "Content-Type": "application/vnd.api+json",
               "CUSTOM-HEADER": "whatever"
             },
             method: "GET"
@@ -309,9 +309,9 @@ describe("fetch middleware", () => {
           expect(before.url).to.eq("http://example.com/api/v1/authors")
           expect(before.options).to.deep.eq({
             headers: {
-              Accept: "application/json",
+              Accept: "application/vnd.api+json",
               "CUSTOM-HEADER": "whatever",
-              "Content-Type": "application/json"
+              "Content-Type": "application/vnd.api+json"
             },
             body: JSON.stringify({ data: { type: "authors" } }),
             method: "POST"

diff --git a/test/unit/model.test.ts b/test/unit/model.test.ts
@@ -1485,8 +1485,8 @@ describe("Model", () => {
 
     it("includes the content headers", () => {
       const headers: any = Author.fetchOptions().headers
-      expect(headers.Accept).to.eq("application/json")
-      expect(headers["Content-Type"]).to.eq("application/json")
+      expect(headers.Accept).to.eq("application/vnd.api+json")
+      expect(headers["Content-Type"]).to.eq("application/vnd.api+json")
     })
   })
 })