refactor(utils): fix getOrderIndependentHash()

Author: borodayev

src/__tests__/utils-test.js

@@ -106,19 +106,25 @@ describe('utils', () => {
 
   it('hashThisString()', () => {
     const hash1 = hashThisString('a');
+    const hashNull = hashThisString('[ type: null ; value: ""]');
+    const empty = hashThisString('');
     const hash2 = hashThisString('abc');
     const hash3 = hashThisString('abcsydghcsdagcyasjdcsdvcgsavdgcvsagdcbjhsdbbc');
 
     expect(hash1).toBe(97);
+    expect(empty).toBe(0);
+    expect(hashNull).toBe(1593002687);
     expect(hash2).toBe(96354);
     expect(hash3).toBe(25191889);
   });
 
   it('getOrderIndependentHash()', () => {
     const array = getOrderIndependentHash(['a', 'b']);
+    const ifNull = getOrderIndependentHash(null);
     const obj = getOrderIndependentHash({ a: 'a', b: 'b' });
     const string = getOrderIndependentHash('ab');
 
+    expect(ifNull).toBe(1933772593);
     expect(array).toBe(976844698);
     expect(obj).toBe(-2385456289);
     expect(string).toBe(35072500);

src/utils.js

@@ -65,7 +65,7 @@ export const realTypeOf = (obj: any): string => {
   if (Object.prototype.toString.call(obj) === '[object Date]') {
     return 'date';
   }
-  if (typeof obj.toString === 'function' && /^\/.*\//.test(obj.toString())) {
+  if (typeof obj?.toString === 'function' && /^\/.*\//.test(obj.toString())) {
     return 'regexp';
   }
   return 'object';
@@ -110,7 +110,8 @@ export const getOrderIndependentHash = (obj: any): number => {
     });
     return accum;
   }
-  const stringToHash = `[ type: ${type} ; value: ${obj.toString()}]`;
+
+  const stringToHash = `[ type: ${type} ; value: ${obj ? obj.toString() : 'hash'}]`;
   return accum + hashThisString(stringToHash);
 };