Skip to content

Commit

Permalink
[fix][build] Disable flaky test BrokerRegistryIntegrationTest (apache…
Browse files Browse the repository at this point in the history
  • Loading branch information
lhotari committed Sep 28, 2024
1 parent 95bd1d1 commit ca4a743
Showing 1 changed file with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@
import org.testng.annotations.Test;

@Slf4j
@Test(groups = "broker")
@Test(groups = "flaky")
public class BrokerRegistryIntegrationTest {

private static final String clusterName = "test";
Expand Down Expand Up @@ -69,7 +69,7 @@ protected void cleanup() throws Exception {
bk.stop();
}

@Test
@Test(enabled = false)
public void testRecoverFromNodeDeletion() throws Exception {
// Simulate the case that the node was somehow deleted (e.g. by session timeout)
Awaitility.await().atMost(Duration.ofSeconds(3)).untilAsserted(() -> Assert.assertEquals(
Expand All @@ -88,7 +88,7 @@ public void testRecoverFromNodeDeletion() throws Exception {
Assert.assertEquals(brokerRegistry.getAvailableBrokersAsync().get(), List.of(pulsar.getBrokerId()));
}

@Test
@Test(enabled = false)
public void testRegisterAgain() throws Exception {
Awaitility.await().atMost(Duration.ofSeconds(3)).untilAsserted(() -> Assert.assertEquals(
brokerRegistry.getAvailableBrokersAsync().join(), List.of(pulsar.getBrokerId())));
Expand Down

0 comments on commit ca4a743

Please sign in to comment.