ArafatKhan2198 commented on PR #4158: URL: https://github.com/apache/ozone/pull/4158#issuecomment-1422215432
@devmadhuu @adoroszlai The `TestReconWithOzoneManager` test class verifies information from the OzoneManager and is designed specifically for this purpose. Adding SCM as a new source in this class would not align with its original intent. Making significant changes to the code for testing a single endpoint undermines the purpose of the class. Fortunately, the functionality we aim to test is already covered by `TestContainerEndpoint`. Given that there is currently no class for handling similar integration testing for SCM recon, I suggest creating a new JIRA to do integration tests on all endpoints that utilize SCM data. This work falls outside the scope of the present JIRA and is better suited for separate consideration. I have already tested my code changes in TestContainerEndpoint, which should meet our needs for the time being. -- 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]
