test_runner: use internal date check function

This fixes an issue where a value could be `instanceof Date` without actually being a native date. Co-authored-by: Antoine du Hamel <duhamelantoine1995@gmail.com>
nodejs · Mar 1, 2024 · 15e3f5c · 15e3f5c
1 parent 9323496
commit 15e3f5c
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/lib/internal/test_runner/reporter/tap.js b/lib/internal/test_runner/reporter/tap.js
@@ -3,7 +3,7 @@ const {
   ArrayPrototypeForEach,
   ArrayPrototypeJoin,
   ArrayPrototypePush,
-  Date,
+  DatePrototypeToISOString,
   ObjectEntries,
   RegExpPrototypeSymbolReplace,
   RegExpPrototypeSymbolSplit,
@@ -164,9 +164,9 @@ function jsToYaml(indent, name, value, seen) {
 
   if (name != null) {
     result += prefix;
-    if (value instanceof Date) {
+    if (internalBinding('types').isDate(value)) {
       // YAML uses the ISO-8601 standard to express dates.
-      result += ' ' + value.toISOString();
+      result += ' ' + DatePrototypeToISOString(value);
     }
     result += '\n';
     propsIndent += '  ';