From 1f472d61eaa1b66635cb900c03e7cd832ad4e862 Mon Sep 17 00:00:00 2001 From: aiongg Date: Fri, 23 Oct 2026 16:49:11 +0000 Subject: [PATCH] Tests: report a bad PHOTOCRAFT_SWEEP_PART with assert!, which testkit's lints allow --- crates/testkit/src/sweep.rs | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/crates/testkit/src/sweep.rs b/crates/testkit/src/sweep.rs index ae3552226e..863a5c9029 100644 --- a/crates/testkit/src/sweep.rs +++ b/crates/testkit/src/sweep.rs @@ -20,10 +20,11 @@ impl Sweep { } let parts = parts.max(1); let part = match std::env::var("PHOTOCRAFT_SWEEP_PART") { - Ok(v) => match v.parse::() { - Ok(p) if p < parts => p, - _ => panic!("PHOTOCRAFT_SWEEP_PART={v:?} is not a part number below {parts}"), - }, + Ok(v) => { + let p = v.parse::().unwrap_or(u64::MAX); + assert!(p < parts, "PHOTOCRAFT_SWEEP_PART={v:?} is not a part number below {parts}"); + p + } Err(_) => SystemTime::now().duration_since(UNIX_EPOCH).map(|d| d.as_secs() / 86_400).unwrap_or_default() % parts, }; eprintln!("sweep part {part} of {parts}: replay with PHOTOCRAFT_SWEEP_PART={part}, run every part with PHOTOCRAFT_FULL_SWEEP=1");