Fixing flaky test

elastic · May 17, 2024 · d5c6ac3 · d5c6ac3
1 parent 36008b0
commit d5c6ac3
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 10 deletions.
diff --git a/...lerting_api_integration/security_and_spaces/group1/tests/alerting/backfill/task_runner.ts b/...lerting_api_integration/security_and_spaces/group1/tests/alerting/backfill/task_runner.ts
@@ -92,7 +92,7 @@ export default function createBackfillTaskRunnerTests({ getService }: FtrProvide
       await esTestIndexTool.setup();
     });
     afterEach(async () => {
-      objectRemover.removeAll();
+      await objectRemover.removeAll();
       await esTestIndexTool.destroy();
     });
     after(async () => {

diff --git a/x-pack/test/alerting_api_integration/security_and_spaces/group1/tests/alerting/find.ts b/x-pack/test/alerting_api_integration/security_and_spaces/group1/tests/alerting/find.ts
@@ -19,8 +19,10 @@ const findTestUtils = (
   supertest: SuperTestAgent,
   supertestWithoutAuth: any
 ) => {
-  describe.skip(describeType, () => {
-    afterEach(() => objectRemover.removeAll());
+  describe(describeType, () => {
+    afterEach(async () => {
+      await objectRemover.removeAll();
+    });
 
     for (const scenario of UserAtSpaceScenarios) {
       const { user, space } = scenario;
@@ -651,9 +653,7 @@ export default function createFindTests({ getService }: FtrProviderContext) {
   const supertest = getService('supertest');
   const supertestWithoutAuth = getService('supertestWithoutAuth');
 
-  // Failing: See https://github.com/elastic/kibana/issues/182263
-  // Failing: See https://github.com/elastic/kibana/issues/182284
-  describe.skip('find', () => {
+  describe('find', () => {
     const objectRemover = new ObjectRemover(supertest);
 
     afterEach(() => objectRemover.removeAll());

diff --git a/...test/alerting_api_integration/security_and_spaces/group1/tests/alerting/find_with_post.ts b/...test/alerting_api_integration/security_and_spaces/group1/tests/alerting/find_with_post.ts
@@ -19,14 +19,12 @@ const findTestUtils = (
   supertest: SuperTestAgent,
   supertestWithoutAuth: any
 ) => {
-  // FLAKY: https://github.com/elastic/kibana/issues/182314
-  describe.skip(describeType, () => {
+  describe(describeType, () => {
     afterEach(() => objectRemover.removeAll());
 
     for (const scenario of UserAtSpaceScenarios) {
       const { user, space } = scenario;
-      // FLAKY: https://github.com/elastic/kibana/issues/182314
-      describe.skip(scenario.id, () => {
+      describe(scenario.id, () => {
         it('should handle find alert request appropriately', async () => {
           const { body: createdAlert } = await supertest
             .post(`${getUrlPrefix(space.id)}/api/alerting/rule`)