diff options
| author | Matt Caswell <matt@openssl.org> | 2019-05-31 15:34:13 +0200 |
|---|---|---|
| committer | Matt Caswell <matt@openssl.org> | 2019-06-03 14:27:11 +0200 |
| commit | cb30e46b45589166a83703f923aa89881dab1e0c (patch) | |
| tree | 42399cadaa74f7a2fc93905c0006e4e41b73c0ef /test/testutil | |
| parent | Ensure we get all the right defines for AES assembler in FIPS module (diff) | |
| download | openssl-cb30e46b45589166a83703f923aa89881dab1e0c.tar.xz openssl-cb30e46b45589166a83703f923aa89881dab1e0c.zip | |
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 <paul.dale@oracle.com>
(Merged from https://github.com/openssl/openssl/pull/9056)
Diffstat (limited to 'test/testutil')
| -rw-r--r-- | test/testutil/driver.c | 2 |
1 files changed, 2 insertions, 0 deletions
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; |