@jsenv/snapshot 2.6.7 → 2.6.9

package/package.json

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

package/src/filesystem_snapshot.js

@@ -170,27 +170,35 @@ export const takeDirectorySnapshot = (
   directoryUrl = new URL(directoryUrl);
   const associations = URL_META.resolveAssociations(
     {
-      included: pattern,
+      action: pattern,
     },
     directoryUrl,
   );
-  const predicate = ({ included }) => included;
-  const shouldVisitDirectory = (url) =>
-    URL_META.urlChildMayMatch({
+  const shouldVisitDirectory = (url) => {
+    return URL_META.urlChildMayMatch({
       url,
       associations,
-      predicate,
+      predicate: (meta) => Boolean(meta.action),
     });
-  const shouldIncludeFile = (url) =>
-    predicate(
-      URL_META.applyAssociations({
-        url,
-        associations,
-      }),
-    );
+  };
+  const shouldIncludeFile = (url) => {
+    const meta = URL_META.applyAssociations({
+      url,
+      associations,
+    });
+    return meta.action === true || meta.action === "presence_only";
+  };
+  const shouldCompareFile = (url) => {
+    const meta = URL_META.applyAssociations({
+      url,
+      associations,
+    });
+    return meta.action === true || meta.action === "compare";
+  };
   const directorySnapshot = createDirectorySnapshot(directoryUrl, {
     shouldVisitDirectory,
     shouldIncludeFile,
+    shouldCompareFile,
     clean: true,
   });
   return {
@@ -199,6 +207,7 @@ export const takeDirectorySnapshot = (
       const nextDirectorySnapshot = createDirectorySnapshot(directoryUrl, {
         shouldVisitDirectory,
         shouldIncludeFile,
+        shouldCompareFile,
       });
       directorySnapshot.compare(nextDirectorySnapshot, { throwWhenDiff });
     },
@@ -226,7 +235,7 @@ export const takeDirectorySnapshot = (
 };
 const createDirectorySnapshot = (
   directoryUrl,
-  { shouldVisitDirectory, shouldIncludeFile, clean },
+  { shouldVisitDirectory, shouldIncludeFile, shouldCompareFile, clean },
 ) => {
   const directorySnapshot = {
     type: "directory",
@@ -311,6 +320,9 @@ ${extraUrls.join("\n")}`);
       // content
       {
         for (const relativeUrl of relativeUrls) {
+          if (!shouldCompareFile(new URL(relativeUrl, directoryUrl))) {
+            continue;
+          }
           const snapshot = directoryContentSnapshot[relativeUrl];
           const nextSnapshot = nextDirectoryContentSnapshot[relativeUrl];
           if (nextSnapshot) {
@@ -370,6 +382,7 @@ ${extraUrls.join("\n")}`);
           {
             shouldVisitDirectory,
             shouldIncludeFile,
+            shouldCompareFile,
             clean,
           },
         );

package/src/get_caller_location.js

@@ -9,6 +9,9 @@ export const getCallerLocation = (callIndex = 1) => {
   const { stack } = new Error();
   Error.prepareStackTrace = prepareStackTrace;
   const callerCallsite = stack[callIndex];
+  if (!callerCallsite) {
+    return null;
+  }
   const fileName = callerCallsite.getFileName();
   const testCallSite = {
     url:

package/src/side_effects/filesystem/filesystem_side_effects.js

@@ -97,6 +97,27 @@ export const filesystemSideEffects = (
         }
         return null;
       });
+      addSkippableHandler((sideEffect) => {
+        // on llinux writing to a file with the same content
+        // is ignored somehow
+        // let's make this consistent on other platforms
+        if (sideEffect.code === "write_file") {
+          return (nextSideEffect, { skip, stop }) => {
+            if (
+              nextSideEffect.code === "write_file" &&
+              nextSideEffect.value.url === sideEffect.value.url &&
+              !Buffer.compare(
+                nextSideEffect.value.buffer,
+                sideEffect.value.buffer,
+              )
+            ) {
+              skip();
+            }
+            stop();
+          };
+        }
+        return null;
+      });
 
       addFinallyCallback((sideEffects) => {
         // gather all file side effect next to each other

package/src/side_effects/snapshot_tests.js

@@ -78,7 +78,12 @@ export const snapshotTests = async (
       `./${asValidFilename(scenario)}/`,
       snapshotFileUrl,
     );
-    const outDirectorySnapshot = takeDirectorySnapshot(outDirectoryUrl);
+    const outDirectorySnapshot = takeDirectorySnapshot(outDirectoryUrl, {
+      pattern: {
+        "**/*": true,
+        "**/*.svg": "presence_only",
+      },
+    });
     const sideEffectsMarkdown = renderSideEffects(sideEffects, {
       sideEffectFileUrl: snapshotFileUrl,
       outDirectoryUrl,