Merge pull request #22 from MapColonies/fix-null-value-freeze

fix: support null valued config attributes

src/utils/helpers.ts

@@ -2,7 +2,7 @@ export function deepFreeze<T>(obj: T): void {
   const propNames = Object.getOwnPropertyNames(obj);
   for (const name of propNames) {
     const value = obj[name as keyof T];
-    if (typeof value === 'object') {
+    if (typeof value === 'object' && value !== null) {
       deepFreeze(value);
     }
   }

tests/helpers.spec.ts

@@ -0,0 +1,43 @@
+import 'jest-extended';
+import { deepFreeze } from '../src/utils/helpers';
+
+describe('helpers', () => {
+  describe('#deepFreeze', () => {
+    it('should return frozen object', () => {
+      const data = {
+        name: 'I am parent',
+        child: {
+          name: 'I am child',
+        },
+      };
+
+      deepFreeze(data);
+      expect(data).toBeFrozen();
+    });
+
+    it('should freeze the input object that include null valued property without error', () => {
+      const data = {
+        name: null,
+        child: {
+          name: 'i am child',
+        },
+      };
+
+      deepFreeze(data);
+      expect(data).toBeFrozen();
+    });
+
+    it('should freeze the input object include nested property', () => {
+      const data = {
+        name: 'I am parent',
+        child: {
+          name: 'I am child',
+        },
+      };
+
+      deepFreeze(data);
+      const child = data.child;
+      expect(child).toBeFrozen();
+    });
+  });
+});