fix: custom config path (#119)

* fix: custom config path

* lint fix

* Update src/hooks/post.ts

Co-authored-by: Copilot <[email protected]>

---------

Co-authored-by: Copilot <[email protected]>

Author: justinmchase

src/context.ts

@@ -4,6 +4,11 @@ import { FormatKind } from "./util/variant.ts";
 
 export interface IContext {
   output?: string;
+
+  // For an unknown reason, this one particular arg is coming through as 'c' instead of 'config'
+  // All of the other options are coming through as their full names.
+  // Therefore, we need to support both 'c' and 'config' for now.
+  c?: string;
   config?: string;
   githubDir: string;
   hooks: {

src/hooks/post.test.ts

@@ -1,5 +1,5 @@
 import { parse } from "../../deps/semver.ts";
-import { resolvesNext, stub } from "../../deps/std.ts";
+import { assertEquals, resolvesNext, stub } from "../../deps/std.ts";
 import { YAML } from "../../deps/yaml.ts";
 import { IContext } from "../context.ts";
 import { postVersionHook } from "./post.ts";
@@ -13,27 +13,73 @@ Deno.test("yml or yaml", async () => {
       regexp: async () => await undefined,
     },
   };
-  stub(
-    Deno,
-    "stat",
-    resolvesNext<Deno.FileInfo>([
-      Object.assign(new Deno.errors.NotFound("not found")), // version.yml, nah
-      {
-        isFile: true,
-      } as Deno.FileInfo, // version.yaml, yah
-    ]),
-  );
-  stub(
-    Deno,
-    "readTextFile",
-    resolvesNext([
-      "1.0.0",
-      YAML.stringify({
-        on: { post: [{ kind: "patch", file: "test/example.csproj" }] },
-      }),
-    ]),
-  );
-  stub(context.hooks, "patch");
-  stub(context.hooks, "replace");
-  await postVersionHook(context, parse("1.0.0"), parse("1.2.3"));
+  const stubs = [
+    stub(
+      Deno,
+      "stat",
+      resolvesNext<Deno.FileInfo>([
+        Object.assign(new Deno.errors.NotFound("not found")), // version.yml, nah
+        {
+          isFile: true,
+        } as Deno.FileInfo, // version.yaml, yah
+      ]),
+    ),
+    stub(
+      Deno,
+      "readTextFile",
+      resolvesNext([
+        "1.0.0",
+        YAML.stringify({
+          on: { post: [{ kind: "patch", file: "test/example.csproj" }] },
+        }),
+      ]),
+    ),
+    stub(context.hooks, "patch"),
+    stub(context.hooks, "replace"),
+  ];
+  try {
+    await postVersionHook(context, parse("1.0.0"), parse("1.2.3"));
+  } finally {
+    stubs.forEach((s) => s.restore());
+  }
+});
+
+Deno.test("custom config", async () => {
+  const context: IContext = {
+    config: ".github/version-test.yml",
+    githubDir: ".github",
+    hooks: {
+      patch: async () => await undefined,
+      replace: async () => await undefined,
+      regexp: async () => await undefined,
+    },
+  };
+  let configPath: string | URL = "";
+  const stubs = [
+    stub(
+      Deno,
+      "stat",
+      resolvesNext<Deno.FileInfo>([
+        { isFile: true } as Deno.FileInfo,
+      ]),
+    ),
+    stub(
+      Deno,
+      "readTextFile",
+      async (path, _opts) => {
+        configPath = path;
+        return await YAML.stringify({
+          on: { post: [] },
+        });
+      },
+    ),
+    stub(context.hooks, "patch"),
+    stub(context.hooks, "replace"),
+  ];
+  try {
+    await postVersionHook(context, parse("1.0.0"), parse("1.2.3"));
+    assertEquals(configPath, ".github/version-test.yml");
+  } finally {
+    stubs.forEach((s) => s.restore());
+  }
 });

src/hooks/post.ts

@@ -57,8 +57,9 @@ export async function postVersionHook(
 }
 
 async function getVersionConfig(context: IContext) {
-  const { config, githubDir } = context;
-  const paths = config ? [config] : [
+  const { c: configShorthand, config, githubDir } = context;
+  const configPath = config ?? configShorthand;
+  const paths = configPath ? [configPath] : [
     [githubDir, "version.yml"].filter((p) => p).join("/"),
     [githubDir, "version.yaml"].filter((p) => p).join("/"),
   ];

test/node/.github/version-test.yml

@@ -0,0 +1,2 @@
+on:
+  post: []