updating the fixture logic. returning the new more complete fixture object and using a generator pattern to avoid calling filter(Boolean) over the array.

Author: Janther

tests/config/get-fixtures.js

@@ -1,52 +1,64 @@
 import fs from "node:fs";
 import path from "node:path";
 import { FORMAT_SCRIPT_FILENAME } from "./constants.js";
+import visualizeEndOfLine from "./utils/visualize-end-of-line.js";
 
-function getFixtures(context) {
-  return [...getFiles(context), ...getSnippets(context)];
+function* getFixtures(context) {
+  yield* getFiles(context);
+  yield* getSnippets(context);
 }
 
-function getFiles({ dirname }) {
-  return fs
-    .readdirSync(dirname, { withFileTypes: true })
-    .map((file) => {
-      const basename = file.name;
-      const filename = path.join(dirname, basename);
-      if (
-        path.extname(basename) === ".snap" ||
-        !file.isFile() ||
-        basename[0] === "." ||
-        basename === FORMAT_SCRIPT_FILENAME ||
-        // VSCode creates this file sometime https://github.com/microsoft/vscode/issues/105191
-        basename === "debug.log"
-      ) {
-        return;
-      }
-
-      const text = fs.readFileSync(filename, "utf8");
-
-      return {
-        name: basename,
-        filename,
-        code: text,
-      };
-    })
-    .filter(Boolean);
+function* getFiles(context) {
+  const { dirname } = context;
+  for (const file of fs.readdirSync(dirname, { withFileTypes: true })) {
+    const filename = file.name;
+    const filepath = path.join(dirname, filename);
+    if (
+      path.extname(filename) === ".snap" ||
+      !file.isFile() ||
+      filename[0] === "." ||
+      filename === FORMAT_SCRIPT_FILENAME ||
+      // VSCode creates this file sometime https://github.com/microsoft/vscode/issues/105191
+      filename === "debug.log"
+    ) {
+      continue;
+    }
+
+    const text = fs.readFileSync(filepath, "utf8");
+
+    yield {
+      context,
+      name: filename,
+      filename,
+      filepath,
+      code: text,
+    };
+  }
 }
 
-function getSnippets({ snippets }) {
-  return snippets.map((test, index) => {
-    test = typeof test === "string" ? { code: test } : test;
+function* getSnippets(context) {
+  for (const [index, snippet] of context.snippets.entries()) {
+    const testCase =
+      typeof snippet === "string"
+        ? { code: snippet, context }
+        : { ...snippet, context };
 
-    if (typeof test.code !== "string") {
-      throw Object.assign(new Error("Invalid test"), { test });
+    if (typeof testCase.code !== "string") {
+      throw Object.assign(new Error("Invalid test"), { snippet });
     }
 
-    return {
-      ...test,
-      name: `snippet: ${test.name || `#${index}`}`,
-    };
-  });
+    if (typeof testCase.output === "string") {
+      testCase.output = visualizeEndOfLine(testCase.output);
+    }
+
+    testCase.name = `snippet: ${testCase.name || `#${index}`}`;
+
+    if (typeof testCase.filename === "string") {
+      testCase.filepath = path.join(context.dirname, testCase.filename);
+    }
+
+    yield testCase;
+  }
 }
 
 export { getFixtures };

tests/config/run-format-test.js

@@ -12,7 +12,7 @@ import { runTest } from "./run-test.js";
 import { shouldThrowOnFormat } from "./utilities.js";
 
 function runFormatTest(rawFixtures, explicitParsers, rawOptions) {
-  let { importMeta, snippets = [] } = rawFixtures.importMeta
+  const { importMeta, snippets = [] } = rawFixtures.importMeta
     ? rawFixtures
     : { importMeta: rawFixtures };