fix(config): properly set filename when running config init

JaredReisinger · Jul 18, 2024 · d90c8f6 · d90c8f6
1 parent 98df4a8
commit d90c8f6
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 2 deletions.
diff --git a/src/cli.test.ts b/src/cli.test.ts
@@ -14,7 +14,7 @@ test('cli... runs the app?', async (t) => {
   process.argv = ['', '', '--help'];
   await t.notThrowsAsync(async () => {
     const result = await import('./cli.js');
-    t.log(result);
+    // t.log(result);
     t.truthy(result);
   });
 });
diff --git a/src/cli.ts b/src/cli.ts
@@ -80,7 +80,7 @@ async function loadConfig(): Promise<ConfigFile> {
   // dbg(0, 'checking for config', { filename });
   try {
     const data = await readFileAsync(filename);
-    const cfg = JSON.parse(data.toString());
+    const cfg = JSON.parse(data.toString()) as ConfigFile;
     cfg._filename = filename;
     return cfg;
   } catch (e) {

diff --git a/src/commands/config.test.ts b/src/commands/config.test.ts
@@ -220,6 +220,7 @@ test('Config.init() should return a Promise', async (t) => {
 test('Config.init() should write config from answers', async (t) => {
   const configMissing = new Config(
     {
+      _filename: 'CONFIG_FILE',
       _missing: true,
       hosts: {},
     },
@@ -267,6 +268,7 @@ test('Config.init() should write config from answers', async (t) => {
 
   t.is(write.callCount, 1);
   t.deepEqual(write.getCall(0).firstArg, {
+    _filename: 'CONFIG_FILE',
     hosts: {
       NAME: {
         key: 'MAGIC_KEY',

diff --git a/src/commands/config.ts b/src/commands/config.ts
@@ -134,6 +134,7 @@ hosts:${Object.entries(this.cfg.hosts)
 
     const { name, url, key, secret } = host;
     const cfg = {
+      _filename: this.cfg._filename,
       timezone,
       hosts: {
         [name]: { url, key, secret },