From 25699e5e9769f45fe5270016aa3fb861e2c39347 Mon Sep 17 00:00:00 2001 From: NotAShelf Date: Mon, 16 Feb 2026 23:54:17 +0300 Subject: [PATCH] queue-runner: make fair-share assertion unconditional Signed-off-by: NotAShelf Change-Id: If983d379a5a369c547dff61b57f3a9a36a6a6964 --- crates/queue-runner/tests/runner_tests.rs | 16 +++++++++------- 1 file changed, 9 insertions(+), 7 deletions(-) diff --git a/crates/queue-runner/tests/runner_tests.rs b/crates/queue-runner/tests/runner_tests.rs index 102967e..7ea6312 100644 --- a/crates/queue-runner/tests/runner_tests.rs +++ b/crates/queue-runner/tests/runner_tests.rs @@ -426,13 +426,15 @@ async fn test_fair_share_scheduling() { .await .expect("list pending after running"); - if !pending2.is_empty() { - assert_eq!( - pending2[0].evaluation_id, eval_lo.id, - "underserved lo-share jobset should be scheduled first when hi-share is \ - over-served" - ); - } + assert!( + !pending2.is_empty(), + "expected pending builds from lo-share jobset" + ); + assert_eq!( + pending2[0].evaluation_id, eval_lo.id, + "underserved lo-share jobset should be scheduled first when hi-share is \ + over-served" + ); // Clean up let _ = fc_common::repo::projects::delete(&pool, project_hi.id).await;