Skip to content

Commit 8b3a02d

Browse files
dkasimovskiyclaude
andcommitted
refactor(tests): drop no-op retry wrapper around TQE subscribe
subscribe() returns void immediately after kicking off the async stream, so retryUntilSuccess cannot retry a failure — the wrapper is dead weight. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
1 parent e7d1518 commit 8b3a02d

1 file changed

Lines changed: 1 addition & 7 deletions

File tree

testcontainers/src/test/java/org/testcontainers/containers/integration/tqe/TQEClusterIntegrationTest.java

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -38,13 +38,7 @@ void testPublishAndConsumeData(TQEVersion version) {
3838
// produced — avoids relying on cursor="" replay, which is unreliable while the
3939
// TQE 3.x broker is still settling.
4040
final Set<User> result = new CopyOnWriteArraySet<>();
41-
Unreliables.retryUntilSuccess(
42-
RETRY_TIMEOUT_SECONDS,
43-
TimeUnit.SECONDS,
44-
() -> {
45-
fx.consumerClient().subscribe(QUEUE_NAME, result::add);
46-
return true;
47-
});
41+
fx.consumerClient().subscribe(QUEUE_NAME, result::add);
4842

4943
Unreliables.retryUntilSuccess(
5044
RETRY_TIMEOUT_SECONDS,

0 commit comments

Comments
 (0)