Fix FTR Add/Save Filter failures

elastic · Aug 9, 2022 · d279fd6 · d279fd6
1 parent 0c42a27
commit d279fd6
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/src/plugins/unified_search/public/query_string_input/add_filter_popover.tsx b/src/plugins/unified_search/public/query_string_input/add_filter_popover.tsx
@@ -47,7 +47,7 @@ export const AddFilterPopover = React.memo(function AddFilterPopover({
         iconType="plusInCircleFilled"
         aria-label={buttonIconLabel}
         data-test-subj="addFilter"
-        onClick={() => setIsAddFilterPopoverOpen(!isAddFilterPopoverOpen)}
+        onClick={() => setIsAddFilterPopoverOpen((isOpen) => !isOpen)}
         size="m"
         {...buttonProps}
       />
@@ -63,6 +63,7 @@ export const AddFilterPopover = React.memo(function AddFilterPopover({
         closePopover={() => setIsAddFilterPopoverOpen(false)}
         anchorPosition="downLeft"
         panelPaddingSize="none"
+        panelProps={{ 'data-test-subj': 'addFilterPopover' }}
         initialFocus=".filterEditor__hiddenItem"
         ownFocus
         repositionOnScroll

diff --git a/test/functional/services/filter_bar.ts b/test/functional/services/filter_bar.ts
@@ -128,6 +128,7 @@ export class FilterBarService extends FtrService {
    */
   public async addFilter(field: string, operator: string, ...values: any): Promise<void> {
     await this.testSubjects.click('addFilter');
+    await this.testSubjects.existOrFail('addFilterPopover');
     await this.comboBox.set('filterFieldSuggestionList', field);
     await this.comboBox.set('filterOperatorList', operator);
     const params = await this.testSubjects.find('filterParams');
@@ -152,7 +153,9 @@ export class FilterBarService extends FtrService {
         }
       }
     }
-    await this.testSubjects.click('saveFilter');
+    await this.testSubjects.existOrFail('addFilterPopover');
+    await this.common.sleep(500); // Stale element errors occur without this wait
+    await this.testSubjects.clickWhenNotDisabled('saveFilter');
     await this.header.awaitGlobalLoadingIndicatorHidden();
   }