Skip to content

Commit

Permalink
Merge pull request #369 from cgwalters/one-more-assert
Browse files Browse the repository at this point in the history
tests: Another missing `assert!` change to dump stderr
  • Loading branch information
jeckersb committed Feb 28, 2024
2 parents 90b4dfb + 80c6aea commit 0072e05
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion lib/src/privtests.rs
Original file line number Diff line number Diff line change
Expand Up @@ -100,7 +100,10 @@ pub(crate) fn impl_run_container() -> Result<()> {
.output()?;
assert!(!o.status.success());
let stderr = String::from_utf8(o.stderr)?;
assert!(stderr.contains("requires root privileges"));
assert!(
stderr.contains("requires root privileges"),
"stderr: {stderr}"
);

let config = cmd!(sh, "bootc install print-configuration").read()?;
let mut config: InstallConfiguration =
Expand Down

0 comments on commit 0072e05

Please sign in to comment.