Merge pull request #1 from CodingItWrong/patch-options

Patch options

Author: CodingItWrong

package.json

@@ -7,7 +7,7 @@
   "author": "Josh Justice <[email protected]>",
   "license": "Apache-2.0",
   "scripts": {
-    "test": "jest",
+    "test": "jest --watchAll",
     "lint": "eslint \"{src,test}/*.js\"",
     "format": "prettier --write \"{src,test}/*.js\"",
     "docs:dev": "vuepress dev docs",

src/Resource.js

@@ -83,12 +83,21 @@ class Resource {
       .catch(extractErrorResponse);
   }
 
-  update(partialRecord) {
+  update({ id, attributes, relationships, options }) {
     // http://jsonapi.org/faq/#wheres-put
-    const record = Object.assign({}, partialRecord, { type: this.name });
+    const record = { type: this.name, id };
+    if (attributes) {
+      record.attributes = attributes;
+    }
+    if (relationships) {
+      record.relationships = relationships;
+    }
     const requestData = { data: record };
     return this.api
-      .patch(`${this.name}/${record.id}`, requestData)
+      .patch(
+        `${this.name}/${record.id}?${getOptionsQuery(options)}`,
+        requestData,
+      )
       .then(extractData)
       .catch(extractErrorResponse);
   }

test/Resource.spec.js

@@ -273,19 +273,38 @@ describe('Resource', () => {
   });
 
   describe('update', () => {
+    const id = '1';
+    const attributes = { key: 'value' };
+    const relationships = { key: 'value' };
+
     it('can update a record', () => {
-      const partialRecord = { id: '1', attributes: { key: 'value' } };
       const responseBody = { data: record };
       api.patch.mockResolvedValue({ data: responseBody });
 
-      const result = resource.update(partialRecord);
+      const result = resource.update({ id, attributes, relationships });
 
-      expect(api.patch).toHaveBeenCalledWith('widgets/1', {
-        data: { ...partialRecord, type: 'widgets' },
+      expect(api.patch).toHaveBeenCalledWith('widgets/1?', {
+        data: { id, type: 'widgets', attributes, relationships },
       });
       return expect(result).resolves.toEqual(responseBody);
     });
 
+    it('passes options', () => {
+      const responseBody = { data: record };
+      api.patch.mockResolvedValue({ data: responseBody });
+
+      const result = resource.update({
+        id,
+        attributes,
+        relationships,
+        options: optionsWithInclude,
+      });
+
+      expect(api.patch).toHaveBeenCalledWith('widgets/1?include=comments', {
+        data: { id, type: 'widgets', attributes, relationships },
+      });
+    });
+
     it('rejects with the response upon error', () => {
       const errorResponse = { dummy: 'data' };
       api.patch.mockRejectedValue({ response: errorResponse });