From 9aafb64849d4339f5c3e0b31ea4ead51cf20dca4 Mon Sep 17 00:00:00 2001 From: David Benjamin Date: Tue, 20 Sep 2016 19:36:53 -0400 Subject: [PATCH] Don't swallow tool output on failure. If the test failed due to non-ASan reasons but ASan also had errors, output those too. Change-Id: Id908fe2a823c59255c6a9585dfaa894a4fcd9f59 Reviewed-on: https://boringssl-review.googlesource.com/11127 Reviewed-by: Adam Langley --- ssl/test/runner/runner.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ssl/test/runner/runner.go b/ssl/test/runner/runner.go index 0523042d..7b45e887 100644 --- a/ssl/test/runner/runner.go +++ b/ssl/test/runner/runner.go @@ -988,7 +988,7 @@ func runTest(test *testCase, shimPath string, mallocNumToFail int64) error { panic("internal error") } - return fmt.Errorf("%s: local error '%s', child error '%s', stdout:\n%s\nstderr:\n%s", msg, localError, childError, stdout, stderr) + return fmt.Errorf("%s: local error '%s', child error '%s', stdout:\n%s\nstderr:\n%s\n%s", msg, localError, childError, stdout, stderr, extraStderr) } if !*useValgrind && (len(extraStderr) > 0 || (!failed && len(stderr) > 0)) {