KAFKA-17555 uncomment all checks of testCommonNameLoggingTrustManagerMixValidAndInvalidCertificates (#17220)

Reviewers: TengYao Chi <kitingiao@gmail.com>, Chia-Ping Tsai <chia7712@gmail.com>
This commit is contained in:
xijiu 2024-09-20 00:00:36 +08:00 committed by GitHub
parent 14c45bed5a
commit e90b246002
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 3 additions and 3 deletions

View File

@ -469,17 +469,17 @@ public class CommonNameLoggingTrustManagerFactoryWrapperTest {
CommonNameLoggingTrustManager testTrustManager = new CommonNameLoggingTrustManager(origTrustManager, 2);
// Call with valid certificate
//assertDoesNotThrow(() -> testTrustManager.checkClientTrusted(validChainWithoutCa, "RSA"));
assertDoesNotThrow(() -> testTrustManager.checkClientTrusted(validChainWithoutCa, "RSA"));
// Call with invalid certificate
assertThrows(CertificateException.class,
() -> testTrustManager.checkClientTrusted(invalidChainWithoutCa, "RSA"));
// Call with valid certificate again
//assertDoesNotThrow(() -> testTrustManager.checkClientTrusted(validChainWithoutCa, "RSA"));
assertDoesNotThrow(() -> testTrustManager.checkClientTrusted(validChainWithoutCa, "RSA"));
// Call with invalid certificate
assertThrows(CertificateException.class,
() -> testTrustManager.checkClientTrusted(invalidChainWithoutCa, "RSA"));
// Call with valid certificate again
//assertDoesNotThrow(() -> testTrustManager.checkClientTrusted(validChainWithoutCa, "RSA"));
assertDoesNotThrow(() -> testTrustManager.checkClientTrusted(validChainWithoutCa, "RSA"));
}
}