yashmayya commented on PR #14361: URL: https://github.com/apache/kafka/pull/14361#issuecomment-1713326240
Thanks Sagar! > a test which would have invoked the older end point and validates that the warning log line gets printed Feels like a bit of overkill, but I guess it doesn't really hurt to add such a test so I've added one. There's currently no integration test class for Connect's REST API - most of the functionality is tested indirectly through other ITs and some of the more specialized ones have their own test classes (`ConnectorRestartApiIntegrationTest`, `ConnectorTopicsIntegrationTest`, `OffsetsApiIntegrationTest` etc.). It didn't seem worth it to introduce a new integration test class just for this small test so I've added it to `ConnectWorkerIntegrationTest` which is the most generic one. -- 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: jira-unsubscr...@kafka.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
