Pārlūkot izejas kodu

Enable MetricsAPMIT and change the assertion that logs on failure (#103405)

this commit enables (temporarely) the test to gather some additional logging
relates #103286

Co-authored-by: Moritz Mack <moritz@mackmail.net>
Przemyslaw Gomulka 1 gadu atpakaļ
vecāks
revīzija
9d27c2fc5a

+ 2 - 3
test/external-modules/apm-integration/src/javaRestTest/java/org/elasticsearch/test/apmintegration/MetricsApmIT.java

@@ -56,7 +56,6 @@ public class MetricsApmIT extends ESRestTestCase {
         return cluster.getHttpAddresses();
     }
 
-    @AwaitsFix(bugUrl = "https://github.com/elastic/elasticsearch/issues/103286")
     @SuppressWarnings("unchecked")
     public void testApmIntegration() throws Exception {
         Map<String, Predicate<Map<String, Object>>> sampleAssertions = new HashMap<>(
@@ -108,8 +107,8 @@ public class MetricsApmIT extends ESRestTestCase {
 
         client().performRequest(new Request("GET", "/_use_apm_metrics"));
 
-        finished.await(30, TimeUnit.SECONDS);
-        assertThat(sampleAssertions, Matchers.anEmptyMap());
+        assertTrue("Timeout when waiting for assertions to complete.", finished.await(30, TimeUnit.SECONDS));
+        assertThat(sampleAssertions, Matchers.equalTo(Collections.emptyMap()));
     }
 
     private <T> Map.Entry<String, Predicate<Map<String, Object>>> assertion(