fix tests and formatting

Author: Yermanaco

src/action.ts

@@ -2,7 +2,7 @@ import * as core from "@actions/core";
 import { v4 as uuid } from "uuid";
 
 const WORKFLOW_TIMEOUT_SECONDS = 5 * 60;
-const WORKFLOW_JOB_STEPS_RETRY_SECONDS = 5
+const WORKFLOW_JOB_STEPS_RETRY_SECONDS = 5;
 
 /**
  * action.yaml definition.

src/main.spec.ts

@@ -174,7 +174,7 @@ describe("main", () => {
       distinctIdRegex: distinctIdRegex,
       workflowId: 0,
       workflowTimeoutMs: testCfg.workflowTimeoutSeconds * 1000,
-      workflowJobStepsRetryMs :testCfg.workflowJobStepsRetrySeconds * 1000,
+      workflowJobStepsRetryMs: testCfg.workflowJobStepsRetrySeconds * 1000,
     });
 
     // Result

src/return-dispatch.spec.ts

@@ -665,7 +665,7 @@ describe("return-dispatch", () => {
 
       const retryMs = 5000;
       const timeoutMs = 60 * 60 * 100;
-    
+
       const getRunIdAndUrlPromise = getRunIdAndUrl({
         ...defaultOpts,
         workflowTimeoutMs: timeoutMs,
@@ -678,8 +678,9 @@ describe("return-dispatch", () => {
 
       assertOnlyCalled(coreInfoLogMock);
 
-      expect(coreInfoLogMock).toHaveBeenCalledOnce();
+      expect(coreInfoLogMock).toHaveBeenCalledTimes(2);
       expect(coreInfoLogMock.mock.calls[0]?.[0]).toMatchSnapshot();
+      expect(coreInfoLogMock.mock.calls[1]?.[0]).toMatchSnapshot();
 
       expect(utilSleepMock).toHaveBeenCalledOnce();
       expect(utilSleepMock).toHaveBeenCalledWith(retryMs);
@@ -788,8 +789,9 @@ describe("return-dispatch", () => {
       expect(apiFetchWorkflowRunJobStepsMock).toHaveBeenCalledOnce();
       assertOnlyCalled(coreDebugLogMock, coreInfoLogMock);
 
-      expect(coreInfoLogMock).toHaveBeenCalledOnce();
+      expect(coreInfoLogMock).toHaveBeenCalledTimes(2);
       expect(coreInfoLogMock.mock.calls[0]?.[0]).toMatchSnapshot();
+      expect(coreInfoLogMock.mock.calls[1]?.[0]).toMatchSnapshot();
 
       expect(coreDebugLogMock).toHaveBeenCalledOnce();
       expect(coreDebugLogMock.mock.calls[0]?.[0]).toMatchSnapshot();

src/return-dispatch.ts

@@ -140,7 +140,7 @@ export async function getRunIdAndUrl({
   workflowJobStepsRetryMs,
 }: GetRunIdAndUrlOpts): Promise<Result<{ id: number; url: string }>> {
   const retryTimeout = Math.max(
-    workflowJobStepsRetryMs,
+    constants.WORKFLOW_FETCH_TIMEOUT_MS,
     workflowTimeoutMs,
   );
 
@@ -194,4 +194,3 @@ export async function getRunIdAndUrl({
 
   return { success: false, reason: "timeout" };
 }
-