From 52d7e5e1eb3574d40a4666e577364fbbea36d913 Mon Sep 17 00:00:00 2001 From: Levi Neuwirth Date: Thu, 30 Jul 2026 21:16:29 -0400 Subject: [PATCH] style(journey): satisfy clippy::manual_assert in the arming gate `-D warnings` with pedantic rejects an `if`-then-`panic!`. Same semantics: the arming variable only makes a missing binary fatal, it never decides whether the pin runs. Co-Authored-By: Claude Opus 5 Claude-Session: https://claude.ai/code/session_01Lv428Fth9LRtffwJSsqH7T --- tests/journey_acceptance.rs | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/tests/journey_acceptance.rs b/tests/journey_acceptance.rs index 9e7cb1c..c7b0569 100644 --- a/tests/journey_acceptance.rs +++ b/tests/journey_acceptance.rs @@ -1503,9 +1503,10 @@ fn journey_step9_the_prompt_runs_in_the_directory_it_captured() { #[test] fn journey_step9_the_offered_command_builds_the_project() { if !binary_available("cargo") { - if std::env::var_os("PMACS_REQUIRE_CARGO_BUILD").is_some() { - panic!("PMACS_REQUIRE_CARGO_BUILD is set but `cargo` is not on PATH"); - } + assert!( + std::env::var_os("PMACS_REQUIRE_CARGO_BUILD").is_none(), + "PMACS_REQUIRE_CARGO_BUILD is set but `cargo` is not on PATH" + ); eprintln!("skipping: `cargo` is not on PATH"); return; }