Fix flaky test (#152650)

## Summary Fixes: #148469, #148468 Flaky test runner: https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/1981 ### Checklist Delete any items that are not applicable to this PR. - [x] [Unit or functional tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html) were updated or added to match the most common scenarios ### For maintainers - [x] This was checked for breaking API changes and was [labeled appropriately](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process) --------- Co-authored-by: Kibana Machine <[email protected]>
elastic · Mar 6, 2023 · 93fbaee · 93fbaee
1 parent f10777e
commit 93fbaee
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 8 deletions.
diff --git a/x-pack/plugins/cases/public/components/all_cases/use_actions.tsx b/x-pack/plugins/cases/public/components/all_cases/use_actions.tsx
@@ -186,7 +186,12 @@ const ActionColumnComponent: React.FC<{ theCase: Case; disableActions: boolean }
         panelPaddingSize="none"
         anchorPosition="downLeft"
       >
-        <EuiContextMenu initialPanelId={0} panels={panels} key={`case-action-menu-${theCase.id}`} />
+        <EuiContextMenu
+          initialPanelId={0}
+          panels={panels}
+          key={`case-action-menu-${theCase.id}`}
+          data-test-subj={`case-action-menu-${theCase.id}`}
+        />
       </EuiPopover>
       {deleteAction.isModalVisible ? (
         <ConfirmDeleteCaseModal

diff --git a/x-pack/test/functional/services/cases/list.ts b/x-pack/test/functional/services/cases/list.ts
@@ -204,9 +204,10 @@ export function CasesTableServiceProvider(
     async changeStatus(status: CaseStatuses, index: number) {
       await this.openRowActions(index);
 
-      await testSubjects.existOrFail('cases-bulk-action-delete');
+      await retry.waitFor('status panel exists', async () => {
+        return find.existsByCssSelector('[data-test-subj*="case-action-status-panel-"');
+      });
 
-      await find.existsByCssSelector('[data-test-subj*="case-action-status-panel-"');
       const statusButton = await find.byCssSelector('[data-test-subj*="case-action-status-panel-"');
 
       statusButton.click();
@@ -218,9 +219,10 @@ export function CasesTableServiceProvider(
     async changeSeverity(severity: CaseSeverity, index: number) {
       await this.openRowActions(index);
 
-      await testSubjects.existOrFail('cases-bulk-action-delete');
+      await retry.waitFor('severity panel exists', async () => {
+        return find.existsByCssSelector('[data-test-subj*="case-action-severity-panel-"');
+      });
 
-      await find.existsByCssSelector('[data-test-subj*="case-action-severity-panel-"');
       const statusButton = await find.byCssSelector(
         '[data-test-subj*="case-action-severity-panel-"'
       );

diff --git a/x-pack/test/functional_with_es_ssl/apps/cases/group2/list_view.ts b/x-pack/test/functional_with_es_ssl/apps/cases/group2/list_view.ts
@@ -590,9 +590,7 @@ export default ({ getPageObject, getService }: FtrProviderContext) => {
         });
       });
 
-      // FLAKY: https://github.com/elastic/kibana/issues/148468
-      // FLAKY: https://github.com/elastic/kibana/issues/148469
-      describe.skip('Severity', () => {
+      describe('Severity', () => {
         before(async () => {
           await cases.api.createNthRandomCases(1);
           await header.waitUntilLoadingHasFinished();