Skip to content

Commit 9bb9dd9

Browse files
committed
Use wait helper
1 parent a9c6990 commit 9bb9dd9

2 files changed

Lines changed: 5 additions & 5 deletions

File tree

packages/common/test/WebSocket.test.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
import { afterEach, beforeEach, expect, test, vi } from "vitest";
22
import WebSocket, { WebSocketServer } from "ws";
3-
import { RetryError } from "../src/Promise.js";
3+
import { RetryError, wait } from "../src/Promise.js";
44
import { err, ok } from "../src/Result.js";
55
import {
66
createWebSocket,
@@ -299,7 +299,7 @@ test("cleans up all resources when dispose is called", async () => {
299299
expect(sendResult).toEqual(err({ type: "WebSocketSendError" }));
300300

301301
// Wait a bit to ensure no reconnection attempts
302-
await new Promise((resolve) => setTimeout(resolve, 500));
302+
await wait(500);
303303

304304
// No additional onOpen calls should happen after disposal
305305
expect(onOpen).toHaveBeenCalledTimes(1);
@@ -436,7 +436,7 @@ test("aborts connection attempts when disposed", async () => {
436436
const callCountAtDispose = onRetry.mock.calls.length;
437437

438438
// Wait some time to ensure no more retries happen
439-
await new Promise((resolve) => setTimeout(resolve, 500));
439+
await wait(500);
440440

441441
// Verify no additional retry calls happened after dispose
442442
expect(onRetry.mock.calls.length).toBe(callCountAtDispose);

packages/web/test/SharedWebWorker.test.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
/* eslint-disable @typescript-eslint/no-unsafe-assignment */
55
import { expect, test, vi, beforeEach, afterEach } from "vitest";
66
import { createSharedWebWorker } from "../src/SharedWebWorker.js";
7-
import { getOrThrow, SimpleName } from "@evolu/common";
7+
import { getOrThrow, SimpleName, wait } from "@evolu/common";
88

99
// Mock BroadcastChannel
1010
class MockBroadcastChannel {
@@ -231,7 +231,7 @@ test("createSharedWebWorker handles multiple tabs - first tab becomes owner", as
231231
});
232232

233233
// Wait for lock acquisition
234-
await new Promise((resolve) => setTimeout(resolve, 10));
234+
await wait(10);
235235

236236
// Only first tab should create worker (it became owner)
237237
expect(mockCreateWorker).toHaveBeenCalledTimes(1);

0 commit comments

Comments
 (0)