Merge pull request #3 from GoIntegro/fix_undefined_relationships

fix undefined relationship

Author: nicosefer

src/index.ts

@@ -484,9 +484,8 @@ export class JSONAPISerializer {
           if (value?.id) {
             output.id = value.id;
           }
-          serializedRelationships[kebabCase(property)] = isNull(value)
-            ? null
-            : output;
+          serializedRelationships[kebabCase(property)] =
+            isNull(value) || isUndefined(value) ? null : output;
         }
       }
     }

tests/index.test.ts

@@ -634,3 +634,33 @@ test("JSONAPIDeserializer: polymorphic megapost", async () => {
 
   expect(data.items[0].author.profile.avatar.url).toEqual("avatar-1.jpg");
 });
+
+test("JSONAPISerializer: null relationship with data null", async () => {
+  const inputData = {
+    id: "1",
+    name: "El discurso del rey",
+    duration: 120,
+    director: null,
+  };
+
+  const movieSerializer = new MovieSerializer();
+  const output = movieSerializer.serialize({ data: inputData });
+  const { data } = output;
+
+  expect(data.relationships.director.data).toBeNull();
+});
+
+test("JSONAPISerializer: undefined relationship with data null", async () => {
+  const inputData = {
+    id: "1",
+    name: "El discurso del rey",
+    duration: 120,
+    director: undefined,
+  };
+
+  const movieSerializer = new MovieSerializer();
+  const output = movieSerializer.serialize({ data: inputData });
+  const { data } = output;
+
+  expect(data.relationships.director.data).toBeNull();
+});