assert,util: handle invalid dates as equal in deep comparison

Invalid dates are now handled as equal in all deep comparisons.

Author: BridgeAR

benchmark/assert/deepequal-prims-and-objs-big-loop.js

@@ -22,6 +22,7 @@ const primValues = {
   'empty_object': {},
   'regexp': /abc/i,
   'date': new Date(),
+  'invalidDate': new Date('foo'),
 };
 
 const primValues2 = {
@@ -33,6 +34,7 @@ const primValues2 = {
   'empty_object': {},
   'regexp': /abc/i,
   'date': new Date(primValues.date),
+  'invalidDate': new Date('foo'),
 };
 
 const primValuesUnequal = {
@@ -47,6 +49,7 @@ const primValuesUnequal = {
   'empty_object': [],
   'regexp': /abc/g,
   'date': new Date(primValues.date.getTime() + 1),
+  'invalidDate': new Date(),
 };
 
 const bench = common.createBenchmark(main, {

doc/api/assert.md

@@ -556,6 +556,9 @@ An alias of [`assert.ok()`][].
 <!-- YAML
 added: v0.1.21
 changes:
+  - version: REPLACEME
+    pr-url: https://github.com/nodejs/node/pull/57627
+    description: Invalid dates are now considered equal.
   - version:
       - v22.2.0
       - v20.15.0
@@ -743,6 +746,9 @@ parameter is an instance of {Error} then it will be thrown instead of the
 <!-- YAML
 added: v1.2.0
 changes:
+  - version: REPLACEME
+    pr-url: https://github.com/nodejs/node/pull/57627
+    description: Invalid dates are now considered equal.
   - version:
     - v22.2.0
     - v20.15.0
@@ -2595,9 +2601,12 @@ added:
   - v23.4.0
   - v22.13.0
 changes:
- - version: REPLACEME
-   pr-url: https://github.com/nodejs/node/pull/57370
-   description: partialDeepStrictEqual is now Stable. Previously, it had been Experimental.
+  - version: REPLACEME
+    pr-url: https://github.com/nodejs/node/pull/57627
+    description: Invalid dates are now considered equal.
+  - version: REPLACEME
+    pr-url: https://github.com/nodejs/node/pull/57370
+    description: partialDeepStrictEqual is now Stable. Previously, it had been Experimental.
 -->
 
 > Stability: 2 - Stable

lib/internal/util/comparisons.js

@@ -230,10 +230,14 @@ function innerDeepEqual(val1, val2, mode, memos) {
   } else if (val1Tag === '[object Object]') {
     return keyCheck(val1, val2, mode, memos, kNoIterator);
   } else if (isDate(val1)) {
-    if (!isDate(val2) ||
-        DatePrototypeGetTime(val1) !== DatePrototypeGetTime(val2)) {
+    if (!isDate(val2)) {
       return false;
     }
+    const time1 = DatePrototypeGetTime(val1);
+    const time2 = DatePrototypeGetTime(val2);
+    if (time1 !== time2) {
+      return NumberIsNaN(time1) && NumberIsNaN(time2);
+    }
   } else if (isRegExp(val1)) {
     if (!isRegExp(val2) || !areSimilarRegExps(val1, val2)) {
       return false;

test/parallel/test-assert-deep.js

@@ -740,6 +740,8 @@ test('Additional tests', () => {
 
   assertNotDeepOrStrict(new Date(), new Date(2000, 3, 14));
 
+  assertDeepAndStrictEqual(new Date('foo'), new Date('bar'));
+
   assertDeepAndStrictEqual(/a/, /a/);
   assertDeepAndStrictEqual(/a/g, /a/g);
   assertDeepAndStrictEqual(/a/i, /a/i);