Add javadoc. Make private

Author: jdcormie

core/src/testFixtures/java/io/grpc/internal/MockServerListener.java

@@ -24,6 +24,12 @@
 import java.util.concurrent.LinkedBlockingQueue;
 import java.util.concurrent.TimeUnit;
 
+/**
+ * A {@link ServerListener} that helps you write blocking unit tests.
+ *
+ * <p>TODO: Rename, since this is not actually a mock:
+ * https://testing.googleblog.com/2013/07/testing-on-toilet-know-your-test-doubles.html
+ */
 public class MockServerListener implements ServerListener {
   private final BlockingQueue<MockServerTransportListener> listeners = new LinkedBlockingQueue<>();
   private final SettableFuture<?> shutdown = SettableFuture.create();

core/src/testFixtures/java/io/grpc/internal/MockServerTransportListener.java

@@ -27,9 +27,15 @@
 import java.util.concurrent.LinkedBlockingQueue;
 import java.util.concurrent.TimeUnit;
 
+/**
+ * A {@link ServerTransportListener} that helps you write blocking unit tests.
+ *
+ * <p>TODO: Rename, since this is not actually a mock:
+ * https://testing.googleblog.com/2013/07/testing-on-toilet-know-your-test-doubles.html
+ */
 public class MockServerTransportListener implements ServerTransportListener {
   public final ServerTransport transport;
-  public final BlockingQueue<StreamCreation> streams = new LinkedBlockingQueue<>();
+  private final BlockingQueue<StreamCreation> streams = new LinkedBlockingQueue<>();
   private final SettableFuture<?> terminated = SettableFuture.create();
 
   public MockServerTransportListener(ServerTransport transport) {

core/src/testFixtures/java/io/grpc/internal/ServerStreamListenerBase.java

@@ -28,7 +28,7 @@
 import java.util.concurrent.TimeUnit;
 
 /**
- * A {@link ServerStreamListener} that helps you write unit tests.
+ * A {@link ServerStreamListener} that helps you write blocking unit tests.
  */
 public class ServerStreamListenerBase implements ServerStreamListener {
   public final BlockingQueue<InputStream> messageQueue = new LinkedBlockingQueue<>();