Skip to content

Commit 4a23d3a

Browse files
committed
Fixed findingBugs
1 parent 4f49afd commit 4a23d3a

File tree

1 file changed

+5
-3
lines changed

1 file changed

+5
-3
lines changed

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

+5-3
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,6 @@
2727
import static org.mockito.Mockito.any;
2828
import static org.mockito.Mockito.anyInt;
2929
import static org.mockito.Mockito.doReturn;
30-
import static org.mockito.Mockito.doThrow;
3130
import static org.mockito.Mockito.mock;
3231
import static org.mockito.Mockito.never;
3332
import static org.mockito.Mockito.reset;
@@ -71,6 +70,7 @@
7170
import org.junit.jupiter.params.ParameterizedTest;
7271
import org.junit.jupiter.params.provider.Arguments;
7372
import org.junit.jupiter.params.provider.MethodSource;
73+
import org.mockito.stubbing.OngoingStubbing;
7474
import org.slf4j.event.Level;
7575

7676
/**
@@ -353,9 +353,11 @@ private static Stream<Arguments> exceptionsNotTriggerRefresh() {
353353
private static ChunkInputStream throwingChunkInputStream(IOException ex,
354354
int len, boolean succeedOnRetry) throws IOException {
355355
final ChunkInputStream stream = mock(ChunkInputStream.class);
356-
doThrow(ex).doReturn(len).when(stream.read(any(), anyInt(), anyInt()));
356+
OngoingStubbing<Integer> stubbing =
357+
when(stream.read(any(), anyInt(), anyInt()))
358+
.thenThrow(ex);
357359
if (succeedOnRetry) {
358-
doReturn(len).when(stream).read(any(), anyInt(), anyInt());
360+
stubbing.thenReturn(len);
359361
}
360362
doReturn((long) len).when(stream.getRemaining());
361363
return stream;

0 commit comments

Comments
 (0)