mdedetrich commented on PR #11478: URL: https://github.com/apache/kafka/pull/11478#issuecomment-1201692067
> I wonder if we should also add an integration test. We should be able to extend MultipleListenersWithSameSecurityProtocolBaseTest to test duplicate listeners. There may be other existing tests which would be good candidates for adding this, if you find a better place go for it. I just did a quick glance of `MultipleListenersWithSameSecurityProtocolBaseTest` and if I understand correctly this appears to be non trivial to extend for this specific test case due to the fact that currently `MultipleListenersWithSameSecurityProtocolBaseTest` configures the endpoints to use `localhost:0` where the `0` is a magic constant which the `def validate` function deliberately filters out to bypass any kind of validation. -- 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]
