Skip to content

Commit

Permalink
[ML] Functional tests - fix and re-activate alerting flyout test (#10…
Browse files Browse the repository at this point in the history
…2368) (#102392)

This PR fixes the ML alerting flyout tests and re-activates it.

Co-authored-by: Robert Oskamp <robert.oskamp@elastic.co>
  • Loading branch information
kibanamachine and pheyos committed Jun 16, 2021
1 parent b16513a commit 5482a70
Showing 1 changed file with 3 additions and 4 deletions.
7 changes: 3 additions & 4 deletions x-pack/test/functional_with_es_ssl/apps/ml/alert_flyout.ts
Original file line number Diff line number Diff line change
Expand Up @@ -67,8 +67,7 @@ export default ({ getPageObjects, getService }: FtrProviderContext) => {

let testJobId = '';

// Failing: See https://github.com/elastic/kibana/issues/102012
describe.skip('anomaly detection alert', function () {
describe('anomaly detection alert', function () {
this.tags('ciGroup13');

before(async () => {
Expand Down Expand Up @@ -119,11 +118,11 @@ export default ({ getPageObjects, getService }: FtrProviderContext) => {

await ml.testExecution.logTestStep('should preview the alert condition');
await ml.alerting.assertPreviewButtonState(false);
await ml.alerting.setTestInterval('2y');
await ml.alerting.setTestInterval('5y');
await ml.alerting.assertPreviewButtonState(true);

// don't check the exact number provided by the backend, just make sure it's > 0
await ml.alerting.checkPreview(/Found [1-9]\d* anomalies in the last 2y/);
await ml.alerting.checkPreview(/Found [1-9]\d* anomal(y|ies) in the last 5y/);

await ml.testExecution.logTestStep('should create an alert');
await pageObjects.triggersActionsUI.setAlertName('ml-test-alert');
Expand Down

0 comments on commit 5482a70

Please sign in to comment.