sarankk commented on code in PR #165:
URL: https://github.com/apache/cassandra-sidecar/pull/165#discussion_r1906124155
##########
server/src/test/java/org/apache/cassandra/sidecar/server/ServerTest.java:
##########
@@ -308,6 +308,16 @@ void unrecognizedAuthenticationHandlerSet()
"UnrecognizedAuthenticationHandler has not been
registered");
}
+ @Test
+ @DisplayName("Invalid access control config, unrecognized authorization
provider set")
+ void unrecognizedAuthorizationProviderSet()
+ {
+ assertThatThrownBy(() ->
configureServer("config/sidecar_unrecognized_authorizer.yaml"))
+ .hasCauseInstanceOf(RuntimeException.class)
Review Comment:
It throws dedicated exception `ConfigurationException` I was by mistake
checking for `RuntimeException` in test. Updated it
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
