From 6b4f68fcb3762535879b75dc0ba25121f9ee91bc Mon Sep 17 00:00:00 2001 From: Martijn van Groningen Date: Wed, 26 Nov 2014 22:00:26 +0100 Subject: [PATCH] Test: made the simple no master node test more useful Original commit: elastic/x-pack-elasticsearch@4ffc11d7022497dd2b510e678382e542b6067b1b --- .../java/org/elasticsearch/alerts/NoMasterNodeTests.java | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/src/test/java/org/elasticsearch/alerts/NoMasterNodeTests.java b/src/test/java/org/elasticsearch/alerts/NoMasterNodeTests.java index 322a003f661..e020afc8f69 100644 --- a/src/test/java/org/elasticsearch/alerts/NoMasterNodeTests.java +++ b/src/test/java/org/elasticsearch/alerts/NoMasterNodeTests.java @@ -76,14 +76,18 @@ public class NoMasterNodeTests extends AbstractAlertingTests { // Bring back the 2nd node and wait for elected master node to come back and alerting to work as expected. startElectedMasterNodeAndWait(); - // Delete an existing alert + // Our first alert should at least have been triggered twice + assertAlertTriggered("my-first-alert", 2); + + // Delete the existing alert DeleteAlertResponse response = alertClient().prepareDeleteAlert("my-first-alert").get(); assertThat(response.deleteResponse().isFound(), is(true)); + // Add a new alert and wait for it get triggered alertClient().preparePutAlert("my-second-alert") .setAlertSource(alertSource) .get(); - assertAlertTriggered("my-second-alert", 2); + assertAlertTriggered("my-second-alert", 1); } @Test