fixup: allow clippy lints in tests, skip unparseable build products

Address CI failures and review feedback:

- Add #[allow(clippy::unwrap_used, clippy::expect_used)] to the test
  module — the crate-level deny triggers on test helpers using unwrap().

- Replace collect::<anyhow::Result<_>>()? with filter_map + warn log
  for build product parsing.  A single corrupt DB row (e.g. legacy
  product with a non-store path) should not abort the entire cached
  build — log and skip instead, consistent with how malformed build
  outputs are already skipped via `let Ok(mut res) ... else { continue }`
  on the line above.

- Fix import ordering (rustfmt).

Author: angerman

subprojects/hydra-queue-runner/src/state/build.rs

@@ -1,7 +1,7 @@
 use std::collections::BTreeMap;
 use std::hash::Hash;
-use std::sync::Arc;
 use std::sync::atomic::{AtomicBool, AtomicI32, Ordering};
+use std::sync::Arc;
 
 use anyhow::Context;
 use hashbrown::{HashMap, HashSet};
@@ -744,6 +744,7 @@ impl Builds {
 }
 
 #[cfg(test)]
+#[allow(clippy::unwrap_used, clippy::expect_used)]
 mod tests {
     use super::*;
 

subprojects/hydra-queue-runner/src/state/mod.rs

@@ -2060,8 +2060,18 @@ impl State {
                     .get_build_products_for_build_id(build_id)
                     .await?
                     .into_iter()
-                    .map(|p| build::BuildProduct::from_db(self.store.store_dir(), p))
-                    .collect::<anyhow::Result<_>>()?;
+                    .filter_map(|p| {
+                        match build::BuildProduct::from_db(self.store.store_dir(), p) {
+                            Ok(bp) => Some(bp),
+                            Err(e) => {
+                                tracing::warn!(
+                                    "Skipping build product with unparseable path: {e:#}"
+                                );
+                                None
+                            }
+                        }
+                    })
+                    .collect();
                 res.metrics = db
                     .get_build_metrics_for_build_id(build_id)
                     .await?