fix #97 (#99)

hasura · Apr 29, 2022 · d22d77e · d22d77e
1 parent 36f7945
commit d22d77e
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 6 deletions.
diff --git a/src/getResponseParser/sanitizeResource.test.ts b/src/getResponseParser/sanitizeResource.test.ts
@@ -0,0 +1,15 @@
+import { sanitizeResource } from './sanitizeResource';
+
+describe('sanitizeResource', () => {
+  it('It keeps null values', () => {
+    expect(
+      sanitizeResource([
+        { name: 'vendor', value: 'Test Vendor' },
+        { name: 'brand', value: null },
+      ])
+    ).toEqual({
+      '0': { name: 'vendor', value: 'Test Vendor' },
+      '1': { name: 'brand', value: null },
+    });
+  });
+});
diff --git a/src/getResponseParser/sanitizeResource.ts b/src/getResponseParser/sanitizeResource.ts
@@ -1,14 +1,11 @@
-export const sanitizeResource = (data: any = {}) => {
-  const result: any = Object.keys(data).reduce((acc, key) => {
+export const sanitizeResource = (data: any = {}): object => {
+  const result = Object.keys(data).reduce((acc, key) => {
     if (key.startsWith('_')) {
       return acc;
     }
 
     const dataKey = data[key];
 
-    if (dataKey === null || dataKey === undefined) {
-      return acc;
-    }
     if (Array.isArray(dataKey)) {
       if (dataKey[0] && typeof dataKey[0] === 'object') {
         // if var is an array of reference objects with id properties
@@ -29,7 +26,11 @@ export const sanitizeResource = (data: any = {}) => {
       }
     }
 
-    if (typeof dataKey === 'object') {
+    if (
+      typeof dataKey === 'object' &&
+      dataKey !== null &&
+      dataKey !== undefined
+    ) {
       return {
         ...acc,
         ...(dataKey &&