slfan1989 commented on code in PR #7757:
URL: https://github.com/apache/hadoop/pull/7757#discussion_r2179392761


##########
hadoop-tools/hadoop-azure-datalake/src/test/java/org/apache/hadoop/fs/adl/live/TestAdlPermissionLive.java:
##########
@@ -71,22 +73,23 @@ public static Collection adlCreateNonRecursiveTestData()
     return datas;
   }
 
-  @AfterClass
+  @AfterAll
   public static void cleanUp() throws IOException, URISyntaxException {
     if (AdlStorageConfiguration.isContractTestEnabled()) {
-      Assert.assertTrue(AdlStorageConfiguration.createStorageConnector()
+      assertTrue(AdlStorageConfiguration.createStorageConnector()
           .delete(testRoot, true));
     }
   }
 
-  @Before

Review Comment:
   @anujmodi2021 Thank you for your question! ParameterizedTest in JUnit 5 is a 
special type of unit test. Unlike JUnit 4, ParameterizedTest requires the 
initialization of certain parameters directly in the test method, so the setup 
function is executed at that point. If we add `@BeforeEach`, the variables in 
`setup` would not be initialized yet. Therefore, we don't need to add 
`@BeforeEach`; instead, we should call `setup` within 
`initTestAdlPermissionLive`.



-- 
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: common-issues-unsubscr...@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org

Reply via email to