diff options
author | Pauli <pauli@openssl.org> | 2021-05-31 14:27:31 +1000 |
---|---|---|
committer | Pauli <pauli@openssl.org> | 2021-06-02 16:30:15 +1000 |
commit | fd009d763a931c4cd01f5181a2b0801d205f782a (patch) | |
tree | 1a43eeda9efa766bba6b983737b84844ee77fbe8 /test/cmp_client_test.c | |
parent | fb6ad22e36a1ade653f4b6881ddeee128e8b5001 (diff) |
test: remove TODOs
Reviewed-by: Tim Hudson <tjh@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/15539)
Diffstat (limited to 'test/cmp_client_test.c')
-rw-r--r-- | test/cmp_client_test.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/test/cmp_client_test.c b/test/cmp_client_test.c index d181a03d19..863a765886 100644 --- a/test/cmp_client_test.c +++ b/test/cmp_client_test.c @@ -101,7 +101,6 @@ static int execute_exec_GENM_ses_test(CMP_SES_TEST_FIXTURE *fixture) if (!TEST_ptr(itavs = OSSL_CMP_exec_GENM_ses(fixture->cmp_ctx))) return 0; sk_OSSL_CMP_ITAV_pop_free(itavs, OSSL_CMP_ITAV_free); - /* TODO: check if the returned value is the expected one (same as sent) */ return 1; } @@ -115,7 +114,6 @@ static int execute_exec_certrequest_ses_test(CMP_SES_TEST_FIXTURE *fixture) if (!TEST_ptr(res) || !TEST_int_eq(X509_cmp(res, client_cert), 0)) return 0; - /* TODO: check that cerfConf has been exchanged unless implicitConfirm */ if (fixture->caPubs != NULL) { STACK_OF(X509) *caPubs = OSSL_CMP_CTX_get1_caPubs(fixture->cmp_ctx); int ret = TEST_int_eq(STACK_OF_X509_cmp(fixture->caPubs, caPubs), 0); @@ -169,7 +167,6 @@ static int test_exec_IR_ses_poll(void) ossl_cmp_mock_srv_set_pollCount(fixture->srv_ctx, 2); ossl_cmp_mock_srv_set_checkAfterTime(fixture->srv_ctx, checkAfter); EXECUTE_TEST(execute_exec_certrequest_ses_test, tear_down); - /* TODO: check that 2 rounds are done or session takes 2..3 seconds */ return result; } |