Skip to content

Commit

Permalink
Rollup merge of rust-lang#93600 - last-partizan:fix-junit-formatter, …
Browse files Browse the repository at this point in the history
…r=yaahc

fix: Remove extra newlines from junit output

This PR fixes extra newline in junit output rust-lang#93454
  • Loading branch information
JohnTitor committed Feb 3, 2022
2 parents 7712dfd + 088474a commit 95efb2b
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions library/test/src/formatters/junit.rs
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,6 @@ impl<T: Write> OutputFormatter for JunitFormatter<T> {
_shuffle_seed: Option<u64>,
) -> io::Result<()> {
// We write xml header on run start
self.out.write_all(b"\n")?;
self.write_message("<?xml version=\"1.0\" encoding=\"UTF-8\"?>")
}

Expand Down Expand Up @@ -138,7 +137,7 @@ impl<T: Write> OutputFormatter for JunitFormatter<T> {
self.write_message("</testsuite>")?;
self.write_message("</testsuites>")?;

self.out.write_all(b"\n\n")?;
self.out.write_all(b"\n")?;

Ok(state.failed == 0)
}
Expand Down

0 comments on commit 95efb2b

Please sign in to comment.