diff options
author | Valerij Fredriksen <freva@users.noreply.github.com> | 2024-05-29 20:09:05 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2024-05-29 20:09:05 +0200 |
commit | 3cf95263b9462367b7f8dda0b6739478dbe4a369 (patch) | |
tree | cb77b7e16dc03e13ca653218010e8abf33a62a40 | |
parent | 7abe0244578d86115bbb32c36b40dd8f8b48817a (diff) | |
parent | e0eef8d7aad00384ab192e27d7a3d39303d87de7 (diff) |
Merge pull request #31345 from vespa-engine/jonmv/feed-client-timeout-fixv8.349.46
Fix test after moving close to different thread MERGEOK
-rw-r--r-- | vespa-feed-client/src/test/java/ai/vespa/feed/client/impl/HttpRequestStrategyTest.java | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/vespa-feed-client/src/test/java/ai/vespa/feed/client/impl/HttpRequestStrategyTest.java b/vespa-feed-client/src/test/java/ai/vespa/feed/client/impl/HttpRequestStrategyTest.java index a95f859df4d..35d6433ffc5 100644 --- a/vespa-feed-client/src/test/java/ai/vespa/feed/client/impl/HttpRequestStrategyTest.java +++ b/vespa-feed-client/src/test/java/ai/vespa/feed/client/impl/HttpRequestStrategyTest.java @@ -251,10 +251,9 @@ class HttpRequestStrategyTest { assertFalse(clusters.get(1).closed.get()); secondVessel.complete(HttpResponse.of(504, null)); assertEquals(504, secondResponse.get().code()); - assertTrue(clusters.get(0).closed.get()); - assertFalse(clusters.get(1).closed.get()); strategy.await(); strategy.destroy(); + assertTrue(clusters.get(0).closed.get()); assertTrue(clusters.get(1).closed.get()); } |