diff --git a/okhttp/src/test/java/io/grpc/okhttp/OkHttpClientTransportTest.java b/okhttp/src/test/java/io/grpc/okhttp/OkHttpClientTransportTest.java
index 6e74983240..4302e3196b 100644
--- a/okhttp/src/test/java/io/grpc/okhttp/OkHttpClientTransportTest.java
+++ b/okhttp/src/test/java/io/grpc/okhttp/OkHttpClientTransportTest.java
@@ -175,7 +175,7 @@ public class OkHttpClientTransportTest {
   private OkHttpClientTransport clientTransport;
   private final MockFrameReader frameReader = new MockFrameReader();
   private final Socket socket = new MockSocket(frameReader);
-  private FrameWriter frameWriter = mock(FrameWriter.class, AdditionalAnswers.delegatesTo(
+  private final FrameWriter frameWriter = mock(FrameWriter.class, AdditionalAnswers.delegatesTo(
       new MockFrameWriter(socket, capturedBuffer)));
   private ExecutorService executor = Executors.newCachedThreadPool();
   private long nanoTime; // backs a ticker, for testing ping round-trip time measurement
@@ -261,7 +261,7 @@ public class OkHttpClientTransportTest {
     clientTransport = new OkHttpClientTransport(
         address,
         "hostname",
-        /*agent=*/ null,
+        /*userAgent=*/ null,
         EAG_ATTRS,
         executor,
         socketFactory,
@@ -2494,10 +2494,6 @@ public class OkHttpClientTransportTest {
       this.capturedBuffer = capturedBuffer;
     }
 
-    void setSocket(Socket socket) {
-      this.socket = socket;
-    }
-
     @Override
     public void close() throws IOException {
       socket.close();