Skip to content

Commit

Permalink
Auto merge of #6364 - dwijnand:test-json-parse-expect-over-unwrap, r=…
Browse files Browse the repository at this point in the history
…alexcrichton

Use expect over unwrap, for panic-in-panic aborts

... doesn't help, but it can't hurt either, right?
  • Loading branch information
bors committed Dec 1, 2018
2 parents d27b47b + b938637 commit de1ef75
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion tests/testsuite/support/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -682,7 +682,7 @@ impl Execs {
self.expect_json = Some(
expected
.split("\n\n")
.map(|obj| obj.parse().unwrap())
.map(|line| line.parse().expect("line to be a valid JSON value"))
.collect(),
);
self
Expand Down

0 comments on commit de1ef75

Please sign in to comment.