fapifta commented on a change in pull request #2814:
URL: https://github.com/apache/ozone/pull/2814#discussion_r748781771
##########
File path:
hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/fs/ozone/TestRootedOzoneFileSystem.java
##########
@@ -1513,4 +1529,65 @@ public void testRenameDestinationParentDoesntExist()
throws Exception {
}
}
+ @Test
+ public void testNonPrivilegedUserMkdirCreateBucket() throws IOException {
+ // This test is only meaningful when ACL is enabled
+ if (!enableAcl) {
Review comment:
I think we should use org.junit.Assume.assumeTrue(enableAcl) here to
avoid the test run in cases where acl-s are disabled. I think it is a more
standard solution to avoid test runs in case of a specific parameter value.
--
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]
