Missed Chages

Author: Anuj Modi

hadoop-tools/hadoop-azure/src/test/java/org/apache/hadoop/fs/azurebfs/ITestAzureBlobFileSystemAuthorization.java

@@ -36,6 +36,7 @@
 import org.apache.hadoop.fs.permission.FsAction;
 import org.apache.hadoop.fs.permission.FsPermission;
 
+import static org.apache.hadoop.fs.azurebfs.constants.ConfigurationKeys.FS_AZURE_ACCOUNT_AUTH_TYPE_PROPERTY_NAME;
 import static org.apache.hadoop.fs.azurebfs.constants.ConfigurationKeys.FS_AZURE_ACCOUNT_IS_HNS_ENABLED;
 import static org.apache.hadoop.fs.azurebfs.constants.ConfigurationKeys.FS_AZURE_SAS_TOKEN_PROVIDER_TYPE;
 import static org.apache.hadoop.fs.azurebfs.constants.TestConfigurationKeys.MOCK_SASTOKENPROVIDER_FAIL_INIT;
@@ -68,7 +69,7 @@ public void setup() throws Exception {
     Assume.assumeTrue(isHNSEnabled);
     loadConfiguredFileSystem();
     this.getConfiguration().set(FS_AZURE_SAS_TOKEN_PROVIDER_TYPE, TEST_AUTHZ_CLASS);
-    this.getConfiguration().set(ConfigurationKeys.FS_AZURE_ACCOUNT_AUTH_TYPE_PROPERTY_NAME, "SAS");
+    this.getConfiguration().set(FS_AZURE_ACCOUNT_AUTH_TYPE_PROPERTY_NAME, "SAS");
     super.setup();
   }
 
@@ -80,6 +81,7 @@ public void testSASTokenProviderInitializeException() throws Exception {
     Configuration testConfig = new Configuration(this.getConfiguration().getRawConfiguration());
     testConfig.set(FS_AZURE_SAS_TOKEN_PROVIDER_TYPE, TEST_ERR_AUTHZ_CLASS);
     testConfig.set(MOCK_SASTOKENPROVIDER_RETURN_EMPTY_SAS_TOKEN, "true");
+    // Setting IS_HNS_ENABLED to avoid the exception thrown by the HNS check.
     testConfig.set(FS_AZURE_ACCOUNT_IS_HNS_ENABLED, this.getIsNamespaceEnabled(fs) + "");
 
     intercept(SASTokenProviderException.class,
@@ -96,6 +98,7 @@ public void testSASTokenProviderEmptySASToken() throws Exception {
     Configuration testConfig = new Configuration(this.getConfiguration().getRawConfiguration());
     testConfig.set(FS_AZURE_SAS_TOKEN_PROVIDER_TYPE, TEST_ERR_AUTHZ_CLASS);
     testConfig.set(MOCK_SASTOKENPROVIDER_RETURN_EMPTY_SAS_TOKEN, "true");
+    // Setting IS_HNS_ENABLED to avoid the exception thrown by the HNS check.
     testConfig.set(FS_AZURE_ACCOUNT_IS_HNS_ENABLED, this.getIsNamespaceEnabled(fs) + "");
 
     testFs.initialize(fs.getUri(), testConfig);
@@ -113,6 +116,7 @@ public void testSASTokenProviderNullSASToken() throws Exception {
     Configuration testConfig = new Configuration(this.getConfiguration().getRawConfiguration());
     testConfig.set(FS_AZURE_SAS_TOKEN_PROVIDER_TYPE, TEST_ERR_AUTHZ_CLASS);
     testConfig.set(MOCK_SASTOKENPROVIDER_RETURN_EMPTY_SAS_TOKEN, "true");
+    // Setting IS_HNS_ENABLED to avoid the exception thrown by the HNS check.
     testConfig.set(FS_AZURE_ACCOUNT_IS_HNS_ENABLED, this.getIsNamespaceEnabled(fs) + "");
 
     testFs.initialize(fs.getUri(), testConfig);