From 9ae5f116b5abecedb78d1ed57f3754af91c4cb5e Mon Sep 17 00:00:00 2001
From: zpencer <spencerfang@google.com>
Date: Thu, 9 Nov 2017 17:13:04 -0800
Subject: netty: TransportTracer test for local window update (#3710)

---
 .../java/io/grpc/netty/NettyServerHandlerTest.java | 42 ++++++++++++++--------
 1 file changed, 28 insertions(+), 14 deletions(-)

(limited to 'netty')

diff --git a/netty/src/test/java/io/grpc/netty/NettyServerHandlerTest.java b/netty/src/test/java/io/grpc/netty/NettyServerHandlerTest.java
index 008ba23ef..707c4e731 100644
--- a/netty/src/test/java/io/grpc/netty/NettyServerHandlerTest.java
+++ b/netty/src/test/java/io/grpc/netty/NettyServerHandlerTest.java
@@ -759,23 +759,37 @@ public class NettyServerHandlerTest extends NettyHandlerTestBase<NettyServerHand
   }
 
   @Test
-  public void transportTracer_windowSize() throws Exception {
+  public void transportTracer_windowUpdate_remote() throws Exception {
     flowControlWindow = 1048576; // 1MiB
     manualSetUp();
-    {
-      TransportTracer.Stats stats = transportTracer.getStats();
-      assertEquals(Http2CodecUtil.DEFAULT_WINDOW_SIZE, stats.remoteFlowControlWindow);
-      assertEquals(flowControlWindow, stats.localFlowControlWindow);
-    }
+    TransportTracer.Stats before = transportTracer.getStats();
+    assertEquals(Http2CodecUtil.DEFAULT_WINDOW_SIZE, before.remoteFlowControlWindow);
+    assertEquals(flowControlWindow, before.localFlowControlWindow);
 
-    {
-      ByteBuf serializedSettings = windowUpdate(0, 1000);
-      channelRead(serializedSettings);
-      TransportTracer.Stats stats = transportTracer.getStats();
-      assertEquals(Http2CodecUtil.DEFAULT_WINDOW_SIZE + 1000,
-          stats.remoteFlowControlWindow);
-      assertEquals(flowControlWindow, stats.localFlowControlWindow);
-    }
+    ByteBuf serializedSettings = windowUpdate(0, 1000);
+    channelRead(serializedSettings);
+    TransportTracer.Stats after = transportTracer.getStats();
+    assertEquals(Http2CodecUtil.DEFAULT_WINDOW_SIZE + 1000,
+        after.remoteFlowControlWindow);
+    assertEquals(flowControlWindow, after.localFlowControlWindow);
+  }
+
+  @Test
+  public void transportTracer_windowUpdate_local() throws Exception {
+    manualSetUp();
+    TransportTracer.Stats before = transportTracer.getStats();
+    assertEquals(Http2CodecUtil.DEFAULT_WINDOW_SIZE, before.remoteFlowControlWindow);
+    assertEquals(flowControlWindow, before.localFlowControlWindow);
+
+    // If the window size is below a certain threshold, netty will wait to apply the update.
+    // Use a large increment to be sure that it exceeds the threshold.
+    connection().local().flowController().incrementWindowSize(
+        connection().connectionStream(), 8 * Http2CodecUtil.DEFAULT_WINDOW_SIZE);
+
+    TransportTracer.Stats after = transportTracer.getStats();
+    assertEquals(Http2CodecUtil.DEFAULT_WINDOW_SIZE, after.remoteFlowControlWindow);
+    assertEquals(flowControlWindow + 8 * Http2CodecUtil.DEFAULT_WINDOW_SIZE,
+        after.localFlowControlWindow);
   }
 
   private void createStream() throws Exception {
-- 
cgit v1.2.3