Fix functional test

elastic · Dec 22, 2020 · 5847ad9 · 5847ad9
1 parent 61774d3
commit 5847ad9
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/src/plugins/discover/public/application/components/discover_grid/discover_grid_flyout.tsx b/src/plugins/discover/public/application/components/discover_grid/discover_grid_flyout.tsx
@@ -124,16 +124,16 @@ export function DiscoverGridFlyout({
             columns={columns}
             indexPattern={indexPattern}
             filter={(mapping, value, mode) => {
-              onClose();
               onFilter(mapping, value, mode);
+              onClose();
             }}
             onRemoveColumn={(columnName: string) => {
-              onClose();
               onRemoveColumn(columnName);
+              onClose();
             }}
             onAddColumn={(columnName: string) => {
-              onClose();
               onAddColumn(columnName);
+              onClose();
             }}
           />
         </EuiFlyoutBody>

diff --git a/test/functional/apps/discover/_data_grid_doc_navigation.ts b/test/functional/apps/discover/_data_grid_doc_navigation.ts
@@ -79,7 +79,9 @@ export default function ({ getService, getPageObjects }: FtrProviderContext) {
         );
         expect(hasInclusiveFilter).to.be(true);
 
-        await dataGrid.removeInclusiveFilter(details, 'referer');
+        await dataGrid.clickRowToggle({ rowIndex: 0 });
+        const detailsExcluding = await dataGrid.getDetailsRow();
+        await dataGrid.removeInclusiveFilter(detailsExcluding, 'referer');
         await PageObjects.discover.waitUntilSearchingHasFinished();
         const hasExcludeFilter = await filterBar.hasFilter('referer', 'exists', true, false, false);
         expect(hasExcludeFilter).to.be(true);