Disable flaky test assertion in ConnectionPoolTest (#8667)

The assertion `taskRunner.activeQueues()` was commented out due to a race condition causing flaky tests. This ensures more consistent test results while the underlying issue is investigated.
square · Feb 1, 2025 · 8d76074 · 8d76074
1 parent 776f413
commit 8d76074
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/okhttp/src/jvmTest/kotlin/okhttp3/internal/connection/ConnectionPoolTest.kt b/okhttp/src/jvmTest/kotlin/okhttp3/internal/connection/ConnectionPoolTest.kt
@@ -200,7 +200,8 @@ class ConnectionPoolTest {
         maxIdleConnections = 2,
       )
     factory.newConnection(pool, routeA1)
-    assertThat(taskRunner.activeQueues()).isNotEmpty()
+    // Racy causing flaky tests
+    // assertThat(taskRunner.activeQueues()).isNotEmpty()
     assertThat(taskRunnerThreads).isNotEmpty()
     Thread.sleep(100)
     for (t in taskRunnerThreads) {