Added Component Integration Test For Force Merging an Index (#122176)

* Added new test for force merge action. Also added an await in the clear index cache test because there were overlapping act calls. * Removed calls from previous copied test. * Update x-pack/plugins/index_management/__jest__/client_integration/home/indices_tab.test.ts Co-authored-by: Ignacio Rivas <[email protected]> * Update x-pack/plugins/index_management/__jest__/client_integration/home/indices_tab.test.ts Co-authored-by: Ignacio Rivas <[email protected]> Co-authored-by: Ignacio Rivas <[email protected]> Co-authored-by: Kibana Machine <[email protected]>
elastic · Jan 1, 2022 · 87f753d · 87f753d
1 parent bfea0b0
commit 87f753d
Show file tree

Hide file tree

Showing 3 changed files with 33 additions and 1 deletion.
diff --git a/x-pack/plugins/index_management/__jest__/client_integration/helpers/test_subjects.ts b/x-pack/plugins/index_management/__jest__/client_integration/helpers/test_subjects.ts
@@ -21,6 +21,7 @@ export type TestSubjects =
   | 'deleteTemplatesConfirmation'
   | 'documentationLink'
   | 'emptyPrompt'
+  | 'forcemergeIndexMenuButton'
   | 'filterList.filterItem'
   | 'ilmPolicyLink'
   | 'includeStatsSwitch'

diff --git a/x-pack/plugins/index_management/__jest__/client_integration/home/indices_tab.helpers.ts b/x-pack/plugins/index_management/__jest__/client_integration/home/indices_tab.helpers.ts
@@ -30,6 +30,7 @@ export interface IndicesTestBed extends TestBed<TestSubjects> {
     clickDataStreamAt: (index: number) => void;
     clickManageContextMenuButton: () => void;
     clickContextMenuOption: (optionDataTestSubject: string) => void;
+    clickModalConfirm: () => void;
   };
   findDataStreamDetailPanel: () => ReactWrapper;
   findDataStreamDetailPanelTitle: () => string;
@@ -97,6 +98,15 @@ export const setup = async (overridingDependencies: any = {}): Promise<IndicesTe
     component.update();
   };
 
+  const clickModalConfirm = async () => {
+    const { find, component } = testBed;
+
+    await act(async () => {
+      find('confirmModalConfirmButton').simulate('click');
+    });
+    component.update();
+  };
+
   const findDataStreamDetailPanel = () => {
     const { find } = testBed;
     return find('dataStreamDetailPanel');
@@ -116,6 +126,7 @@ export const setup = async (overridingDependencies: any = {}): Promise<IndicesTe
       clickDataStreamAt,
       clickManageContextMenuButton,
       clickContextMenuOption,
+      clickModalConfirm,
     },
     findDataStreamDetailPanel,
     findDataStreamDetailPanelTitle,

diff --git a/x-pack/plugins/index_management/__jest__/client_integration/home/indices_tab.test.ts b/x-pack/plugins/index_management/__jest__/client_integration/home/indices_tab.test.ts
@@ -261,7 +261,7 @@ describe('<IndexManagementHome />', () => {
 
     test("should be able to clear an index's cache", async () => {
       const { actions } = testBed;
-      actions.clickManageContextMenuButton();
+      await actions.clickManageContextMenuButton();
 
       await actions.clickManageContextMenuButton();
       await actions.clickContextMenuOption('clearCacheIndexMenuButton');
@@ -291,6 +291,26 @@ describe('<IndexManagementHome />', () => {
       // The unfreeze action should not be present anymore
       expect(exists('unfreezeIndexMenuButton')).toBe(false);
     });
+
+    test('should be able to force merge an index', async () => {
+      const { actions, exists } = testBed;
+
+      httpRequestsMockHelpers.setReloadIndicesResponse([{ ...indexMockA, isFrozen: false }]);
+
+      // Open context menu
+      await actions.clickManageContextMenuButton();
+      // Check that the force merge action exists for the current index and merge it
+      expect(exists('forcemergeIndexMenuButton')).toBe(true);
+      await actions.clickContextMenuOption('forcemergeIndexMenuButton');
+
+      await actions.clickModalConfirm();
+
+      const requestsCount = server.requests.length;
+      expect(server.requests[requestsCount - 2].url).toBe(`${API_BASE_PATH}/indices/forcemerge`);
+      // After the index is force merged, we immediately do a reload. So we need to expect to see
+      // a reload server call also.
+      expect(server.requests[requestsCount - 1].url).toBe(`${API_BASE_PATH}/indices/reload`);
+    });
   });
 
   describe('Edit index settings', () => {