Skip to content

Commit 62c3d36

Browse files
committed
test: satisfy cron cancellation lint
1 parent 9a1ba06 commit 62c3d36

1 file changed

Lines changed: 9 additions & 3 deletions

File tree

src/cron/service/timer.regression.test.ts

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -760,7 +760,7 @@ describe("cron service timer regressions", () => {
760760
});
761761
await saveCronStore(store.storePath, { version: 1, jobs: [cronJob] });
762762

763-
let now = scheduledAt;
763+
const now = scheduledAt;
764764
let abortObserved = false;
765765
let timerSettled = false;
766766
const runnerStarted = createDeferred<void>();
@@ -806,7 +806,10 @@ describe("cron service timer regressions", () => {
806806
expect(cancelResult.cancelled).toBe(true);
807807
expect(abortObserved).toBe(true);
808808

809-
for (let attempt = 0; attempt < 5 && !timerSettled; attempt += 1) {
809+
for (let attempt = 0; attempt < 5; attempt += 1) {
810+
if (timerSettled) {
811+
break;
812+
}
810813
await vi.advanceTimersByTimeAsync(0);
811814
await Promise.resolve();
812815
}
@@ -985,7 +988,10 @@ describe("cron service timer regressions", () => {
985988
expect(cancelled).toBe(true);
986989
expect(observedAbortSignal?.aborted).toBe(true);
987990

988-
for (let attempt = 0; attempt < 5 && !timerSettled; attempt += 1) {
991+
for (let attempt = 0; attempt < 5; attempt += 1) {
992+
if (timerSettled) {
993+
break;
994+
}
989995
await vi.advanceTimersByTimeAsync(0);
990996
await Promise.resolve();
991997
}

0 commit comments

Comments
 (0)