TEST_STANDALONE should be a constant

Author: Janther

tests/config/constants.js

@@ -10,7 +10,7 @@ export const FORMAT_TEST_DIRECTORY = normalizeDirectory(
   path.join(__dirname, "../format/"),
 );
 
-export const { FULL_TEST } = process.env;
+export const { FULL_TEST, TEST_STANDALONE } = process.env;
 export const BOM = "\uFEFF";
 
 export const CURSOR_PLACEHOLDER = "<|>";

tests/config/get-create-parser.js

@@ -1,5 +1,7 @@
+import { TEST_STANDALONE } from "./constants.js";
+
 function getCreateParserInternal() {
-  const entry = process.env.TEST_STANDALONE
+  const entry = TEST_STANDALONE
     ? "../../dist/create-parser.js"
     : "../../src/slang-utils/create-parser.js";
 

tests/config/get-plugins.js

@@ -1,11 +1,12 @@
 import path from "node:path";
 import createEsmUtils from "esm-utils";
 import getPrettier from "./get-prettier.js";
+import { TEST_STANDALONE } from "./constants.js";
 
 const { __dirname } = createEsmUtils(import.meta);
 
 // populate the root object for the standalone in node
-if (process.env.TEST_STANDALONE) {
+if (TEST_STANDALONE) {
   const root =
     typeof globalThis !== "undefined"
       ? globalThis
@@ -19,7 +20,7 @@ if (process.env.TEST_STANDALONE) {
 
 function getPluginsInternal() {
   return Promise.all(
-    process.env.TEST_STANDALONE
+    TEST_STANDALONE
       ? [
           import("prettier/plugins/babel"),
           import("prettier/plugins/estree"),

tests/config/get-prettier.js

@@ -1,7 +1,7 @@
+import { TEST_STANDALONE } from "./constants.js";
+
 function getPrettierInternal() {
-  const entry = process.env.TEST_STANDALONE
-    ? "prettier/standalone"
-    : "prettier";
+  const entry = TEST_STANDALONE ? "prettier/standalone" : "prettier";
 
   return import(entry).then((module) => module.default);
 }

tests/config/get-variant-coverage.js

@@ -1,5 +1,7 @@
+import { TEST_STANDALONE } from "./constants.js";
+
 function getVariantCoverageInternal() {
-  const entry = process.env.TEST_STANDALONE
+  const entry = TEST_STANDALONE
     ? "../../dist/variant-coverage.js"
     : "../../variant-coverage/index.js";
 

tests/config/run-test.js

@@ -69,20 +69,30 @@ async function testFixture(fixture) {
         test: { run: testBom.run },
         skip: (testCase) => !FULL_TEST || testCase !== testCaseForSnapshot,
       },
+      // The following cases only need run if the parser is Slang
       {
         name: "ANTLR format",
         test: { run: testAntlrFormat.run },
-        skip: (testCase) => !FULL_TEST || testCase !== testCaseForSnapshot,
+        skip: (testCase) =>
+          !FULL_TEST ||
+          testCase !== testCaseForSnapshot ||
+          testCase.parser !== "slang",
       },
       {
         name: "bytecode comparison",
         test: { run: testBytecodeCompare.run },
-        skip: (testCase) => !FULL_TEST || testCase !== testCaseForSnapshot,
+        skip: (testCase) =>
+          !FULL_TEST ||
+          testCase !== testCaseForSnapshot ||
+          testCase.parser !== "slang",
       },
       {
         name: "variant coverage",
         test: { run: testVariantCoverage.run },
-        skip: (testCase) => !FULL_TEST || testCase !== testCaseForSnapshot,
+        skip: (testCase) =>
+          !FULL_TEST ||
+          testCase !== testCaseForSnapshot ||
+          testCase.parser !== "slang",
       },
     ]) {
       for (const testCase of testCases) {