aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorSorin Basca <sorinbasca@google.com>2021-07-16 07:27:39 +0000
committerAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>2021-07-16 07:27:39 +0000
commitc733e370eaa6bec83cd0a06b816808c99963ee49 (patch)
tree68c3c5d845fbb32f7070af14a0fa14ae42174d58
parent903572dc5320a2e5525fc9bc50b7afbe90bf5e3b (diff)
parent541335a9306579edf064d1be862192891697050d (diff)
downloadokhttp-c733e370eaa6bec83cd0a06b816808c99963ee49.tar.gz
Fixing stability of headersOnlyStreamIsClosedAfterReplyHeaders am: 541335a930
Original change: https://googleplex-android-review.googlesource.com/c/platform/external/okhttp/+/15299577 Change-Id: I4460c2134752d229e2e9cca7732a1dd79b9858d8
-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());