devadmin/ProLibertateGames
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 1 Activity

Test: wait for the guest's sheet before comparing it
CI / Build and check (push) Failing after 7m34s
Details
CI / Deploy to Cloudflare Pages (push) Skipped
Details

Browse Source 
The sheet arrives over the socket just after connected flips, so on a slower
runner the comparison raced it (ScorekeeperSessionTest, line 167). Same
pattern the sibling test already uses.

Co-Authored-By: Claude Sonnet 5.5 <[email protected]>
Claude-Session: https://claude.ai/code/session_01JdMNbY5hLR7FjtfEE712xG
This commit is contained in:
Claude
2026-10-14 17:23:42 +00:00
parent e63c2cf7ac
commit 7b5b041c42
1 changed files with 3 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
app/src/test/java/org/prolibertate/games/net/ScorekeeperSessionTest.kt
+3
View File
@@ -164,6 +164,9 @@ class ScorekeeperSessionTest {
await("the guest to be let in") { guest.state.value.connected } await("the guest to be let in") { guest.state.value.connected }
assertFalse(guest.state.value.awaiting) assertFalse(guest.state.value.awaiting)
// The sheet follows the connection over the socket, so on a slow runner
// it can still be in flight when `connected` flips: wait for it.
await("the guest to receive the sheet") { guest.state.value.sheet == host.state.sheet }
assertEquals(host.state.sheet, guest.state.value.sheet) assertEquals(host.state.sheet, guest.state.value.sheet)
assertTrue(guest.state.value.sheet.started) assertTrue(guest.state.value.sheet.started)
assertTrue(host.state.requests.isEmpty()) assertTrue(host.state.requests.isEmpty())