tests for custom config file location

chris48s · Oct 23, 2024 · f9546ff · f9546ff
1 parent 921c039
commit f9546ff
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 9 deletions.
diff --git a/src/bootstrap.spec.js b/src/bootstrap.spec.js
@@ -160,10 +160,7 @@ describe("getConfig", function () {
     const { config } = await bootstrap(
       ["node", "index.js", "infile.json"],
       undefined,
-      {
-        searchPlaces: ["./testfiles/does-not-exist.json"],
-        cache: false,
-      },
+      { cache: false },
     );
     assert.equal(config.ignoreErrors, false);
     assert.equal(config.cacheTtl, 600);
@@ -176,9 +173,22 @@ describe("getConfig", function () {
     assert(logContainsInfo("No config file found"));
   });
 
+  it("should throw if V8R_CONFIG_FILE does not exist", async function () {
+    process.env.V8R_CONFIG_FILE = "./testfiles/does-not-exist.json";
+    await assert.rejects(
+      bootstrap(["node", "index.js", "infile.json"], undefined, {
+        cache: false,
+      }),
+      {
+        name: "Error",
+        message: "File ./testfiles/does-not-exist.json does not exist.",
+      },
+    );
+  });
+
   it("should read options from config file if available", async function () {
+    process.env.V8R_CONFIG_FILE = "./testfiles/configs/config.json";
     const { config } = await bootstrap(["node", "index.js"], undefined, {
-      searchPlaces: ["./testfiles/configs/config.json"],
       cache: false,
     });
     assert.equal(config.ignoreErrors, true);
@@ -208,6 +218,7 @@ describe("getConfig", function () {
   });
 
   it("should override options from config file with args if specified", async function () {
+    process.env.V8R_CONFIG_FILE = "./testfiles/configs/config.json";
     const { config } = await bootstrap(
       [
         "node",
@@ -219,10 +230,7 @@ describe("getConfig", function () {
         "-vv",
       ],
       undefined,
-      {
-        searchPlaces: ["./testfiles/configs/config.json"],
-        cache: false,
-      },
+      { cache: false },
     );
     assert.deepStrictEqual(config.patterns, ["infile.json"]);
     assert.equal(config.ignoreErrors, true);

diff --git a/src/test-helpers.js b/src/test-helpers.js
@@ -4,13 +4,15 @@ import logger from "./logger.js";
 const origWriteOut = logger.writeOut;
 const origWriteErr = logger.writeErr;
 const testCacheName = process.env.V8R_CACHE_NAME;
+const env = process.env;
 
 function setUp() {
   flatCache.clearCacheById(testCacheName);
   logger.resetStdout();
   logger.resetStderr();
   logger.writeOut = function () {};
   logger.writeErr = function () {};
+  process.env = { ...env };
 }
 
 function tearDown() {
@@ -19,6 +21,7 @@ function tearDown() {
   logger.resetStderr();
   logger.writeOut = origWriteOut;
   logger.writeErr = origWriteErr;
+  process.env = env;
 }
 
 function isString(el) {