michaeljmarshall opened a new pull request, #19430: URL: https://github.com/apache/pulsar/pull/19430
### Motivation While working on #19409, I noticed we do not have any tests in the `ServerCnxTests` class verifying the `authRole` cannot change. This seems like an oversight, and this PR adds tests to assert the existing behavior works as intended. ### Modifications * Add new test `AuthenticationProvider` and `AuthenticationState` implementations to simplify testing * Add two tests to verify the behavior that a connection is closed when the `authRole` changes. ### Verifying this change This change is to add tests. ### Documentation - [x] `doc-not-needed` ### Matching PR in forked repository PR in forked repository: Skipping testing in my fork since these are new tests and I got them to pass locally. -- 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]
