fix(queryCache): stop deepIncludes exception comparing null (#1403)

typeof null === 'object'

Co-authored-by: mike-shtil <mike.shtil@gmail.com>

src/core/tests/utils.test.tsx

@@ -68,6 +68,30 @@ describe('core/utils', () => {
       const b = [1, 2]
       expect(partialDeepEqual(a, b)).toEqual(true)
     })
+
+    it('should return `false` if a is null and b is not', () => {
+      const a = null
+      const b = { a: { b: 'b' }, c: 'c', d: [{ d: 'd ' }] }
+      expect(partialDeepEqual(a, b)).toEqual(false)
+    })
+
+    it('should return `false` if a contains null and b is not', () => {
+      const a = { a: null, c: 'c', d: [] }
+      const b = { a: { b: 'b' }, c: 'c', d: [{ d: 'd ' }] }
+      expect(partialDeepEqual(a, b)).toEqual(false)
+    })
+
+    it('should return `false` if b is null and a is not', () => {
+      const a = { a: { b: 'b' }, c: 'c', d: [] }
+      const b = null
+      expect(partialDeepEqual(a, b)).toEqual(false)
+    })
+
+    it('should return `false` if b contains null and a is not', () => {
+      const a = { a: { b: 'b' }, c: 'c', d: [] }
+      const b = { a: null, c: 'c', d: [{ d: 'd ' }] }
+      expect(partialDeepEqual(a, b)).toEqual(false)
+    })
   })
 
   describe('replaceEqualDeep', () => {

src/core/utils.ts

@@ -241,7 +241,7 @@ export function partialDeepEqual(a: any, b: any): boolean {
     return false
   }
 
-  if (typeof a === 'object' && typeof b === 'object') {
+  if (a && b && typeof a === 'object' && typeof b === 'object') {
     return !Object.keys(b).some(key => !partialDeepEqual(a[key], b[key]))
   }