we don't even need to check the internal state of vitest

Rationale: if we need to regenerate the snapshots (vitest -u aor interactively), it's not to overcome the platform issues. For that you just have to rm all the files and start fresh…

Author: Fil

test/plot.js

@@ -106,22 +106,19 @@ const imageRe = /data:image\/png;base64,[^"]+/g;
 // This lets toMatchFileSnapshot's exact comparison pass despite platform
 // differences in PNG encoding.
 async function withImages(html, outfile) {
-  const updating = expect.getState().snapshotState["_updateSnapshot"] === "all";
-  if (!updating) {
-    try {
-      const expected = await readFile(outfile, "utf8");
-      const actualImages = Array.from(html.matchAll(imageRe), (m) => m[0]);
-      const expectedImages = Array.from(expected.matchAll(imageRe), (m) => m[0]);
-      if (actualImages.length !== expectedImages.length)
-        throw new Error(`Expected ${expectedImages.length} images, got ${actualImages.length}`);
-      for (let i = 0; i < actualImages.length; ++i) {
-        if (actualImages[i] !== expectedImages[i] && (await compareImage(actualImages[i], expectedImages[i]))) {
-          html = html.replace(actualImages[i], expectedImages[i]);
-        }
+  try {
+    const expected = await readFile(outfile, "utf8");
+    const actualImages = Array.from(html.matchAll(imageRe), (m) => m[0]);
+    const expectedImages = Array.from(expected.matchAll(imageRe), (m) => m[0]);
+    if (actualImages.length !== expectedImages.length)
+      throw new Error(`Expected ${expectedImages.length} images, got ${actualImages.length}`);
+    for (let i = 0; i < actualImages.length; ++i) {
+      if (actualImages[i] !== expectedImages[i] && (await compareImage(actualImages[i], expectedImages[i]))) {
+        html = html.replace(actualImages[i], expectedImages[i]);
       }
-    } catch (error) {
-      if (error.code !== "ENOENT") throw error;
     }
+  } catch (error) {
+    if (error.code !== "ENOENT") throw error;
   }
   return html;
 }