@redflow/client 0.0.3 → 0.0.5

package/tests/redflow.e2e.test.ts

@@ -52,7 +52,7 @@ test("manual run: succeeds and records steps", async () => {
     },
   );
 
-  const worker = await startWorker({
+  const worker = await startWorker({ app: "test-app",
     redis,
     prefix,
     queues: ["q1"],
@@ -82,7 +82,7 @@ test("runAt: scheduled -> promoted -> executed", async () => {
     return { ok: true };
   });
 
-  const worker = await startWorker({
+  const worker = await startWorker({ app: "test-app",
     redis,
     prefix,
     queues: ["q2"],
@@ -120,7 +120,7 @@ test("runAt: cleared after retry exhaustion final failure", async () => {
     },
   );
 
-  const worker = await startWorker({
+  const worker = await startWorker({ app: "test-app",
     redis,
     prefix,
     queues: ["q2_retry_fail"],
@@ -153,7 +153,7 @@ test("NonRetriableError: skips retries and fails immediately on attempt 1", asyn
     },
   );
 
-  const worker = await startWorker({
+  const worker = await startWorker({ app: "test-app",
     redis,
     prefix,
     queues: ["q_non_retriable"],
@@ -196,7 +196,7 @@ test("onFailure: called after retry exhaustion with correct context", async () =
     },
   );
 
-  const worker = await startWorker({
+  const worker = await startWorker({ app: "test-app",
     redis,
     prefix,
     queues: ["q_on_failure_1"],
@@ -239,7 +239,7 @@ test("onFailure: called immediately with NonRetriableError", async () => {
     },
   );
 
-  const worker = await startWorker({
+  const worker = await startWorker({ app: "test-app",
     redis,
     prefix,
     queues: ["q_on_failure_2"],
@@ -283,7 +283,7 @@ test("onFailure: NOT called on cancellation", async () => {
     },
   );
 
-  const worker = await startWorker({
+  const worker = await startWorker({ app: "test-app",
     redis,
     prefix,
     queues: ["q_on_failure_3"],
@@ -333,7 +333,7 @@ test("cron: creates runs and executes workflow", async () => {
     },
   );
 
-  const worker = await startWorker({
+  const worker = await startWorker({ app: "test-app",
     redis,
     prefix,
     queues: ["q4"],
@@ -352,6 +352,75 @@ test("cron: creates runs and executes workflow", async () => {
   redis.close();
 });
 
+test("cron: skips when workflow already running", async () => {
+  const prefix = testPrefix();
+  const redis = new RedisClient(redisServer.url);
+  const client = createClient({ redis, prefix });
+  setDefaultClient(client);
+
+  const manualCountKey = `${prefix}:t:cronSkipManualCount`;
+  const cronCountKey = `${prefix}:t:cronSkipCronCount`;
+
+  defineWorkflow(
+    "cron-skip-running-wf",
+    {
+      queue: "q4_skip",
+      cron: [{ expression: "*/5 * * * * *", input: { source: "cron" } }],
+    },
+    async ({ input, step }) => {
+      await step.run({ name: "record-source" }, async () => {
+        const source = (input as { source?: string } | null | undefined)?.source;
+        if (source === "cron") {
+          await redis.incr(cronCountKey);
+        } else {
+          await redis.incr(manualCountKey);
+        }
+        return true;
+      });
+
+      await step.run({ name: "hold" }, async () => {
+        await new Promise((resolve) => setTimeout(resolve, 6500));
+        return true;
+      });
+
+      return { ok: true };
+    },
+  );
+
+  const worker = await startWorker({ app: "test-app",
+    redis,
+    prefix,
+    queues: ["q4_skip"],
+    concurrency: 2,
+    runtime: { leaseMs: 500 },
+  });
+
+  try {
+    const manualHandle = await client.emitWorkflow("cron-skip-running-wf", { source: "manual" });
+
+    await waitFor(
+      async () => Number((await redis.get(manualCountKey)) ?? "0") >= 1,
+      { timeoutMs: 5000, label: "manual run started" },
+    );
+
+    // Wait long enough for one cron tick while manual run is still executing.
+    await new Promise((resolve) => setTimeout(resolve, 5500));
+    const cronWhileManualRunning = Number((await redis.get(cronCountKey)) ?? "0");
+    expect(cronWhileManualRunning).toBe(0);
+
+    await manualHandle.result({ timeoutMs: 15_000 });
+
+    // Cron should resume once the running instance is gone.
+    await waitFor(
+      async () => Number((await redis.get(cronCountKey)) ?? "0") >= 1,
+      { timeoutMs: 10_000, label: "cron resumed after manual run" },
+    );
+  } finally {
+    await worker.stop();
+    redis.close();
+  }
+}, { timeout: 30_000 });
+
 test("step.runWorkflow: auto idempotency and override are forwarded to child runs", async () => {
   const prefix = testPrefix();
   const redis = new RedisClient(redisServer.url);
@@ -388,7 +457,7 @@ test("step.runWorkflow: auto idempotency and override are forwarded to child run
     return { auto, custom };
   });
 
-  const worker = await startWorker({
+  const worker = await startWorker({ app: "test-app",
     redis,
     prefix,
     queues: ["q_rw_parent", "q_rw_child"],
@@ -451,7 +520,7 @@ test("step.runWorkflow: child workflow executes once across parent retries", asy
     },
   );
 
-  const worker = await startWorker({
+  const worker = await startWorker({ app: "test-app",
     redis,
     prefix,
     queues: ["q_rw_retry_parent", "q_rw_retry_child"],
@@ -508,7 +577,7 @@ test("step.runWorkflow: same queue avoids self-deadlock with concurrency 1", asy
     return await step.runWorkflow({ name: "call-child" }, child, {});
   });
 
-  const worker = await startWorker({
+  const worker = await startWorker({ app: "test-app",
     redis,
     prefix,
     queues: ["q_rw_single"],
@@ -552,7 +621,7 @@ test("step.emitWorkflow: supports workflow name strings", async () => {
     return { childRunId };
   });
 
-  const worker = await startWorker({
+  const worker = await startWorker({ app: "test-app",
     redis,
     prefix,
     queues: ["q_emit_name_parent", "q_emit_name_child"],
@@ -606,7 +675,7 @@ test("retries: step results are cached across attempts", async () => {
     },
   );
 
-  const worker = await startWorker({
+  const worker = await startWorker({ app: "test-app",
     redis,
     prefix,
     queues: ["q5"],
@@ -654,7 +723,7 @@ test("retries: run queued before worker start keeps workflow maxAttempts", async
   const queuedState = await client.getRun(handle.id);
   expect(queuedState?.maxAttempts).toBe(2);
 
-  const worker = await startWorker({
+  const worker = await startWorker({ app: "test-app",
     redis,
     prefix,
     queues: ["q5_before_sync"],
@@ -687,7 +756,7 @@ test("step timeout: run fails and error is recorded", async () => {
     return { ok: true };
   });
 
-  const worker = await startWorker({
+  const worker = await startWorker({ app: "test-app",
     redis,
     prefix,
     queues: ["q6"],
@@ -737,7 +806,7 @@ test("cancellation: scheduled/queued/running", async () => {
 
   // Scheduled cancel.
   {
-    const worker = await startWorker({
+    const worker = await startWorker({ app: "test-app",
       redis,
       prefix,
       queues: ["q7"],
@@ -773,7 +842,7 @@ test("cancellation: scheduled/queued/running", async () => {
 
   // Running cancel.
   {
-    const worker = await startWorker({
+    const worker = await startWorker({ app: "test-app",
       redis,
       prefix,
       queues: ["q7"],
@@ -856,7 +925,7 @@ test("cancel race: queued cancel before start does not execute handler", async (
 
   let worker: Awaited<ReturnType<typeof startWorker>> | null = null;
   try {
-    worker = await startWorker({
+    worker = await startWorker({ app: "test-app",
       redis,
       prefix,
       queues: [queue],
@@ -977,7 +1046,7 @@ test("idempotencyKey: same key returns same run id and executes once", async ()
     return { ok: true };
   });
 
-  const worker = await startWorker({
+  const worker = await startWorker({ app: "test-app",
     redis,
     prefix,
     queues: ["q9"],
@@ -1015,7 +1084,7 @@ test("idempotencyKey: delayed TTL refresh cannot fork duplicate runs", async ()
     return { ok: true };
   });
 
-  const worker = await startWorker({
+  const worker = await startWorker({ app: "test-app",
     redis,
     prefix,
     queues: ["q9_race_fix"],
@@ -1063,7 +1132,7 @@ test("enqueue: producer-side zadd failure does not leave runs orphaned", async (
   setDefaultClient(client);
 
   const wf = defineWorkflow("enqueue-atomic", {queue: "q_enqueue_atomic" }, async () => ({ ok: true }));
-  await client.syncRegistry(getDefaultRegistry());
+  await client.syncRegistry(getDefaultRegistry(), { app: "test-app" });
 
   const originalZadd = redis.zadd.bind(redis);
   let injected = false;
@@ -1082,7 +1151,7 @@ test("enqueue: producer-side zadd failure does not leave runs orphaned", async (
     redis.zadd = originalZadd;
   }
 
-  const worker = await startWorker({
+  const worker = await startWorker({ app: "test-app",
     redis,
     prefix,
     queues: ["q_enqueue_atomic"],
@@ -1107,7 +1176,7 @@ test("idempotencyKey: delimiter-like workflow/key pairs do not collide", async (
   const wfA = defineWorkflow("idem:a", {queue: "q9a" }, async () => ({ workflow: "idem:a" }));
   const wfB = defineWorkflow("idem:a:b", {queue: "q9b" }, async () => ({ workflow: "idem:a:b" }));
 
-  const worker = await startWorker({
+  const worker = await startWorker({ app: "test-app",
     redis,
     prefix,
     queues: ["q9a", "q9b"],
@@ -1150,7 +1219,7 @@ test("idempotencyTtl: short TTL expires and allows a new run with same key", asy
     return { ok: true };
   });
 
-  const worker = await startWorker({
+  const worker = await startWorker({ app: "test-app",
     redis,
     prefix,
     queues: ["q_idem_ttl"],
@@ -1199,7 +1268,7 @@ test("input validation: invalid input fails once and is not retried", async () =
     },
   );
 
-  const worker = await startWorker({
+  const worker = await startWorker({ app: "test-app",
     redis,
     prefix,
     queues: ["q11"],
@@ -1234,7 +1303,7 @@ test("unknown workflow: run fails and is not retried", async () => {
   const redis = new RedisClient(redisServer.url);
   const client = createClient({ redis, prefix });
 
-  const worker = await startWorker({
+  const worker = await startWorker({ app: "test-app",
     redis,
     prefix,
     queues: ["qx"],
@@ -1287,7 +1356,7 @@ test("cancel during step: run becomes canceled and step error kind is canceled",
     return { ok: true };
   });
 
-  const worker = await startWorker({
+  const worker = await startWorker({ app: "test-app",
     redis,
     prefix,
     queues: ["q12"],
@@ -1345,7 +1414,7 @@ test("terminal run re-queued is ignored (no re-execution)", async () => {
     return { ok: true };
   });
 
-  const worker = await startWorker({
+  const worker = await startWorker({ app: "test-app",
     redis,
     prefix,
     queues: [queue],
@@ -1393,7 +1462,7 @@ test("lease+reaper: long running step is not duplicated", async () => {
     return { ok: true };
   });
 
-  const worker = await startWorker({
+  const worker = await startWorker({ app: "test-app",
     redis,
     prefix,
     queues: [queue],
@@ -1438,8 +1507,8 @@ test(
     },
   );
 
-  const w1 = await startWorker({ redis, prefix, queues: [queue], concurrency: 1, runtime: { leaseMs: 500 } });
-  const w2 = await startWorker({ redis, prefix, queues: [queue], concurrency: 1, runtime: { leaseMs: 500 } });
+  const w1 = await startWorker({ app: "test-app", redis, prefix, queues: [queue], concurrency: 1, runtime: { leaseMs: 500 } });
+  const w2 = await startWorker({ app: "test-app", redis, prefix, queues: [queue], concurrency: 1, runtime: { leaseMs: 500 } });
 
   await new Promise((r) => setTimeout(r, 3500));
   const count = Number((await redis.get(counterKey)) ?? "0");
@@ -1505,7 +1574,7 @@ test(
       },
     );
 
-    const worker = await startWorker({
+    const worker = await startWorker({ app: "test-app",
       redis,
       prefix,
       queues: ["q_parent", "q_child"],
@@ -1560,7 +1629,7 @@ test(
     const queuedMembers = await redis.zrange(queuedIndex, 0, -1);
     expect(queuedMembers.includes(handle.id)).toBe(true);
 
-    const worker = await startWorker({
+    const worker = await startWorker({ app: "test-app",
       redis,
       prefix,
       queues: [queue],
@@ -1606,7 +1675,7 @@ test(
       cron: [{ expression: "*/1 * * * * *" }],
     }, async () => ({ ok: true }));
 
-    await client.syncRegistry(getDefaultRegistry());
+    await client.syncRegistry(getDefaultRegistry(), { app: "test-app" });
 
     const workflowKey = `${prefix}:workflow:${name}`;
     const cronIdsJson = await redis.hget(workflowKey, "cronIdsJson");
@@ -1624,7 +1693,7 @@ test(
       queue: "q_update",
     }, async () => ({ ok: true }));
 
-    await client.syncRegistry(getDefaultRegistry());
+    await client.syncRegistry(getDefaultRegistry(), { app: "test-app" });
 
     expect(await redis.hget(`${prefix}:cron:def`, cronId)).toBeNull();
     const cronNext2 = await redis.zrange(`${prefix}:cron:next`, 0, -1);
@@ -1658,7 +1727,7 @@ test(
       },
     );
 
-    const worker = await startWorker({
+    const worker = await startWorker({ app: "test-app",
       redis,
       prefix,
       queues: [queue],
@@ -1708,8 +1777,8 @@ test(
       },
     );
 
-    const w1 = await startWorker({ redis, prefix, queues: [queue], concurrency: 1, runtime: { leaseMs: 500 } });
-    const w2 = await startWorker({ redis, prefix, queues: [queue], concurrency: 1, runtime: { leaseMs: 500 } });
+    const w1 = await startWorker({ app: "test-app", redis, prefix, queues: [queue], concurrency: 1, runtime: { leaseMs: 500 } });
+    const w2 = await startWorker({ app: "test-app", redis, prefix, queues: [queue], concurrency: 1, runtime: { leaseMs: 500 } });
 
     await waitFor(
       async () => Number((await redis.get(counterKey)) ?? "0") >= 1,
@@ -1744,7 +1813,7 @@ test("listRuns: status + workflow filters are applied together", async () => {
     throw new Error("expected failure");
   });
 
-  const worker = await startWorker({ redis, prefix, queues: [queue], runtime: { leaseMs: 500 } });
+  const worker = await startWorker({ app: "test-app", redis, prefix, queues: [queue], runtime: { leaseMs: 500 } });
 
   const successHandle = await succeeds.run({});
   const failHandle = await fails.run({});
@@ -1795,7 +1864,7 @@ test("cron trigger ids: same custom id in two workflows does not collide", async
     async () => ({ ok: true }),
   );
 
-  await client.syncRegistry(getDefaultRegistry());
+  await client.syncRegistry(getDefaultRegistry(), { app: "test-app" });
 
   const idsA = JSON.parse((await redis.hget(`${prefix}:workflow:cron-id-a`, "cronIdsJson")) ?? "[]") as string[];
   const idsB = JSON.parse((await redis.hget(`${prefix}:workflow:cron-id-b`, "cronIdsJson")) ?? "[]") as string[];
@@ -1823,7 +1892,7 @@ test("syncRegistry: cron trigger with explicit null input preserves null payload
     async () => ({ ok: true }),
   );
 
-  await client.syncRegistry(getDefaultRegistry());
+  await client.syncRegistry(getDefaultRegistry(), { app: "test-app" });
 
   const workflowKey = `${prefix}:workflow:${workflowName}`;
   const cronIds = JSON.parse((await redis.hget(workflowKey, "cronIdsJson")) ?? "[]") as string[];
@@ -1854,7 +1923,7 @@ test("syncRegistry: invalid cron expression removes existing next schedule for s
     async () => ({ ok: true }),
   );
 
-  await client.syncRegistry(getDefaultRegistry());
+  await client.syncRegistry(getDefaultRegistry(), { app: "test-app" });
 
   const cronIds = JSON.parse((await redis.hget(workflowKey, "cronIdsJson")) ?? "[]") as string[];
   expect(cronIds.length).toBe(1);
@@ -1870,7 +1939,7 @@ test("syncRegistry: invalid cron expression removes existing next schedule for s
     async () => ({ ok: true }),
   );
 
-  await client.syncRegistry(getDefaultRegistry());
+  await client.syncRegistry(getDefaultRegistry(), { app: "test-app" });
 
   const after = await redis.zrange(cronNextKey, 0, -1);
   expect(after.includes(cronId)).toBe(false);
@@ -1891,7 +1960,7 @@ test("syncRegistry: removes stale workflow metadata (cron)", async () => {
     async () => ({ ok: true }),
   );
 
-  await client.syncRegistry(getDefaultRegistry());
+  await client.syncRegistry(getDefaultRegistry(), { app: "test-app" });
 
   const workflowKey = `${prefix}:workflow:${workflowName}`;
   const cronIds = JSON.parse((await redis.hget(workflowKey, "cronIdsJson")) ?? "[]") as string[];
@@ -1903,7 +1972,7 @@ test("syncRegistry: removes stale workflow metadata (cron)", async () => {
   // Force stale age beyond grace period and sync an empty registry.
   await redis.hset(workflowKey, { updatedAt: "1" });
   __unstableResetDefaultRegistryForTests();
-  await client.syncRegistry(getDefaultRegistry());
+  await client.syncRegistry(getDefaultRegistry(), { app: "test-app" });
 
   expect((await redis.smembers(`${prefix}:workflows`)).includes(workflowName)).toBe(false);
   expect(await redis.hget(workflowKey, "queue")).toBeNull();
@@ -1914,37 +1983,37 @@ test("syncRegistry: removes stale workflow metadata (cron)", async () => {
   redis.close();
 });
 
-test("syncRegistry: stale cleanup is isolated by owner", async () => {
+test("syncRegistry: stale cleanup is isolated by app", async () => {
   const prefix = testPrefix();
   const redis = new RedisClient(redisServer.url);
   const client = createClient({ redis, prefix });
 
   __unstableResetDefaultRegistryForTests();
-  defineWorkflow("owner-a-wf", {queue: "qa",
+  defineWorkflow("app-a-wf", {queue: "qa",
       cron: [{ id: "a", expression: "*/30 * * * * *" }],
     },
     async () => ({ ok: true }),
   );
-  await client.syncRegistry(getDefaultRegistry(), { owner: "svc-a" });
+  await client.syncRegistry(getDefaultRegistry(), { app: "svc-a" });
 
   __unstableResetDefaultRegistryForTests();
-  defineWorkflow("owner-b-wf", {queue: "qb",
+  defineWorkflow("app-b-wf", {queue: "qb",
       cron: [{ id: "b", expression: "*/30 * * * * *" }],
     },
     async () => ({ ok: true }),
   );
-  await client.syncRegistry(getDefaultRegistry(), { owner: "svc-b" });
+  await client.syncRegistry(getDefaultRegistry(), { app: "svc-b" });
 
-  await redis.hset(`${prefix}:workflow:owner-a-wf`, { updatedAt: "1" });
-  await redis.hset(`${prefix}:workflow:owner-b-wf`, { updatedAt: "1" });
+  await redis.hset(`${prefix}:workflow:app-a-wf`, { updatedAt: "1" });
+  await redis.hset(`${prefix}:workflow:app-b-wf`, { updatedAt: "1" });
 
   __unstableResetDefaultRegistryForTests();
-  await client.syncRegistry(getDefaultRegistry(), { owner: "svc-a" });
+  await client.syncRegistry(getDefaultRegistry(), { app: "svc-a" });
 
-  expect(await redis.hget(`${prefix}:workflow:owner-a-wf`, "queue")).toBeNull();
-  expect(await redis.hget(`${prefix}:workflow:owner-b-wf`, "queue")).not.toBeNull();
-  expect((await redis.smembers(`${prefix}:workflows`)).includes("owner-a-wf")).toBe(false);
-  expect((await redis.smembers(`${prefix}:workflows`)).includes("owner-b-wf")).toBe(true);
+  expect(await redis.hget(`${prefix}:workflow:app-a-wf`, "queue")).toBeNull();
+  expect(await redis.hget(`${prefix}:workflow:app-b-wf`, "queue")).not.toBeNull();
+  expect((await redis.smembers(`${prefix}:workflows`)).includes("app-a-wf")).toBe(false);
+  expect((await redis.smembers(`${prefix}:workflows`)).includes("app-b-wf")).toBe(true);
 
   redis.close();
 });
@@ -1957,7 +2026,7 @@ test("cancelRun: terminal runs are unchanged", async () => {
 
   const wf = defineWorkflow("cancel-terminal", {queue: "q_cancel_terminal" }, async () => ({ ok: true }));
 
-  const worker = await startWorker({
+  const worker = await startWorker({ app: "test-app",
     redis,
     prefix,
     queues: ["q_cancel_terminal"],
@@ -1992,7 +2061,7 @@ test("output serialization: non-JSON output fails once and is not stuck", async
     async () => ({ bad: 1n }),
   );
 
-  const worker = await startWorker({
+  const worker = await startWorker({ app: "test-app",
     redis,
     prefix,
     queues: ["q_serial"],
@@ -2047,7 +2116,7 @@ test("retry step sequence: new steps keep monotonic order after cached steps", a
     },
   );
 
-  const worker = await startWorker({
+  const worker = await startWorker({ app: "test-app",
     redis,
     prefix,
     queues: ["q_seq"],
@@ -2078,7 +2147,7 @@ test("listRuns: negative offset is clamped to zero", async () => {
   setDefaultClient(client);
 
   const wf = defineWorkflow("offset-wf", {queue: "q_offset" }, async () => ({ ok: true }));
-  const worker = await startWorker({
+  const worker = await startWorker({ app: "test-app",
     redis,
     prefix,
     queues: ["q_offset"],
@@ -2142,7 +2211,7 @@ test("error serialization: non-serializable thrown values do not wedge a run", a
     },
   );
 
-  const worker = await startWorker({
+  const worker = await startWorker({ app: "test-app",
     redis,
     prefix,
     queues: ["q_bigint"],
@@ -2185,7 +2254,7 @@ test("cron trigger ids: delimiter-like custom ids do not collide", async () => {
     async () => ({ ok: true }),
   );
 
-  await client.syncRegistry(getDefaultRegistry());
+  await client.syncRegistry(getDefaultRegistry(), { app: "test-app" });
 
   const idsA = JSON.parse((await redis.hget(`${prefix}:workflow:wf:a`, "cronIdsJson")) ?? "[]") as string[];
   const idsB = JSON.parse((await redis.hget(`${prefix}:workflow:wf:a:b`, "cronIdsJson")) ?? "[]") as string[];
@@ -2219,7 +2288,7 @@ test("idempotencyKey: stale pointer is recovered instead of returning missing ru
 
   const wf = defineWorkflow(workflowName, {queue }, async () => ({ ok: true }));
 
-  const worker = await startWorker({
+  const worker = await startWorker({ app: "test-app",
     redis,
     prefix,
     queues: [queue],
@@ -2262,7 +2331,7 @@ test("idempotencyKey: partial existing run is repaired and executed", async () =
     return { ok: true };
   });
 
-  const worker = await startWorker({
+  const worker = await startWorker({ app: "test-app",
     redis,
     prefix,
     queues: [queue],
@@ -2314,7 +2383,7 @@ test("syncRegistry: duplicate custom cron ids in one workflow are rejected", asy
     async () => ({ ok: true }),
   );
 
-  await expect(client.syncRegistry(getDefaultRegistry())).rejects.toThrow("Duplicate cron trigger id");
+  await expect(client.syncRegistry(getDefaultRegistry(), { app: "test-app" })).rejects.toThrow("Duplicate cron trigger id");
 
   redis.close();
 });
@@ -2331,7 +2400,7 @@ test("scheduled promoter: stale scheduled entry does not resurrect terminal run"
     return { ok: true };
   });
 
-  const worker = await startWorker({
+  const worker = await startWorker({ app: "test-app",
     redis,
     prefix,
     queues: [queue],
@@ -2398,7 +2467,7 @@ test(
 
     let worker: Awaited<ReturnType<typeof startWorker>> | null = null;
     try {
-      worker = await startWorker({
+      worker = await startWorker({ app: "test-app",
         redis,
         prefix,
         queues: [queue],
@@ -2449,7 +2518,7 @@ test(
 
     let worker: Awaited<ReturnType<typeof startWorker>> | null = null;
     try {
-      worker = await startWorker({
+      worker = await startWorker({ app: "test-app",
         redis,
         prefix,
         queues: [queue],
@@ -2492,7 +2561,7 @@ test("workflow names ending with ':runs' do not collide with workflow run indexe
 
   expect(keys.workflow(prefix, "keyspace-base:runs")).not.toBe(keys.workflowRuns(prefix, "keyspace-base"));
 
-  const worker = await startWorker({
+  const worker = await startWorker({ app: "test-app",
     redis,
     prefix,
     queues: [queue],
@@ -2537,7 +2606,7 @@ test("cancelRun race: near-finish cancellation settles as canceled", async () =>
     return await originalFinalizeRun.call(this, runId, args);
   }) as RedflowClient["finalizeRun"];
 
-  const worker = await startWorker({
+  const worker = await startWorker({ app: "test-app",
     redis,
     prefix,
     queues: [queue],
@@ -2600,7 +2669,7 @@ test("cancelRun race: cancellation requested during finalize wins over success",
     return await originalFinalizeRun.call(this, runId, args);
   }) as RedflowClient["finalizeRun"];
 
-  const worker = await startWorker({
+  const worker = await startWorker({ app: "test-app",
     redis,
     prefix,
     queues: [queue],
@@ -2650,7 +2719,7 @@ test(
       await new Promise<never>(() => {});
     });
 
-    const worker = await startWorker({
+    const worker = await startWorker({ app: "test-app",
       redis,
       prefix,
       queues: [queue],