@jsenv/snapshot 2.2.4 → 2.2.6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/snapshot",
-  "version": "2.2.4",
+  "version": "2.2.6",
   "description": "Snapshot testing",
   "license": "MIT",
   "author": {
@@ -35,9 +35,9 @@
   },
   "dependencies": {
     "@jsenv/assert": "4.1.5",
-    "@jsenv/ast": "6.2.5",
-    "@jsenv/filesystem": "4.9.1",
-    "@jsenv/urls": "2.3.1",
+    "@jsenv/ast": "6.2.6",
+    "@jsenv/filesystem": "4.9.2",
+    "@jsenv/urls": "2.3.2",
     "@jsenv/utils": "2.1.1",
     "pixelmatch": "6.0.0",
     "prettier": "3.3.3",

package/src/filesystem_snapshot.js

@@ -8,7 +8,11 @@ import {
   writeFileSync,
 } from "@jsenv/filesystem";
 import { URL_META } from "@jsenv/url-meta";
-import { urlToFilename, urlToRelativeUrl } from "@jsenv/urls";
+import {
+  ensurePathnameTrailingSlash,
+  urlToFilename,
+  urlToRelativeUrl,
+} from "@jsenv/urls";
 import { CONTENT_TYPE } from "@jsenv/utils/src/content_type/content_type.js";
 import { readdirSync, readFileSync, statSync } from "node:fs";
 import { fileURLToPath } from "node:url";
@@ -357,20 +361,21 @@ ${extraUrls.join("\n")}`);
       }
       const relativeUrl = urlToRelativeUrl(directoryItemUrl, directoryUrl);
       if (directoryItemStat.isDirectory()) {
-        if (!shouldIncludeFile(directoryUrl)) {
-          continue;
-        }
-        if (!shouldVisitDirectory(directoryUrl)) {
+        ensurePathnameTrailingSlash(directoryItemUrl);
+        if (!shouldVisitDirectory(directoryItemUrl)) {
           continue;
         }
         contentSnapshotNaturalOrder[relativeUrl] = createDirectorySnapshot(
-          new URL(`${directoryItemUrl}/`),
+          directoryItemUrl,
           {
             shouldVisitDirectory,
             shouldIncludeFile,
             clean,
           },
         );
+        if (clean) {
+          removeDirectorySync(directoryItemUrl);
+        }
         continue;
       }
       if (!shouldIncludeFile(directoryItemUrl)) {

package/src/function_snapshot.js

@@ -4,7 +4,7 @@ import {
   removeFileSync,
   writeFileSync,
 } from "@jsenv/filesystem";
-import { urlToRelativeUrl } from "@jsenv/urls/src/url_to_relative_url.js";
+import { urlToFilename, urlToRelativeUrl } from "@jsenv/urls";
 import { takeDirectorySnapshot } from "./filesystem_snapshot.js";
 import { replaceFluctuatingValues } from "./replace_fluctuating_values.js";
 
@@ -17,7 +17,7 @@ export const snapshotFunctionSideEffects = (
     captureConsole = true,
     filesystemEffects,
     filesystemEffectsInline,
-    restoreFilesystem,
+    restoreFilesystem = true,
   } = {},
 ) => {
   const sideEffectDirectoryUrl = new URL(
@@ -27,10 +27,8 @@ export const snapshotFunctionSideEffects = (
   const sideEffectDirectorySnapshot = takeDirectorySnapshot(
     sideEffectDirectoryUrl,
   );
-  const sideEffectFileUrl = new URL(
-    "./side_effect.txt",
-    sideEffectDirectoryUrl,
-  );
+  const sideEffectFilename = `${urlToFilename(sideEffectDirectoryUrl)}_side_effects.txt`;
+  const sideEffectFileUrl = new URL(sideEffectFilename, sideEffectDirectoryUrl);
   const sideEffects = [];
   const finallyCallbackSet = new Set();
   const onError = (e) => {