xd009642 / llvm-profparser / #213

69%

push

Give a better error message if llvm-profdata isn't installed (#55)

Previously, the test failed to check if the command actually succeeded,
and so would run assertions on the command's output even if it failed.

Before:
```
$ cargo test --test profdata -- profraws
LLVM tools failed:
error: no such command: `profdata`

help: view all installed commands with `cargo --list`
help: find a package to install `profdata` with `cargo search
cargo-profdata`

"c-general.profraw"
LLVM tools failed:
error: no such command: `profdata`

help: view all installed commands with `cargo --list`
help: find a package to install `profdata` with `cargo search
cargo-profdata`

"compressed.profraw"

<... repeated for each failing test ...>
```

After:
```
$ cargo test --test profdata -- profdatas
---- show_profdatas stdout ----

thread 'show_profraws' panicked at
/rustc/ec7c02612/library/core/src/ops/function.rs:253:5:
Unexpected failure.
code=101
stderr=```"error: No such file or directory (os error 2)\n"```
command=`"cargo" "profdata" "--" "--version"`
help=`run 'rustup component add llvm-tools-preview'`
code=101
stdout=""
stderr="error: No such file or directory (os error 2)\n"

note: run with `RUST_BACKTRACE=1` environment variable to display a
backtrace
```

16 of 16 new or added lines in 1 file covered. (100.0%)

1213 of 1748 relevant lines covered (69.39%)

3.42 hits per line