Skip to content

Commit 61f17ad

Browse files
committed
fix X509 test
Capture logs earlier to account for secure channel loading the PEM file when SSL options are set. This revealed the OpenSSL implementation also logs.
1 parent 08ad128 commit 61f17ad

File tree

1 file changed

+2
-4
lines changed

1 file changed

+2
-4
lines changed

src/libmongoc/tests/test-mongoc-x509.c

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -346,15 +346,13 @@ test_x509_auth (void *unused)
346346
bson_error_t error = {0};
347347
bool ok;
348348
{
349-
mongoc_client_t *client = test_framework_client_new_from_uri (uri, NULL);
350349
capture_logs (true);
350+
mongoc_client_t *client = test_framework_client_new_from_uri (uri, NULL);
351351
ok = try_insert (client, &error);
352-
#if defined(MONGOC_ENABLE_SSL_SECURE_TRANSPORT)
352+
#if defined(MONGOC_ENABLE_SSL_SECURE_TRANSPORT) || defined(MONGOC_ENABLE_SSL_OPENSSL)
353353
ASSERT_CAPTURED_LOG ("tls", MONGOC_LOG_LEVEL_ERROR, "Cannot find certificate");
354354
#elif defined(MONGOC_ENABLE_SSL_SECURE_CHANNEL)
355355
ASSERT_CAPTURED_LOG ("tls", MONGOC_LOG_LEVEL_ERROR, "Failed to open file");
356-
#elif defined(MONGOC_ENABLE_SSL_OPENSSL)
357-
ASSERT_NO_CAPTURED_LOGS ("tls");
358356
#endif
359357
mongoc_client_destroy (client);
360358
}

0 commit comments

Comments
 (0)