Separate entrypoint

action.yml

@@ -76,4 +76,4 @@ outputs:
 
 runs:
   using: "node20"
-  main: "lib/src/index.js"
+  main: "lib/src/entrypoint.js"

package.json

@@ -2,7 +2,7 @@
   "name": "mabl-github-deployments-action",
   "version": "1.13.1",
   "description": "mabl github action for GitHub pipelines integration",
-  "main": "lib/src/index.js",
+  "main": "lib/src/entrypoint.js",
   "scripts": {
     "test": "jest",
     "build": "tsc",

src/entrypoint.ts

@@ -0,0 +1,5 @@
+import {run} from './index';
+
+// This exists in a separate file, so that we don't implicitly invoke it during unit tests, allowing for overrides
+// eslint-disable-next-line
+run();

src/index.ts

@@ -57,13 +57,12 @@ export function booleanInput(name: string): boolean {
   );
 }
 
-export async function run(): Promise<void> {
+export async function run(enableFailureExitCodes = true): Promise<void> {
   const wrappedFailed = (message: string): void => {
     // Allow disabling, otherwise units will always fail in workflow builds w/ process exit code 1
-    if (process.env.DISABLE_FAILURE_EXIT_CODES !== 'true') {
-      // core.setFailed(message);
+    if (enableFailureExitCodes) {
+      core.setFailed(message);
     }
-    console.log(message);
   };
 
   try {
@@ -300,6 +299,3 @@ async function getRelatedPullRequest(): Promise<Option<PullRequest>> {
 
   return;
 }
-
-// eslint-disable-next-line
-run();

test/suite.test.ts

@@ -16,13 +16,13 @@ describe('GitHub Action tests', () => {
   }
 
   function assertExitCodeNot(expected: number): void {
-    expect(process.exitCode).not.toEqual(expected);
+    expect(process.exitCode).not.toBe(expected);
   }
 
   it('handles invalid application/environment ids', async () => {
     setGithubInput(ActionInputs.ApplicationId, '');
     setGithubInput(ActionInputs.EnvironmentId, '');
-    await run();
+    await run(false);
     assertExitCodeNot(1); // We'd normally seek '1', but that will break the CI/CD test harness
   });