JyotinderSingh commented on pull request #2832: URL: https://github.com/apache/ozone/pull/2832#issuecomment-966841796
> Thanks @JyotinderSingh for the contribution. > > Can you please automate/add test case to verify the behavior. Below is one idea for test verification: > > step-1) Add assert statement in [TestRootedOzoneFileSystem.java - line no# 432](https://github.com/apache/ozone/blob/master/hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/fs/ozone/TestRootedOzoneFileSystem.java#L432) `Assert.assertEquals(bucketLayout, ozoneBucket.getBucketLayout());` > > step-2) Presently [TestRootedOzoneFileSystemWithFSO#testMkdirNonExistentVolumeBucket](https://github.com/apache/ozone/blob/master/hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/fs/ozone/TestRootedOzoneFileSystemWithFSO.java#L97) is ignored and this has been implemented now. Please delete this overridden method from TestRootedOzoneFileSystemWithFSO.java class, so that it will run along with other tests. Thanks for the feedback @rakeshadr. I have added the assertion to verify bucket layout and unblocked testMkdirNonExistentVolumeBucket. -- 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]
