From a7881a426bff05dfe1d662c52691f6295c2e3032 Mon Sep 17 00:00:00 2001 From: nils <48135649+Nilstrieb@users.noreply.github.com> Date: Tue, 18 Apr 2023 08:13:52 +0200 Subject: [PATCH] require cargo check pass in testsuite --- testsuite/src/bin/regression_checker.rs | 16 +++++++++++++++- 1 file changed, 15 insertions(+), 1 deletion(-) diff --git a/testsuite/src/bin/regression_checker.rs b/testsuite/src/bin/regression_checker.rs index 7c1c6bf..1f0a636 100644 --- a/testsuite/src/bin/regression_checker.rs +++ b/testsuite/src/bin/regression_checker.rs @@ -1,3 +1,5 @@ +use anyhow::bail; + fn main() -> anyhow::Result<()> { if std::env::var("MINIMIZE_LINTS").as_deref() == Ok("1") { std::process::Command::new("cargo") @@ -14,5 +16,17 @@ fn main() -> anyhow::Result<()> { let proj_dir = std::env::current_dir().expect("current dir not found"); - testsuite::ensure_roots_kept(&proj_dir, roots) + testsuite::ensure_roots_kept(&proj_dir, roots)?; + + let check = std::process::Command::new("cargo") + .arg("check") + .spawn() + .unwrap() + .wait() + .unwrap(); + + if !check.success() { + bail!("cargo check failed"); + } + Ok(()) }