fix(flattener): fields null assignment is now properly handled (#796)

Author: SteveBunlon

src/services/flattener.js

@@ -249,7 +249,7 @@ module.exports = class Flattener {
     const flattenedRecord = {};
 
     Object.keys(record).forEach((attribute) => {
-      if (typeof record[attribute] === 'object') {
+      if (record[attribute] !== null && typeof record[attribute] === 'object') {
         const flattenedPath = (flattenComposedKey) ? `${flattenComposedKey}${FLATTEN_SEPARATOR}${attribute}` : attribute;
 
         if (flattenedFields.find((flattenedField) => flattenedField === flattenedPath)) {

test/tests/services/flattener.test.js

@@ -898,8 +898,7 @@ describe('service > Flattener', () => {
         engine: {
           horsePower: '78',
           identification: {
-            manufacturer: 'Renault'
-              + '',
+            manufacturer: 'Renault',
           },
         },
         name: 'Car',
@@ -918,6 +917,33 @@ describe('service > Flattener', () => {
         name: 'Car',
       });
     });
+
+    it('should correctly set values to null', () => {
+      expect.assertions(1);
+
+      const record = {
+        engine: {
+          horsePower: '78',
+          identification: {
+            manufacturer: null,
+          },
+        },
+        name: null,
+      };
+
+      const flattenedFields = [
+        `engine${FLATTEN_SEPARATOR}horsePower`,
+        `engine${FLATTEN_SEPARATOR}identification${FLATTEN_SEPARATOR}manufacturer`,
+      ];
+
+      const flattenedRecord = Flattener.flattenRecordDataForUpdates(record, null, flattenedFields);
+
+      expect(flattenedRecord).toStrictEqual({
+        'engine.horsePower': '78',
+        'engine.identification.manufacturer': null,
+        name: null,
+      });
+    });
   });
 
   describe('getFlattenedFieldsName', () => {