rakeshadr commented on pull request #1503: URL: https://github.com/apache/ozone/pull/1503#issuecomment-719328435
> Thanks for updating the PR, @rakeshadr . One further review comment below. > > In additional, current test change not fully cover the test for listStatusV1. Example, current OzoneFS test change doesn't address the case for listStatus with other startKey specified. > I see the test class TestKeyManagerImpl does the good coverage for listStatus call, can we add a test unit like that or make a minor refactor based on that? I've created HDDS-4412 jira to refactor TestKeyManagerImpl unit testing. Today, it uses stale APIs and IMHO will refactor this in a separate task to make this patch comparatively smaller:-) ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
