JyotinderSingh commented on a change in pull request #3109:
URL: https://github.com/apache/ozone/pull/3109#discussion_r810510601
##########
File path:
hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/fs/ozone/TestRootedOzoneFileSystem.java
##########
@@ -918,10 +926,20 @@ public void testListStatusRootAndVolumeContinuation()
throws IOException {
fileStatusesLimit1[fileStatusesLimit1.length - 1].getPath().toString();
FileStatus[] fileStatusesLimit2 = customListStatus(new Path("/"),
false, nextStartPath, 3);
+
// Note: at the time of writing this test,
OmMetadataManagerImpl#listVolumes
// excludes startVolume (startPath) from the result. Might change.
- Assert.assertEquals(fileStatusesOver.length,
- fileStatusesLimit1.length + fileStatusesLimit2.length);
+ if (enableAcl) {
+ // 1 is added to actual number of fileStatuses due to volume created by
+ // userXXXXX in initClusterAndEnv being included in the result when ACL
+ // is enabled.
+ Assert.assertEquals(fileStatusesOver.length,
+ fileStatusesLimit1.length + fileStatusesLimit2.length + 1);
+ } else {
+ Assert.assertEquals(fileStatusesOver.length,
+ fileStatusesLimit1.length + fileStatusesLimit2.length);
+ }
Review comment:
@rakeshadr
This is a behavior change in the test, could you review this?
--
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]
