Skip to content

Commit

Permalink
fix: Remove unncecessary @afterclass cleanup in LagReportingAgentFunc…
Browse files Browse the repository at this point in the history
…tionalTest which is causing flakiness (#4429)

* fix: Remove unncecessary @afterclass cleanup in LagReportingAgentFunctionalTest which is causing flakiness
  • Loading branch information
AlanConfluent committed Feb 3, 2020
1 parent 399c732 commit e422b77
Showing 1 changed file with 0 additions and 10 deletions.
Expand Up @@ -68,8 +68,6 @@ public class LagReportingAgentFunctionalTest {

private static final KsqlHostEntity HOST0 = new KsqlHostEntity("localhost", 8088);
private static final KsqlHostEntity HOST1 = new KsqlHostEntity("localhost", 8089);
private static final String HOST0_STR = "localhost:8088";
private static final String HOST1_STR = "localhost:8089";
private static final IntegrationTestHarness TEST_HARNESS = IntegrationTestHarness.build();
private static final TestKsqlRestApp REST_APP_0 = TestKsqlRestApp
.builder(TEST_HARNESS::kafkaBootstrapServers)
Expand All @@ -86,7 +84,6 @@ public class LagReportingAgentFunctionalTest {
// Lag Reporting
.withProperty(KsqlRestConfig.KSQL_LAG_REPORTING_ENABLE_CONFIG, true)
.withProperty(KsqlRestConfig.KSQL_LAG_REPORTING_SEND_INTERVAL_MS_CONFIG, 3000)
.withProperties(ClientTrustStore.trustStoreProps())
.build();
private static final TestKsqlRestApp REST_APP_1 = TestKsqlRestApp
.builder(TEST_HARNESS::kafkaBootstrapServers)
Expand All @@ -103,7 +100,6 @@ public class LagReportingAgentFunctionalTest {
// Lag Reporting
.withProperty(KsqlRestConfig.KSQL_LAG_REPORTING_ENABLE_CONFIG, true)
.withProperty(KsqlRestConfig.KSQL_LAG_REPORTING_SEND_INTERVAL_MS_CONFIG, 3000)
.withProperties(ClientTrustStore.trustStoreProps())
.build();

@ClassRule
Expand Down Expand Up @@ -135,12 +131,6 @@ public static void setUpClass() {
);
}

@AfterClass
public static void tearDownClass() {
REST_APP_0.closePersistentQueries();
REST_APP_0.dropSourcesExcept();
}

@Test(timeout = 60000)
public void shouldExchangeLags() {
// Given:
Expand Down

0 comments on commit e422b77

Please sign in to comment.