shasank112001 commented on code in PR #22692:
URL: https://github.com/apache/pulsar/pull/22692#discussion_r1596917883
##########
pulsar-testclient/src/test/java/org/apache/pulsar/testclient/PerfClientUtilsTest.java:
##########
@@ -77,7 +78,7 @@ public void testClientCreation() throws Exception {
final ClientConfigurationData conf =
builder.getClientConfigurationData();
Assert.assertTrue(conf.isTlsHostnameVerificationEnable());
- Assert.assertEquals(conf.getAuthPluginClassName(),
MyAuth.class.getName());
+ Assert.assertEquals(conf.getAuthentication().getClass(), MyAuth.class);
Review Comment:
That's my bad, I wanted to make that change on the adminCreation test case,
because the PulsarAdminBuilder doesn't store the authPluginClassName in the
ClientConfigurationData object, rather just gets the instance of the
Authentication and keeps that. I must have messed up the line numbers for the
change.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
