@jsenv/snapshot 2.8.0 → 2.8.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/snapshot",
-  "version": "2.8.0",
+  "version": "2.8.2",
   "description": "Snapshot testing",
   "license": "MIT",
   "author": {
@@ -34,7 +34,7 @@
   },
   "dependencies": {
     "@jsenv/assert": "4.1.14",
-    "@jsenv/ast": "6.2.13",
+    "@jsenv/ast": "6.2.14",
     "@jsenv/exception": "1.0.1",
     "@jsenv/filesystem": "4.9.9",
     "@jsenv/terminal-recorder": "1.4.3",

package/src/filesystem_well_known_values.js

@@ -58,6 +58,7 @@ export const createWellKnown = (name, replacement = name) => {
 
 export const createReplaceFilesystemWellKnownValues = ({
   rootDirectoryUrl,
+  localhostUrl,
   // for unit tests
   isWindows = process.platform === "win32",
   ancestorPackagesDisabled,
@@ -126,9 +127,32 @@ export const createReplaceFilesystemWellKnownValues = ({
       }
     };
   };
+  const addWellKnownUrl = (url, replacement) => {
+    const wellKnownUrl = {
+      url,
+      replace: (string) => {
+        return string.replaceAll(url, replacement);
+      },
+    };
+    wellKownUrlArray.push(wellKnownUrl);
+    return () => {
+      const urlIndex = wellKownUrlArray.indexOf(wellKnownUrl);
+      if (urlIndex > -1) {
+        wellKownUrlArray.splice(urlIndex, 1);
+      }
+    };
+  };
   if (rootDirectoryUrl) {
     addWellKnownFileUrl(rootDirectoryUrl, WELL_KNOWN_ROOT);
   }
+  if (localhostUrl) {
+    addWellKnownUrl(
+      localhostUrl,
+      localhostUrl.startsWith("https")
+        ? "https://localhost"
+        : "http://localhost",
+    );
+  }
   /*
    * When running code inside a node project ancestor packages
    * should make things super predictible because

package/src/replace_fluctuating_values.js

@@ -21,11 +21,13 @@ export const replaceFluctuatingValues = (
   {
     stringType,
     rootDirectoryUrl,
+    localhostUrl,
     fileUrl,
     preserveAnsi,
     // for unit test
     replaceFilesystemWellKnownValues = createReplaceFilesystemWellKnownValues({
       rootDirectoryUrl,
+      localhostUrl,
     }),
   } = {},
 ) => {

package/src/side_effects/filesystem/spy_filesystem_calls.js

@@ -53,27 +53,31 @@ export const spyFilesystemCalls = (
           : stateBefore.mtimeMs === stateAfter.mtimeMs
             ? ""
             : "mtime_modified";
-    if (reason) {
-      if (undoFilesystemSideEffects && !fileRestoreMap.has(fileUrl)) {
-        if (stateBefore.found) {
-          fileRestoreMap.set(fileUrl, () => {
-            writeFileSync(fileUrl, stateBefore.buffer);
-          });
-        } else {
-          fileRestoreMap.set(fileUrl, () => {
-            removeFileSync(fileUrl, { allowUseless: true });
-          });
-        }
-      }
-      if (shouldReport(fileUrl)) {
-        onWriteFile(fileUrl, stateAfter.buffer, reason);
-        return;
+    if (!reason) {
+      // file is exactly the same
+      // function did not have any effect on the file
+      return;
+    }
+    if (!shouldReport(fileUrl)) {
+      return;
+    }
+    if (undoFilesystemSideEffects && !fileRestoreMap.has(fileUrl)) {
+      if (stateBefore.found) {
+        fileRestoreMap.set(fileUrl, () => {
+          writeFileSync(fileUrl, stateBefore.buffer);
+        });
+      } else {
+        fileRestoreMap.set(fileUrl, () => {
+          removeFileSync(fileUrl, { allowUseless: true });
+        });
       }
     }
-    // file is exactly the same
-    // function did not have any effect on the file
+    onWriteFile(fileUrl, stateAfter.buffer, reason);
   };
   const onWriteDirectoryDone = (directoryUrl) => {
+    if (!shouldReport(directoryUrl)) {
+      return;
+    }
     if (undoFilesystemSideEffects && !fileRestoreMap.has(directoryUrl)) {
       fileRestoreMap.set(directoryUrl, () => {
         removeDirectorySync(directoryUrl, {
@@ -82,9 +86,7 @@ export const spyFilesystemCalls = (
         });
       });
     }
-    if (shouldReport(directoryUrl)) {
-      onWriteDirectory(directoryUrl);
-    }
+    onWriteDirectory(directoryUrl);
   };
   const restoreCallbackSet = new Set();