PrivateCoffee/tldraw
4
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

attempted fix of a flaky ClientWebSocketAdapter test (#3114)

Browse source 
### Change Type

- [x] `tests` — Changes to any test code only
This commit is contained in:
Dan Groshev 2024-03-11 17:33:02 +00:00 • committed by GitHub
parent 5e54526776
commit 19a1d01b8e
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 3 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

3
apps/dotcom/src/utils/remote-sync/ClientWebSocketAdapter.test.ts
View file

@ -65,6 +65,9 @@ describe(ClientWebSocketAdapter, () => {
const prevServerSocket = connectedServerSocket const prevServerSocket = connectedServerSocket
prevServerSocket.terminate() prevServerSocket.terminate()
await waitFor(() => connectedServerSocket !== prevServerSocket) await waitFor(() => connectedServerSocket !== prevServerSocket)
// there is a race here, the server could've opened a new socket already, but it hasn't
// transitioned to OPEN yet, thus the second waitFor
await waitFor(() => connectedServerSocket.readyState === WebSocket.OPEN)
expect(adapter._ws).not.toBe(prevClientSocket) expect(adapter._ws).not.toBe(prevClientSocket)
expect(adapter._ws?.readyState).toBe(WebSocket.OPEN) expect(adapter._ws?.readyState).toBe(WebSocket.OPEN)
}) })