test: fix ErrorProne MissingFail warnings in unit tests

Author: kannanjgithub

core/src/test/java/io/grpc/internal/DelayedClientCallTest.java

@@ -36,6 +36,7 @@
 import io.grpc.Metadata;
 import io.grpc.Status;
 import io.grpc.StatusException;
+import static org.junit.Assert.assertThrows;
 import static org.junit.Assert.fail;
 import java.lang.reflect.Method;
 import java.lang.reflect.Modifier;
@@ -233,6 +234,7 @@ public void delayedCallsRunUnderContext() throws Exception {
   }
 
   @Test
+  @SuppressWarnings("MissingFail")
   public void drainPendingCallFails() {
     DelayedClientCall<String, Integer> delayedClientCall =
         new DelayedClientCall<>(callExecutor, fakeClock.getScheduledExecutorService(), null);
@@ -259,6 +261,7 @@ public Void answer(org.mockito.invocation.InvocationOnMock invocation) {
   }
 
   @Test
+  @SuppressWarnings("unchecked")
   public void drainPendingCallbacksFails() {
     DelayedClientCall<String, Integer> delayedClientCall =
         new DelayedClientCall<>(callExecutor, fakeClock.getScheduledExecutorService(), null);
@@ -303,6 +306,7 @@ public Void answer(org.mockito.invocation.InvocationOnMock invocation) {
     } catch (RuntimeException e) {
       assertThat(e).isSameInstanceAs(error);
     }
+
     // Verify it was called twice (once during drain, once just now)
     verify(listener, times(2)).onReady();
   }

core/src/test/java/io/grpc/internal/DelayedStreamTest.java

@@ -21,6 +21,7 @@
 import static org.junit.Assert.assertFalse;
 import static org.junit.Assert.assertNotNull;
 import static org.junit.Assert.assertNull;
+import static org.junit.Assert.assertThrows;
 import static org.junit.Assert.assertTrue;
 import static org.junit.Assert.fail;
 import static org.mockito.ArgumentMatchers.any;
@@ -474,6 +475,7 @@ public Void answer(InvocationOnMock in) {
   }
 
   @Test
+  @SuppressWarnings({"unchecked", "MissingFail"})
   public void drainPendingCallFails() {
     stream.start(listener);
     stream.request(1);
@@ -504,6 +506,7 @@ public Void answer(InvocationOnMock invocation) {
   }
 
   @Test
+  @SuppressWarnings("unchecked")
   public void drainPendingCallbacksFails() {
     stream.start(listener);
     final RuntimeException error = new RuntimeException("fail");
@@ -544,6 +547,7 @@ public Void answer(InvocationOnMock invocation) {
     } catch (RuntimeException e) {
       assertThat(e).isSameInstanceAs(error);
     }
+
     // Verify it was called twice (once during drain, once just now)
     verify(listener, times(2)).onReady();
   }