devmadhuu commented on code in PR #6503:
URL: https://github.com/apache/ozone/pull/6503#discussion_r1583177483


##########
hadoop-ozone/recon/src/test/java/org/apache/hadoop/ozone/recon/api/TestNSSummaryEndpointWithLegacy.java:
##########
@@ -134,6 +134,7 @@ public class TestNSSummaryEndpointWithLegacy {
   private static final String BUCKET_TWO = "bucket2";
   private static final String BUCKET_THREE = "bucket3";
   private static final String BUCKET_FOUR = "bucket4";
+  private static final String BUCKET_FIVE = "bucket5";

Review Comment:
   > Are we asserting anything in the test class 
`TestNSSummaryEndpointWithLegacy`? We have created a bunch of new files and 
directories but have not asserted anything yet.
   
   I added the test case for both legacy and obs buckets in 
`TestNSSummaryEndpointWithOBSAndLegacy` class. Will remove the tree structure 
from here.



-- 
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]

Reply via email to