manika137 commented on code in PR #7765:
URL: https://github.com/apache/hadoop/pull/7765#discussion_r2174128613


##########
hadoop-tools/hadoop-azure/src/test/java/org/apache/hadoop/fs/azurebfs/ITestGetNameSpaceEnabled.java:
##########
@@ -341,6 +342,66 @@ public void testAccountSpecificConfig() throws Exception {
     }
   }
 
+  /**
+   * Tests the behavior of AbfsConfiguration when the namespace-enabled
+   * configuration is not explicitly set (i.e., set to an empty string).
+   *
+   * Expects the namespace value to be set as UNKNOWN.
+   *
+   * @throws Exception if any error occurs during configuration setup or 
evaluation
+   */
+  @Test
+  public void testNameSpaceConfigNotSet() throws Exception {

Review Comment:
   Would it be better if we have a single test to verify the expected 
isnamespace enabled value for all 3 cases? We could set and reset the values 
along with the respective assertions



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