@jsenv/snapshot 2.2.3 → 2.2.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/snapshot",
-  "version": "2.2.3",
+  "version": "2.2.5",
   "description": "Snapshot testing",
   "license": "MIT",
   "author": {

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

@@ -16,6 +16,7 @@ export const snapshotFunctionSideEffects = (
     rootDirectoryUrl = new URL("./", fnFileUrl),
     captureConsole = true,
     filesystemEffects,
+    filesystemEffectsInline,
     restoreFilesystem,
   } = {},
 ) => {
@@ -50,7 +51,10 @@ export const snapshotFunctionSideEffects = (
     }
     writeFileSync(
       sideEffectFileUrl,
-      stringifySideEffects(sideEffects, { rootDirectoryUrl }),
+      stringifySideEffects(sideEffects, {
+        rootDirectoryUrl,
+        filesystemEffectsInline,
+      }),
     );
     sideEffectDirectorySnapshot.compare();
   };
@@ -89,6 +93,7 @@ export const snapshotFunctionSideEffects = (
         if (atStartState.found && !nowState.found) {
           onFileSystemSideEffect({
             type: `remove file "${relativeUrl}"`,
+            value: atStartState.content,
           });
           if (restoreFilesystem) {
             writeFileSync(from, atStartState.content);
@@ -104,10 +109,18 @@ export const snapshotFunctionSideEffects = (
           atStartState.content !== nowState.content ||
           atStartState.mtimeMs !== nowState.mtimeMs
         ) {
-          writeFileSync(toUrl, nowState.content);
-          onFileSystemSideEffect({
-            type: `write file "./fs/${relativeUrl}"`,
-          });
+          if (filesystemEffectsInline) {
+            onFileSystemSideEffect({
+              type: `write file "${relativeUrl}"`,
+              value: nowState.content,
+            });
+          } else {
+            writeFileSync(toUrl, nowState.content);
+            onFileSystemSideEffect({
+              type: `write file "${relativeUrl}" (see ./fs/${relativeUrl})`,
+              value: nowState.content,
+            });
+          }
           if (restoreFilesystem) {
             if (atStartState.found) {
               if (atStartState.content !== nowState.content) {
@@ -152,7 +165,10 @@ export const snapshotFunctionSideEffects = (
   }
 };
 
-const stringifySideEffects = (sideEffects, { rootDirectoryUrl }) => {
+const stringifySideEffects = (
+  sideEffects,
+  { rootDirectoryUrl, filesystemEffectsInline },
+) => {
   let string = "";
   let index = 0;
   for (const sideEffect of sideEffects) {
@@ -172,6 +188,10 @@ const stringifySideEffects = (sideEffects, { rootDirectoryUrl }) => {
       sideEffect.type.startsWith("remove file") ||
       sideEffect.type.startsWith("write file")
     ) {
+      if (filesystemEffectsInline) {
+        string += "\n";
+        string += value;
+      }
     } else if (sideEffect.type === "throw") {
       value = replaceFluctuatingValues(value.stack, {
         stringType: "error",