Skip to content

test(mothership): keep the preview turn-budget test fast under load #8458

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
waleedlatif1 merged 1 commit into from
Sep 30, 2026
Merged

test(mothership): keep the preview turn-budget test fast under load #8458

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 7 additions & 1 deletion apps/sim/lib/mothership/request/go/file-preview-adapter.test.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -268,11 +268,17 @@ describe('processFilePreviewStreamEvent — preview byte rate', () => {
expect(completed).toBe(true)
})

/**
* Each 30 s patch of the ~300 KB file would stream ~6 MB of snapshots on its own,
* so four of them exceed the turn budget several times over without its cap. The
* 500 ms tick keeps the delta count, and so the test's CPU time, small enough to
* stay far inside the test timeout on a loaded machine.
*/
it('bounds all preview content in a turn of four long patches, and still completes each', async () => {
const turn = newTurn()
let contentBytes = 0
for (let edit = 0; edit < 4; edit++) {
const result = await streamPatch(120_000, 200, { turn, edit })
const result = await streamPatch(30_000, 500, { turn, edit })
expect(result.completed).toBe(true)
contentBytes += result.contentBytes
}
Expand Down
Loading