diff options
Diffstat (limited to 'src/tests')
| -rw-r--r-- | src/tests/test_certstor.cpp | 4 | ||||
| -rw-r--r-- | src/tests/test_ffi.cpp | 3 | ||||
| -rw-r--r-- | src/tests/test_ocsp.cpp | 6 | ||||
| -rw-r--r-- | src/tests/test_x509_path.cpp | 4 |
4 files changed, 8 insertions, 9 deletions
diff --git a/src/tests/test_certstor.cpp b/src/tests/test_certstor.cpp index 185c6a64b..c05772561 100644 --- a/src/tests/test_certstor.cpp +++ b/src/tests/test_certstor.cpp @@ -334,7 +334,7 @@ Test::Result test_certstor_find_hash_subject(const std::vector<CertificateAndKey Test::Result test_certstor_load_allcert() { Test::Result result("Certificate Store - Load every cert of every files"); - // test_dir_bundled dir should contain only one file with 2 certificates + // test_dir_bundled dir should contain only one file with 2 certificates // concatenated (ValidCert and root) const std::string test_dir_bundled = Test::data_dir() + "/x509/misc/bundledcertdir"; @@ -343,7 +343,7 @@ Test::Result test_certstor_load_allcert() result.test_note("load certs from dir: " + test_dir_bundled); // Certificate_Store_In_Memory constructor loads every cert of every files of the dir. Botan::Certificate_Store_In_Memory store(test_dir_bundled); - + // X509_Certificate constructor loads only the first certificate found in the file. Botan::X509_Certificate root_cert(Test::data_dir() + "/x509/x509test/root.pem"); Botan::X509_Certificate valid_cert(Test::data_dir() + "/x509/x509test/ValidCert.pem"); diff --git a/src/tests/test_ffi.cpp b/src/tests/test_ffi.cpp index 58b491a9a..ce9dc94d5 100644 --- a/src/tests/test_ffi.cpp +++ b/src/tests/test_ffi.cpp @@ -1237,7 +1237,7 @@ class FFI_Unit_Tests final : public Test str_len = sizeof(str_buf); TEST_FFI_OK(botan_mp_to_str, (x, 10, str_buf, &str_len)); result.test_eq("botan_mp_add", std::string(str_buf), "259"); - + TEST_FFI_RC(1, botan_mp_is_odd, (x)); TEST_FFI_RC(0, botan_mp_is_even, (x)); TEST_FFI_RC(0, botan_mp_is_negative, (x)); @@ -2708,4 +2708,3 @@ BOTAN_REGISTER_TEST("ffi", FFI_Unit_Tests); } } - diff --git a/src/tests/test_ocsp.cpp b/src/tests/test_ocsp.cpp index 3b681e193..753e4455b 100644 --- a/src/tests/test_ocsp.cpp +++ b/src/tests/test_ocsp.cpp @@ -11,7 +11,7 @@ #include <botan/x509path.h> #include <botan/certstor.h> #include <botan/calendar.h> - #include <botan/cert_status.h> + #include <botan/cert_status.h> #include <fstream> #endif @@ -164,7 +164,7 @@ class OCSP_Tests final : public Test const std::vector<std::shared_ptr<const Botan::X509_Certificate>> cert_path = { ee, ca, trust_root }; - std::shared_ptr<const Botan::OCSP::Response> ocsp = + std::shared_ptr<const Botan::OCSP::Response> ocsp = std::make_shared<const Botan::OCSP::Response>(Botan::Certificate_Status_Code::OSCP_NO_REVOCATION_URL); Botan::Certificate_Store_In_Memory certstore; @@ -177,7 +177,7 @@ class OCSP_Tests final : public Test if(result.test_eq("Expected size of ocsp_status", ocsp_status.size(), 1)) { if(result.test_eq("Expected size of ocsp_status[0]", ocsp_status[0].size(), 1)) - { + { result.confirm("Status warning", ocsp_status[0].count(Botan::Certificate_Status_Code::OSCP_NO_REVOCATION_URL)); } } diff --git a/src/tests/test_x509_path.cpp b/src/tests/test_x509_path.cpp index e86c7b70f..0e9f8eba0 100644 --- a/src/tests/test_x509_path.cpp +++ b/src/tests/test_x509_path.cpp @@ -135,7 +135,7 @@ class X509test_Path_Validation_Tests final : public Test Botan::Path_Validation_Result path_result = Botan::x509_path_validate( certs, restrictions, trusted, "www.tls.test", Botan::Usage_Type::TLS_SERVER_AUTH, - validation_time, + validation_time, /* activate check_ocsp_online */ std::chrono::milliseconds(1000), {}); if(path_result.successful_validation() && path_result.trust_root() != root) @@ -152,7 +152,7 @@ class X509test_Path_Validation_Tests final : public Test result.test_eq("test warnings string", path_result.warnings_string(), "[0] OCSP URL not available"); #endif result.end_timer(); - results.push_back(result); + results.push_back(result); } return results; |