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/d2i_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/d2i_test.c')
| -rw-r--r-- | test/d2i_test.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/test/d2i_test.c b/test/d2i_test.c index 3ce38485bd..84ba30fbcc 100644 --- a/test/d2i_test.c +++ b/test/d2i_test.c @@ -127,6 +127,11 @@ int setup_tests(void) {"compare", ASN1_COMPARE} }; + if (!test_skip_common_options()) { + TEST_error("Error parsing test options\n"); + return 0; + } + if (!TEST_ptr(test_type_name = test_get_argument(0)) || !TEST_ptr(expected_error_string = test_get_argument(1)) || !TEST_ptr(test_file = test_get_argument(2))) |