diff options
author | Matt Caswell <matt@openssl.org> | 2020-01-30 15:30:17 +0000 |
---|---|---|
committer | Matt Caswell <matt@openssl.org> | 2020-02-03 11:41:56 +0000 |
commit | 8d242823ed2270e2907914fb09004ae30263fb00 (patch) | |
tree | bbcfc0f2c0b01536b3dd2018ff4244c6e5331ccb /test/cmp_msg_test.c | |
parent | ef071222020be2096fb9f3aaef8bfe18ae9a40c9 (diff) |
Fix common test framework options
PR#6975 added the ability to our test framework to have common options to
all tests. For example providing the option "-test 5" to one of our test
programs will just run test number 5. This can be useful when debugging
tests.
Unforuntately this does not work well for a number of tests. In particular
those tests that call test_get_argument() without first skipping over these
common test options will not get the expected value. Some tests did this
correctly but a large number did not.
A helper function is introduced, test_skip_common_options(), to make this
easier for those tests which do not have their own specialised test option
handling, but yet still need to call test_get_argument(). This function
call is then added to all those tests that need it.
Reviewed-by: Shane Lontis <shane.lontis@oracle.com>
Reviewed-by: Paul Dale <paul.dale@oracle.com>
(Merged from https://github.com/openssl/openssl/pull/10975)
Diffstat (limited to 'test/cmp_msg_test.c')
-rw-r--r-- | test/cmp_msg_test.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/test/cmp_msg_test.c b/test/cmp_msg_test.c index 463c60789b..7fa0619284 100644 --- a/test/cmp_msg_test.c +++ b/test/cmp_msg_test.c @@ -538,6 +538,11 @@ void cleanup_tests(void) int setup_tests(void) { + if (!test_skip_common_options()) { + TEST_error("Error parsing test options\n"); + return 0; + } + if (!TEST_ptr(server_cert_f = test_get_argument(0)) || !TEST_ptr(pkcs10_f = test_get_argument(1))) { TEST_error("usage: cmp_msg_test server.crt pkcs10.der\n"); |