BidirectionalStreamTest: don't cancel and block in teardown
Calling callback.blockForDone() on a non-started stream's callback will block until it timeouts (12s). Currently, only a single test within BidirectionalStreamTest actually starts the stream. This means that every other tests is calling callback.blockForDone() during teardown for no reason at all and they will just get stuck for 12s. Change-Id: Ia40cbc371b4e2450dcd92a3ad429ff4351931a21
This commit is contained in:
@@ -58,11 +58,6 @@ class BidirectionalStreamTest {
|
|||||||
@After
|
@After
|
||||||
@Throws(Exception::class)
|
@Throws(Exception::class)
|
||||||
fun tearDown() {
|
fun tearDown() {
|
||||||
// cancel active requests to enable engine shutdown.
|
|
||||||
stream?.let {
|
|
||||||
it.cancel()
|
|
||||||
callback.blockForDone()
|
|
||||||
}
|
|
||||||
httpEngine.shutdown()
|
httpEngine.shutdown()
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user