anmolanmol1234 commented on code in PR #7713: URL: https://github.com/apache/hadoop/pull/7713#discussion_r2115333156
########## hadoop-tools/hadoop-azure/src/test/java/org/apache/hadoop/fs/azurebfs/services/ITestAbfsClient.java: ########## @@ -355,6 +356,44 @@ public void verifyUserAgentClusterType() throws Exception { .contains(DEFAULT_VALUE_UNKNOWN); } + @Test + // Test to verify the unique identifier in user agent string for FNS-Blob accounts + public void verifyUserAgentForFNSBlob() throws Exception { + Assume.assumeTrue(JDK_HTTP_URL_CONNECTION == httpOperationType); + assumeHnsDisabled(); + Assume.assumeTrue(getAbfsServiceType() == AbfsServiceType.BLOB); + final AzureBlobFileSystem fs = getFileSystem(getRawConfiguration()); + final AbfsConfiguration configuration = fs.getAbfsStore() + .getAbfsConfiguration(); + + String userAgentStr = getUserAgentString(configuration, false); + verifyBasicInfo(userAgentStr); + Assertions.assertThat(userAgentStr) + .describedAs( + "User-Agent string for FNS accounts on Blob endpoint should contain " + + FNS_BLOB_USER_AGENT_IDENTIFIER) + .contains(FNS_BLOB_USER_AGENT_IDENTIFIER); + } + + @Test + // Test to verify that the user agent string for non-FNS-Blob accounts + // does not contain the FNS identifier. + public void verifyUserAgentForDFS() throws Exception { Review Comment: assumeHnsDisabled() as well -- 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