@jsenv/snapshot 2.8.1 → 2.8.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/snapshot",
-  "version": "2.8.1",
+  "version": "2.8.3",
   "description": "Snapshot testing",
   "license": "MIT",
   "author": {
@@ -33,7 +33,7 @@
     "test": "node --conditions=development ./scripts/test.mjs"
   },
   "dependencies": {
-    "@jsenv/assert": "4.1.14",
+    "@jsenv/assert": "4.1.15",
     "@jsenv/ast": "6.2.14",
     "@jsenv/exception": "1.0.1",
     "@jsenv/filesystem": "4.9.9",

package/src/filesystem_snapshot.js

@@ -123,6 +123,8 @@ ${fileUrl}`);
         expect: fileContent,
         details: fileUrl,
         forceMultilineDiff: true,
+        MAX_CONTEXT_BEFORE_DIFF: 10,
+        MAX_CONTEXT_AFTER_DIFF: 10,
       });
     },
   };

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();