WorkspaceHeartbeatModal > loads global heartbeat defaults when the workspace has no saved heartbeat config failed once in a merge group. That failure removed #6085 from the merge queue.
Likely cause (a test race, not verified by a repro): the test calls waitFor(() => getByLabelText("Heartbeat interval in minutes")). That resolves as soon as the field renders, possibly before config.getConfig() has resolved and replaced the built-in default (30 min) with the global default (6 min). Then expect(value).toBe("6") runs outside waitFor and reads 30.
Fix: assert the value inside waitFor (wait for "6"), or await getConfigMock before reading the field. The coordinator desk owns this issue.
Generated with xum • Model: anthropic:claude-opus-5-5 • Thinking: high
WorkspaceHeartbeatModal > loads global heartbeat defaults when the workspace has no saved heartbeat configfailed once in a merge group. That failure removed #6085 from the merge queue.Test / Unit (4/6)(job 114325051871), commit = main40de096e6bplus 🤖 tests: bug-bash sandbox self-check and CI job (D2) #6085. 🤖 tests: bug-bash sandbox self-check and CI job (D2) #6085 changes onlytests/bugbash/**, the Makefile,pr.ymland docs.WorkspaceHeartbeatModal.test.tsx:242: expected"6", received"30".40de096e6bandccdd3dcf35passed. The test has not changed since 🤖 fix: reject leading-dash workspace names and let a heartbeat go back to the default interval #5726 (Oct 6).Likely cause (a test race, not verified by a repro): the test calls
waitFor(() => getByLabelText("Heartbeat interval in minutes")). That resolves as soon as the field renders, possibly beforeconfig.getConfig()has resolved and replaced the built-in default (30 min) with the global default (6 min). Thenexpect(value).toBe("6")runs outsidewaitForand reads 30.Fix: assert the value inside
waitFor(wait for"6"), or awaitgetConfigMockbefore reading the field. The coordinator desk owns this issue.Generated with
xum• Model:anthropic:claude-opus-5-5• Thinking:high