Merge pull request #3 from CodingItWrong/update-type

Allow Omitting Type for Update

Author: CodingItWrong

src/Resource.js

@@ -83,8 +83,9 @@ class Resource {
       .catch(extractErrorResponse);
   }
 
-  update(record) {
+  update(partialRecord) {
     // http://jsonapi.org/faq/#wheres-put
+    const record = Object.assign({}, partialRecord, { type: this.name });
     const requestData = { data: record };
     return this.api
       .patch(`${this.name}/${record.id}`, requestData)

test/Resource.spec.js

@@ -244,19 +244,19 @@ describe('Resource', () => {
 
   describe('create', () => {
     it('can create a record', () => {
-      const expectedRequestBody = {
-        data: {
-          type: 'widgets',
-          ...record,
-        },
-      };
+      const partialRecord = { attributes: { key: 'value' } };
 
       const responseBody = { data: record };
       api.post.mockResolvedValue({ data: responseBody });
 
-      const result = resource.create(record);
+      const result = resource.create(partialRecord);
 
-      expect(api.post).toHaveBeenCalledWith('widgets', expectedRequestBody);
+      expect(api.post).toHaveBeenCalledWith('widgets', {
+        data: {
+          ...partialRecord,
+          type: 'widgets',
+        },
+      });
       return expect(result).resolves.toEqual(responseBody);
     });
 
@@ -274,12 +274,15 @@ describe('Resource', () => {
 
   describe('update', () => {
     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(record);
+      const result = resource.update(partialRecord);
 
-      expect(api.patch).toHaveBeenCalledWith('widgets/1', { data: record });
+      expect(api.patch).toHaveBeenCalledWith('widgets/1', {
+        data: { ...partialRecord, type: 'widgets' },
+      });
       return expect(result).resolves.toEqual(responseBody);
     });