test: add tests when relationship.data is missing

Refs #778
swaggerexpert · Feb 22, 2023 · c7da0f5 · c7da0f5
1 parent 29da7a7
commit c7da0f5
Showing 1 changed file with 39 additions and 1 deletion.
diff --git a/test/index.js b/test/index.js
@@ -639,7 +639,7 @@ describe('jsonApiMerge', function () {
       ],
     };
 
-    context('should create full linkage', function () {
+    specify('should create full linkage', function () {
       const expected = [
         {
           type: 'articles',
@@ -732,6 +732,44 @@ describe('jsonApiMerge', function () {
     });
   });
 
+  context("given relationships doesn't have data property", function () {
+    const jsonApiData = {
+      data: {
+        id: 1,
+        type: 'resource',
+        attributes: {
+          name: 'Resource name',
+        },
+        relationships: {
+          related: {
+            links: {
+              related: {
+                href: 'http://example.com/related-resource/',
+                title: 'Related',
+              },
+            },
+          },
+        },
+      },
+      included: [
+        {
+          id: 2,
+          type: 'related_resource',
+          attributes: {
+            name: 'Related resource name',
+          },
+        },
+      ],
+    };
+
+    specify('should throw error', function () {
+      const included = jsonApiMerge(jsonApiData.included, jsonApiData.included);
+      const thunk = () => jsonApiMerge(included, jsonApiData.data);
+
+      assert.throws(thunk, "Cannot read property 'type' of undefined");
+    });
+  });
+
   it('should curry', function () {
     const jsonApiData = {
       data: {