From cb30e46b45589166a83703f923aa89881dab1e0c Mon Sep 17 00:00:00 2001 From: Matt Caswell Date: Fri, 31 May 2019 14:34:13 +0100 Subject: [PATCH] Really fail if we have a test failure Commit c5f7a99645a broke the test framework such that some tests might fail, but the test framework still gives a PASS result overall. Reviewed-by: Paul Dale (Merged from https://github.com/openssl/openssl/pull/9056) --- test/testutil/driver.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/test/testutil/driver.c b/test/testutil/driver.c index 2da83ed727..7a1a5893b3 100644 --- a/test/testutil/driver.c +++ b/test/testutil/driver.c @@ -353,6 +353,8 @@ int run_tests(const char *test_prog_name) verdict = all_tests[i].test_fn(); test_verdict(verdict, "%d - %s", ii + 1, test_title); finalize(verdict != 0); + if (verdict == 0) + num_failed++; } else { int num_failed_inner = 0; -- 2.39.5