fix: attemptToFindRunId would return timeout if called with an empty …

…array

src/return-dispatch.spec.ts

@@ -277,6 +277,22 @@ describe("return-dispatch", () => {
       assertNoneCalled();
     });
 
+    it("does nothing if called with an empty array", async () => {
+      const result = await attemptToFindRunId(new RegExp(testId), []);
+      if (result.success) {
+        expect.fail("result found when none expected");
+      }
+
+      // Behaviour
+      expect(result.success).toStrictEqual(false);
+      expect(result.reason).toStrictEqual("invalid input");
+      expect(getWorkflowRunJobStepMock).not.toHaveBeenCalled();
+      expect(fetchWorkflowRunUrlMock).not.toHaveBeenCalled();
+
+      // Logging
+      assertNoneCalled();
+    });
+
     describe("server error retries", () => {
       beforeEach(() => {
         vi.spyOn(

src/return-dispatch.ts

@@ -42,6 +42,13 @@ export async function attemptToFindRunId(
   idRegex: RegExp,
   workflowRunIds: number[],
 ): Promise<Result<{ id: number; url: string }>> {
+  if (workflowRunIds.length === 0) {
+    return {
+      success: false,
+      reason: "invalid input",
+    };
+  }
+
   let currentWorkflowRunIndex = 0;
   let currentFetchWorkflowRunJobStepsAttempt = 0;
   while (currentWorkflowRunIndex < workflowRunIds.length) {

src/types.ts

@@ -1,4 +1,4 @@
-export type Result<T> = ResultSuccess<T> | ResultTimeout;
+export type Result<T> = ResultSuccess<T> | ResultTimeout | ResultInvalidInput;
 
 interface ResultSuccess<T> {
   success: true;
@@ -9,3 +9,8 @@ interface ResultTimeout {
   success: false;
   reason: "timeout";
 }
+
+interface ResultInvalidInput {
+  success: false;
+  reason: "invalid input";
+}