Fixed findingbugs

Author: chiacyu

hadoop-hdds/client/src/test/java/org/apache/hadoop/hdds/scm/storage/DummyBlockInputStreamWithRetry.java

@@ -40,6 +40,7 @@
  * A dummy BlockInputStream with pipeline refresh function to mock read
  * block call to DN.
  */
+@SuppressWarnings("java:S1854")
 final class DummyBlockInputStreamWithRetry
     extends DummyBlockInputStream {
 

hadoop-hdds/client/src/test/java/org/apache/hadoop/hdds/scm/storage/TestBlockInputStream.java

@@ -293,14 +293,15 @@ public void testRefreshPipelineFunction() throws Exception {
 
   @ParameterizedTest
   @MethodSource("exceptionsTriggersRefresh")
+  @SuppressWarnings("java:S1854")
   void refreshesPipelineOnReadFailure(IOException ex) throws Exception {
     // GIVEN
     Pipeline pipeline = MockPipeline.createSingleNodePipeline();
     BlockLocationInfo blockLocationInfo = mock(BlockLocationInfo.class);
-    doReturn(pipeline).when(blockLocationInfo).getPipeline();
     Pipeline newPipeline = MockPipeline.createSingleNodePipeline();
     BlockLocationInfo newBlockLocationInfo = mock(BlockLocationInfo.class);
 
+    doReturn(pipeline).when(blockLocationInfo).getPipeline();
     testRefreshesPipelineOnReadFailure(ex, blockLocationInfo,
         id -> newBlockLocationInfo);
 
@@ -406,6 +407,7 @@ public void testReadNotRetriedOnOtherException(IOException ex)
 
   @ParameterizedTest
   @MethodSource("exceptionsTriggersRefresh")
+  @SuppressWarnings("java:S1854")
   public void testRefreshOnReadFailureAfterUnbuffer(IOException ex)
       throws Exception {
     // GIVEN

hadoop-hdds/client/src/test/java/org/apache/hadoop/ozone/client/io/TestBlockInputStreamFactoryImpl.java

@@ -51,6 +51,7 @@ public class TestBlockInputStreamFactoryImpl {
   private OzoneConfiguration conf = new OzoneConfiguration();
 
   @Test
+  @SuppressWarnings("java:S1854")
   public void testNonECGivesBlockInputStream() throws IOException {
     BlockInputStreamFactory factory = new BlockInputStreamFactoryImpl();
     ReplicationConfig repConfig =