Merge pull request #1313 from guswynn/master

collect all test failures before failing
This commit is contained in:
Eric Huss 2020-09-06 09:12:42 -07:00 committed by GitHub
commit 6fed9e52f9
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 6 additions and 1 deletions

View File

@ -250,6 +250,7 @@ impl MDBook {
// Index Preprocessor is disabled so that chapter paths continue to point to the // Index Preprocessor is disabled so that chapter paths continue to point to the
// actual markdown files. // actual markdown files.
let mut failed = false;
for item in book.iter() { for item in book.iter() {
if let BookItem::Chapter(ref ch) = *item { if let BookItem::Chapter(ref ch) = *item {
let chapter_path = match ch.path { let chapter_path = match ch.path {
@ -282,7 +283,8 @@ impl MDBook {
let output = cmd.output()?; let output = cmd.output()?;
if !output.status.success() { if !output.status.success() {
bail!( failed = true;
error!(
"rustdoc returned an error:\n\ "rustdoc returned an error:\n\
\n--- stdout\n{}\n--- stderr\n{}", \n--- stdout\n{}\n--- stderr\n{}",
String::from_utf8_lossy(&output.stdout), String::from_utf8_lossy(&output.stdout),
@ -291,6 +293,9 @@ impl MDBook {
} }
} }
} }
if failed {
bail!("One or more tests failed");
}
Ok(()) Ok(())
} }