docs: add note about directExecutor in example tests.

grpc · May 6, 2019 · 08bed14 · 08bed14
1 parent d50c8d4
commit 08bed14
Show file tree

Hide file tree

Showing 4 changed files with 16 additions and 0 deletions.
diff --git a/examples/src/test/java/io/grpc/examples/helloworld/HelloWorldClientTest.java b/examples/src/test/java/io/grpc/examples/helloworld/HelloWorldClientTest.java
@@ -39,6 +39,10 @@
  * For demonstrating how to write gRPC unit test only.
  * Not intended to provide a high code coverage or to test every major usecase.
  *
+ * Note we use directExecutor() here to speed up the test
+ * but this requires the application to not block under any circumstances.
+ * Particilarly in bidirectional streaming it's better to use the default executor!
+ *
  * <p>For more unit test examples see {@link io.grpc.examples.routeguide.RouteGuideClientTest} and
  * {@link io.grpc.examples.routeguide.RouteGuideServerTest}.
  */

diff --git a/examples/src/test/java/io/grpc/examples/helloworld/HelloWorldServerTest.java b/examples/src/test/java/io/grpc/examples/helloworld/HelloWorldServerTest.java
@@ -32,6 +32,10 @@
  * For demonstrating how to write gRPC unit test only.
  * Not intended to provide a high code coverage or to test every major usecase.
  *
+ * Note we use directExecutor() here to speed up the test
+ * but this requires the application to not block under any circumstances.
+ * Particilarly in bidirectional streaming it's better to use the default executor!
+ *
  * <p>For more unit test examples see {@link io.grpc.examples.routeguide.RouteGuideClientTest} and
  * {@link io.grpc.examples.routeguide.RouteGuideServerTest}.
  */

diff --git a/examples/src/test/java/io/grpc/examples/routeguide/RouteGuideClientTest.java b/examples/src/test/java/io/grpc/examples/routeguide/RouteGuideClientTest.java
@@ -53,6 +53,10 @@
  * For demonstrating how to write gRPC unit test only.
  * Not intended to provide a high code coverage or to test every major usecase.
  *
+ * Note we use directExecutor() here to speed up the test
+ * but this requires the application to not block under any circumstances.
+ * Particilarly in bidirectional streaming it's better to use the default executor!
+ *
  * <p>For basic unit test examples see {@link io.grpc.examples.helloworld.HelloWorldClientTest} and
  * {@link io.grpc.examples.helloworld.HelloWorldServerTest}.
  */

diff --git a/examples/src/test/java/io/grpc/examples/routeguide/RouteGuideServerTest.java b/examples/src/test/java/io/grpc/examples/routeguide/RouteGuideServerTest.java
@@ -49,6 +49,10 @@
  * For demonstrating how to write gRPC unit test only.
  * Not intended to provide a high code coverage or to test every major usecase.
  *
+ * Note we use directExecutor() here to speed up the test
+ * but this requires the application to not block under any circumstances.
+ * Particilarly in bidirectional streaming it's better to use the default executor!
+ *
  * <p>For basic unit test examples see {@link io.grpc.examples.helloworld.HelloWorldClientTest} and
  * {@link io.grpc.examples.helloworld.HelloWorldServerTest}.
  */