Tests: refuse a PHOTOCRAFT_SWEEP_PART that isn't a part number

2026-10-24 09:57:08 -06:00 · 2026-10-23 16:40:24 +00:00
parent 18f4b5348c
commit 066ad384cd
1 changed files with 6 additions and 3 deletions
@@ -19,9 +19,12 @@ impl Sweep {
            return Self { part: 0, parts: 1 };
        }
        let parts = parts.max(1);
-        let part = match std::env::var("PHOTOCRAFT_SWEEP_PART").ok().and_then(|v| v.parse::<u64>().ok()) {
-            Some(p) => p % parts,
-            None => SystemTime::now().duration_since(UNIX_EPOCH).map(|d| d.as_secs() / 86_400).unwrap_or_default() % parts,
+        let part = match std::env::var("PHOTOCRAFT_SWEEP_PART") {
+            Ok(v) => match v.parse::<u64>() {
+                Ok(p) if p < parts => p,
+                _ => panic!("PHOTOCRAFT_SWEEP_PART={v:?} is not a part number below {parts}"),
+            },
+            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");
        Self { part, parts }