From f33d8df98ac2ec63c763a1819225c6a5cbaa5d5f Mon Sep 17 00:00:00 2001 From: Bryan Helmkamp Date: Mon, 20 Apr 2026 09:27:55 -0400 Subject: [PATCH] fix(proc): silence clippy in signal.rs test module Adds expect(disallowed_types) at the tests module for the intentional sync BufReader usage in the zombie-process-group helper, and drops the absolute-path call site by bringing pre_exec_setpgid into scope. Co-Authored-By: Claude Opus 4.7 (1M context) --- lib/crates/fabro-proc/src/signal.rs | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/lib/crates/fabro-proc/src/signal.rs b/lib/crates/fabro-proc/src/signal.rs index 04f650030..a418b6dca 100644 --- a/lib/crates/fabro-proc/src/signal.rs +++ b/lib/crates/fabro-proc/src/signal.rs @@ -146,12 +146,17 @@ pub fn sigusr2(pid: u32) { } #[cfg(test)] +#[expect( + clippy::disallowed_types, + reason = "Tests use sync std::io::BufReader to read a short-lived helper's stdout synchronously." +)] mod tests { use std::io::{BufRead, BufReader}; use std::process::{Command, Stdio}; use std::time::Duration; use super::{process_exists, process_group_alive, process_running}; + use crate::pre_exec::pre_exec_setpgid; #[test] fn process_running_returns_true_for_current_process() { @@ -199,7 +204,7 @@ mod tests { .stdin(Stdio::null()) .stdout(Stdio::null()) .stderr(Stdio::null()); - crate::pre_exec::pre_exec_setpgid(&mut child); + pre_exec_setpgid(&mut child); let mut child = child.spawn().expect("group leader should spawn"); let pgid = child.id();