review comment refactor.

Author: saxenapranav

hadoop-tools/hadoop-azure/src/main/java/org/apache/hadoop/fs/azurebfs/services/AbfsHttpOperation.java

@@ -347,9 +347,7 @@ public void sendRequest(byte[] buffer, int offset, int length) throws IOExceptio
         if (expectHeader != null && expectHeader.equals(HUNDRED_CONTINUE)
             && e instanceof ProtocolException
             && EXPECT_100_JDK_ERROR.equals(e.getMessage())) {
-          LOG.debug(
-              "Getting output stream failed with expect header enabled, returning back ",
-              e);
+          LOG.debug("Getting output stream failed with expect header enabled, returning back ", e);
           expect100failureReceived = true;
           return;
         } else {

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

@@ -165,27 +165,30 @@ public void testExpect100ContinueFailureInAppend() throws Exception {
         (AbfsOutputStream) fs.create(path).getWrappedStream());
     AbfsClient spiedClient = Mockito.spy(os.getClient());
     AbfsHttpOperation[] httpOpForAppendTest = new AbfsHttpOperation[2];
-    readyMocksForAppendTest(httpOpForAppendTest, spiedClient);
+    mockSetupForAppend(httpOpForAppendTest, spiedClient);
     Mockito.doReturn(spiedClient).when(os).getClient();
     fs.delete(path, true);
     os.write(1);
     LambdaTestUtils.intercept(FileNotFoundException.class, () -> {
       os.close();
     });
     Assertions.assertThat(httpOpForAppendTest[0].getExpect100failureReceived())
-        .isTrue();
+        .describedAs("First try from AbfsClient will have expect-100 "
+            + "header and should fail with expect-100 error.").isTrue();
     Mockito.verify(httpOpForAppendTest[0], Mockito.times(0))
         .processConnHeadersAndInputStreams(Mockito.any(byte[].class),
             Mockito.anyInt(), Mockito.anyInt());
 
     Assertions.assertThat(httpOpForAppendTest[1].getExpect100failureReceived())
-        .isFalse();
+        .describedAs("The retried operation from AbfsClient should not "
+            + "fail with expect-100 error. The retried operation does not have"
+            + "expect-100 header.").isFalse();
     Mockito.verify(httpOpForAppendTest[1], Mockito.times(1))
         .processConnHeadersAndInputStreams(Mockito.any(byte[].class),
             Mockito.anyInt(), Mockito.anyInt());
   }
 
-  private void readyMocksForAppendTest(final AbfsHttpOperation[] httpOpForAppendTest,
+  private void mockSetupForAppend(final AbfsHttpOperation[] httpOpForAppendTest,
       final AbfsClient spiedClient) {
     int[] index = new int[1];
     index[0] = 0;