auto merge of #9603 : sfackler/rust/rustpkg-test, r=catamorphism
Previously, if tests failed, you'd only get stderr which isn't very useful, especially if the failure didn't happen directly in a test function (e.g None.unwrap()).
This commit is contained in:
commit
727d4c306c
|
@ -620,14 +620,8 @@ impl CtxMethods for BuildContext {
|
||||||
match built_test_in_workspace(pkgid, workspace) {
|
match built_test_in_workspace(pkgid, workspace) {
|
||||||
Some(test_exec) => {
|
Some(test_exec) => {
|
||||||
debug!("test: test_exec = %s", test_exec.to_str());
|
debug!("test: test_exec = %s", test_exec.to_str());
|
||||||
let p_output = run::process_output(test_exec.to_str(), [~"--test"]);
|
let status = run::process_status(test_exec.to_str(), [~"--test"]);
|
||||||
if p_output.status == 0 {
|
os::set_exit_status(status);
|
||||||
println(str::from_utf8(p_output.output));
|
|
||||||
}
|
|
||||||
else {
|
|
||||||
println(str::from_utf8(p_output.error));
|
|
||||||
}
|
|
||||||
os::set_exit_status(p_output.status);
|
|
||||||
}
|
}
|
||||||
None => {
|
None => {
|
||||||
error(fmt!("Internal error: test executable for package ID %s in workspace %s \
|
error(fmt!("Internal error: test executable for package ID %s in workspace %s \
|
||||||
|
|
Loading…
Reference in New Issue