aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorSorin Basca <sorinbasca@google.com>2021-07-16 07:28:28 +0000
committerAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>2021-07-16 07:28:28 +0000
commit7ee3b70400312ecd7b99524923bdd9e682243da4 (patch)
tree68c3c5d845fbb32f7070af14a0fa14ae42174d58
parent7ee8085f31f613ea02f868e1538cc4d3cc505dd7 (diff)
parent541335a9306579edf064d1be862192891697050d (diff)
downloadokhttp-7ee3b70400312ecd7b99524923bdd9e682243da4.tar.gz
Fixing stability of headersOnlyStreamIsClosedAfterReplyHeaders am: 541335a930
Original change: https://googleplex-android-review.googlesource.com/c/platform/external/okhttp/+/15299577 Change-Id: I274a2312aa53a0a028884cbefe7b9c45415a5bed
-rw-r--r--okhttp-tests/src/test/java/com/squareup/okhttp/internal/framed/Spdy3ConnectionTest.java3
1 files changed, 2 insertions, 1 deletions
diff --git a/okhttp-tests/src/test/java/com/squareup/okhttp/internal/framed/Spdy3ConnectionTest.java b/okhttp-tests/src/test/java/com/squareup/okhttp/internal/framed/Spdy3ConnectionTest.java
index 38961ef..e0f22d7 100644
--- a/okhttp-tests/src/test/java/com/squareup/okhttp/internal/framed/Spdy3ConnectionTest.java
+++ b/okhttp-tests/src/test/java/com/squareup/okhttp/internal/framed/Spdy3ConnectionTest.java
@@ -103,7 +103,8 @@ public final class Spdy3ConnectionTest {
FramedConnection connection = connection(peer, SPDY3);
FramedStream stream = connection.newStream(headerEntries("a", "android"), false, false);
- assertEquals(1, connection.openStreamCount());
+ // We cannot reliably check connection.openStreamCount as the stream gets closed once headers
+ // are exchanged and this can happen before newStream returns
assertEquals(headerEntries("b", "banana"), stream.getResponseHeaders());
connection.ping().roundTripTime(); // Ensure that inFinished has been received.
assertEquals(0, connection.openStreamCount());