From 8015ee2f4631dab0db4a333ee6007df4da3cbeb3 Mon Sep 17 00:00:00 2001 From: Ed Savage Date: Thu, 21 Jan 2021 11:52:18 +0000 Subject: [PATCH] [ML] Re-enable temporarily muted integration tests Unmute tests that were temporarily disabled in order to change how the Java ML process communicates with the C++ backend. --- .../elasticsearch/xpack/ml/integration/DetectionRulesIT.java | 1 - .../elasticsearch/xpack/ml/integration/ScheduledEventsIT.java | 2 -- 2 files changed, 3 deletions(-) diff --git a/x-pack/plugin/ml/qa/native-multi-node-tests/src/javaRestTest/java/org/elasticsearch/xpack/ml/integration/DetectionRulesIT.java b/x-pack/plugin/ml/qa/native-multi-node-tests/src/javaRestTest/java/org/elasticsearch/xpack/ml/integration/DetectionRulesIT.java index b1cacea65867b..c62893b142dae 100644 --- a/x-pack/plugin/ml/qa/native-multi-node-tests/src/javaRestTest/java/org/elasticsearch/xpack/ml/integration/DetectionRulesIT.java +++ b/x-pack/plugin/ml/qa/native-multi-node-tests/src/javaRestTest/java/org/elasticsearch/xpack/ml/integration/DetectionRulesIT.java @@ -122,7 +122,6 @@ public void testCondition() throws Exception { assertThat(records.get(0).getByFieldValue(), equalTo("low")); } - @AwaitsFix(bugUrl = "https://github.com/elastic/ml-cpp/issues/1253") public void testScope() throws Exception { MlFilter safeIps = MlFilter.builder("safe_ips").setItems("111.111.111.111", "222.222.222.222").build(); assertThat(putMlFilter(safeIps).getFilter(), equalTo(safeIps)); diff --git a/x-pack/plugin/ml/qa/native-multi-node-tests/src/javaRestTest/java/org/elasticsearch/xpack/ml/integration/ScheduledEventsIT.java b/x-pack/plugin/ml/qa/native-multi-node-tests/src/javaRestTest/java/org/elasticsearch/xpack/ml/integration/ScheduledEventsIT.java index 6f496af7a2ac9..b682abe3a7f9e 100644 --- a/x-pack/plugin/ml/qa/native-multi-node-tests/src/javaRestTest/java/org/elasticsearch/xpack/ml/integration/ScheduledEventsIT.java +++ b/x-pack/plugin/ml/qa/native-multi-node-tests/src/javaRestTest/java/org/elasticsearch/xpack/ml/integration/ScheduledEventsIT.java @@ -195,7 +195,6 @@ public void testScheduledEventWithInterimResults() throws IOException { /** * Test an open job picks up changes to scheduled events/calendars */ - @AwaitsFix(bugUrl = "https://github.com/elastic/ml-cpp/issues/1253") public void testAddEventsToOpenJob() throws Exception { TimeValue bucketSpan = TimeValue.timeValueMinutes(30); Job.Builder job = createJob("scheduled-events-add-events-to-open-job", bucketSpan); @@ -264,7 +263,6 @@ public void testAddEventsToOpenJob() throws Exception { /** * An open job that later gets added to a calendar, should take the scheduled events into account */ - @AwaitsFix(bugUrl = "https://github.com/elastic/ml-cpp/issues/1253") public void testAddOpenedJobToGroupWithCalendar() throws Exception { TimeValue bucketSpan = TimeValue.timeValueMinutes(30); String groupName = "opened-calendar-job-group";