@jsenv/snapshot 2.11.37 → 2.13.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/snapshot",
-  "version": "2.11.37",
+  "version": "2.13.0",
   "description": "Snapshot testing",
   "license": "MIT",
   "author": {
@@ -30,14 +30,14 @@
     "/src/"
   ],
   "dependencies": {
-    "@jsenv/assert": "4.4.2",
-    "@jsenv/ast": "6.5.2",
-    "@jsenv/exception": "1.1.4",
-    "@jsenv/humanize": "1.2.8",
-    "@jsenv/filesystem": "4.13.5",
-    "@jsenv/terminal-recorder": "1.4.31",
-    "@jsenv/urls": "2.6.0",
-    "@jsenv/utils": "2.1.3",
+    "@jsenv/assert": "4.4.3",
+    "@jsenv/ast": "6.6.0",
+    "@jsenv/exception": "1.1.5",
+    "@jsenv/humanize": "1.3.0",
+    "@jsenv/filesystem": "4.14.0",
+    "@jsenv/terminal-recorder": "1.5.0",
+    "@jsenv/urls": "2.6.1",
+    "@jsenv/utils": "2.2.0",
     "ansi-regex": "6.1.0",
     "pixelmatch": "6.0.0",
     "prettier": "3.4.2",

package/src/filesystem_snapshot.js

@@ -367,7 +367,13 @@ ${extraUrls.join("\n")}`);
   try {
     const directoryItemArray = readdirSync(directoryUrl);
     for (const directoryItem of directoryItemArray) {
-      const directoryItemUrl = new URL(directoryItem, directoryUrl);
+      let directoryItemUrl;
+      if (directoryItem === "file:") {
+        // without this check it would create infinite loop
+        directoryItemUrl = new URL(`${directoryUrl}${directoryItem}`);
+      } else {
+        directoryItemUrl = new URL(directoryItem, directoryUrl);
+      }
       let directoryItemStat;
       try {
         directoryItemStat = readEntryStatSync(directoryItemUrl);

package/src/replace_fluctuating_values.js

@@ -156,6 +156,14 @@ export const replaceFluctuatingValues = (
     if (Buffer.isBuffer(value)) {
       return value;
     }
+    if (value instanceof RegExp) {
+      let regexpSource = value.source;
+      if (regexpSource === "(?:)") {
+        regexpSource = "";
+      }
+      regexpSource = `/${regexpSource}/${value.flags}`;
+      return regexpSource;
+    }
     const jsValueReplaced = replaceInObject(value, { replace: replaceThings });
     return JSON.stringify(jsValueReplaced, null, "  ");
   }
@@ -188,7 +196,10 @@ const replaceInObject = (object, { replace }) => {
       for (const keyToVisit of keysToVisit) {
         const nestedValue = value[keyToVisit];
         copy[keyToVisit] = deepCopy(nestedValue, {
-          shouldReplaceStrings: shouldReplaceStrings || keyToVisit === "os",
+          shouldReplaceStrings:
+            shouldReplaceStrings ||
+            keyToVisit === "os" ||
+            keyToVisit === "date",
           shouldReplaceNumbers:
             shouldReplaceNumbers ||
             keyToVisit === "timings" ||
@@ -227,12 +238,10 @@ const replaceHttpUrls = (source) => {
   );
   // we force "localhost" to "127.0.0.1"
   // in case the machine does not have localhost mapping
-  source = source.replace(
-    /(https?):\/\/localhost(:\d+)?/g,
-    (match, protocol) => {
-      return `${protocol}://127.0.0.1`;
-    },
-  );
+  // we also remove the port that can be dynamic
+  source = source.replace(/localhost(:\d+)?/g, () => {
+    return `127.0.0.1`;
+  });
   // we force [::1] to "127.0.0.1"
   // in case the machine does not have ipv6
   source = source.replace(/(https?):\/\/\[::1\](:\d+)?/g, (match, protocol) => {