moby/brewlog
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix: stabilize flaky timeline CLI test under parallel execution

Browse source 
Fetch timeline with page_size=all so concurrent test data from other
CLI tests doesn't push events off page 1, causing assertion failures.
This commit is contained in:
Jon Seager 2026-02-22 10:03:56 +00:00
parent dfb7e86f6d
commit 7b9cf8147e
No known key found for this signature in database
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
tests/cli/timeline_cli.rs
View file

@ -16,7 +16,7 @@ fn test_editing_roaster_updates_timeline_events_after_rebuild() {
// Verify original roaster name appears in timeline HTML // Verify original roaster name appears in timeline HTML
let client = reqwest::blocking::Client::new(); let client = reqwest::blocking::Client::new();
let body = client let body = client
.get(format!("{address}/timeline")) .get(format!("{address}/timeline?page_size=all"))
.send() .send()
.expect("failed to fetch timeline") .expect("failed to fetch timeline")
.text() .text()
@ -58,7 +58,7 @@ fn test_editing_roaster_updates_timeline_events_after_rebuild() {
// Verify both timeline events now show the updated roaster name // Verify both timeline events now show the updated roaster name
let body = client let body = client
.get(format!("{address}/timeline")) .get(format!("{address}/timeline?page_size=all"))
.send() .send()
.expect("failed to fetch timeline after rebuild") .expect("failed to fetch timeline after rebuild")
.text() .text()