Add failing tests for #58

#58 "Comparing object with "hasOwnProperty" keys fails"
mattphillips · Jan 25, 2022 · ca791b3 · ca791b3
1 parent ff1235e
commit ca791b3
Show file tree

Hide file tree

Showing 3 changed files with 24 additions and 0 deletions.
diff --git a/src/added/index.test.js b/src/added/index.test.js
@@ -105,5 +105,13 @@ describe('.addedDiff', () => {
         expect(addedDiff(lhs, rhs)).toEqual({ date: new Date('2016') });
       });
     });
+
+    describe('object with non-function hasOwnProperty property', () => {
+      test('can represent the property in diff despite it being part of Object.prototype', () => {
+        const lhs = {};
+        const rhs = { hasOwnProperty: true };
+        expect(addedDiff(lhs, rhs)).toEqual({ hasOwnProperty: true });
+      });
+    });
   });
 });
diff --git a/src/deleted/index.test.js b/src/deleted/index.test.js
@@ -107,5 +107,13 @@ describe('.deletedDiff', () => {
         expect(deletedDiff({ date: new Date('2016') }, rhs)).toEqual({ date: undefined });
       });
     });
+
+    describe('object with non-function hasOwnProperty property', () => {
+      test('can represent the property in diff despite it being part of Object.prototype', () => {
+        const lhs = { hasOwnProperty: true };
+        const rhs = {};
+        expect(deletedDiff(lhs, rhs)).toEqual({ hasOwnProperty: undefined });
+      });
+    });
   });
 });
diff --git a/src/updated/index.test.js b/src/updated/index.test.js
@@ -147,5 +147,13 @@ describe('.updatedDiff', () => {
         expect(updatedDiff(lhs, rhs)).toEqual({ date: new Date('2017') });
       });
     });
+
+    describe('object with non-function hasOwnProperty property', () => {
+      test('can represent the property in diff despite it being part of Object.prototype', () => {
+        const lhs = { hasOwnProperty: false };
+        const rhs = { hasOwnProperty: true };
+        expect(updatedDiff(lhs, rhs)).toEqual({ hasOwnProperty: true });
+      });
+    });
   });
 });