From 34deabc6bbe03c6ee3e1ebe5b4b7d45a77ac8502 Mon Sep 17 00:00:00 2001
From: Ying Mao <ying.mao@elastic.co>
Date: Wed, 5 Oct 2022 08:26:07 -0400
Subject: [PATCH] Fixing flaky `disable` test (#142687)

* Unskipping test

* Adding retries

* Cleanup

* Cleanup
---
 .../spaces_only/tests/alerting/disable.ts     | 39 ++++++++++---------
 1 file changed, 21 insertions(+), 18 deletions(-)

diff --git a/x-pack/test/alerting_api_integration/spaces_only/tests/alerting/disable.ts b/x-pack/test/alerting_api_integration/spaces_only/tests/alerting/disable.ts
index 495e423e43194..d4149c9cf2fb8 100644
--- a/x-pack/test/alerting_api_integration/spaces_only/tests/alerting/disable.ts
+++ b/x-pack/test/alerting_api_integration/spaces_only/tests/alerting/disable.ts
@@ -26,8 +26,7 @@ export default function createDisableRuleTests({ getService }: FtrProviderContex
   const retry = getService('retry');
   const supertest = getService('supertest');
 
-  // Failing: See https://github.com/elastic/kibana/issues/141864
-  describe.skip('disable', () => {
+  describe('disable', () => {
     const objectRemover = new ObjectRemover(supertestWithoutAuth);
     const ruleUtils = new RuleUtils({ space: Spaces.space1, supertestWithoutAuth });
 
@@ -52,15 +51,17 @@ export default function createDisableRuleTests({ getService }: FtrProviderContex
       await ruleUtils.disable(createdRule.id);
 
       // task doc should still exist but be disabled
-      const taskRecord = await getScheduledTask(createdRule.scheduled_task_id);
-      expect(taskRecord.type).to.eql('task');
-      expect(taskRecord.task.taskType).to.eql('alerting:test.noop');
-      expect(JSON.parse(taskRecord.task.params)).to.eql({
-        alertId: createdRule.id,
-        spaceId: Spaces.space1.id,
-        consumer: 'alertsFixture',
+      await retry.try(async () => {
+        const taskRecord = await getScheduledTask(createdRule.scheduled_task_id);
+        expect(taskRecord.type).to.eql('task');
+        expect(taskRecord.task.taskType).to.eql('alerting:test.noop');
+        expect(JSON.parse(taskRecord.task.params)).to.eql({
+          alertId: createdRule.id,
+          spaceId: Spaces.space1.id,
+          consumer: 'alertsFixture',
+        });
+        expect(taskRecord.task.enabled).to.eql(false);
       });
-      expect(taskRecord.task.enabled).to.eql(false);
 
       // Ensure AAD isn't broken
       await checkAAD({
@@ -196,15 +197,17 @@ export default function createDisableRuleTests({ getService }: FtrProviderContex
           .expect(204);
 
         // task doc should still exist but be disabled
-        const taskRecord = await getScheduledTask(createdRule.scheduled_task_id);
-        expect(taskRecord.type).to.eql('task');
-        expect(taskRecord.task.taskType).to.eql('alerting:test.noop');
-        expect(JSON.parse(taskRecord.task.params)).to.eql({
-          alertId: createdRule.id,
-          spaceId: Spaces.space1.id,
-          consumer: 'alertsFixture',
+        await retry.try(async () => {
+          const taskRecord = await getScheduledTask(createdRule.scheduled_task_id);
+          expect(taskRecord.type).to.eql('task');
+          expect(taskRecord.task.taskType).to.eql('alerting:test.noop');
+          expect(JSON.parse(taskRecord.task.params)).to.eql({
+            alertId: createdRule.id,
+            spaceId: Spaces.space1.id,
+            consumer: 'alertsFixture',
+          });
+          expect(taskRecord.task.enabled).to.eql(false);
         });
-        expect(taskRecord.task.enabled).to.eql(false);
 
         // Ensure AAD isn't broken
         await checkAAD({