[ILM] Fix jest test

elastic · Jul 31, 2020 · 8ad94dc · 8ad94dc
1 parent e9575f4
commit 8ad94dc
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 2 deletions.
diff --git a/x-pack/plugins/index_lifecycle_management/__jest__/components/edit_policy.test.js b/x-pack/plugins/index_lifecycle_management/__jest__/components/edit_policy.test.js
@@ -253,6 +253,9 @@ describe('edit policy', () => {
     beforeEach(() => {
       server.respondImmediately = true;
       httpRequestsMockHelpers.setNodesListResponse({});
+      httpRequestsMockHelpers.setNodesDetailsResponse('attribute:true', [
+        { nodeId: 'testNodeId', stats: { name: 'testNodeName', host: 'testHost' } },
+      ]);
     });
 
     test('should show number required error when trying to save empty warm phase', async () => {
@@ -395,7 +398,9 @@ describe('edit policy', () => {
       rendered.update();
       const flyoutButton = findTestSubject(rendered, 'warm-viewNodeDetailsFlyoutButton');
       expect(flyoutButton.exists()).toBeTruthy();
-      flyoutButton.simulate('click');
+      await act(async () => {
+        await flyoutButton.simulate('click');
+      });
       rendered.update();
       expect(rendered.find('.euiFlyout').exists()).toBeTruthy();
     });
@@ -404,6 +409,9 @@ describe('edit policy', () => {
     beforeEach(() => {
       server.respondImmediately = true;
       httpRequestsMockHelpers.setNodesListResponse({});
+      httpRequestsMockHelpers.setNodesDetailsResponse('attribute:true', [
+        { nodeId: 'testNodeId', stats: { name: 'testNodeName', host: 'testHost' } },
+      ]);
     });
     test('should allow 0 for phase timing', async () => {
       const rendered = mountWithIntl(component);
@@ -470,7 +478,9 @@ describe('edit policy', () => {
       rendered.update();
       const flyoutButton = findTestSubject(rendered, 'cold-viewNodeDetailsFlyoutButton');
       expect(flyoutButton.exists()).toBeTruthy();
-      flyoutButton.simulate('click');
+      await act(async () => {
+        await flyoutButton.simulate('click');
+      });
       rendered.update();
       expect(rendered.find('.euiFlyout').exists()).toBeTruthy();
     });

diff --git a/x-pack/plugins/index_lifecycle_management/__jest__/components/helpers/http_requests.ts b/x-pack/plugins/index_lifecycle_management/__jest__/components/helpers/http_requests.ts
@@ -25,9 +25,18 @@ const registerHttpRequestMockHelpers = (server: SinonFakeServer) => {
     ]);
   };
 
+  const setNodesDetailsResponse = (nodeAttributes: string, response: HttpResponse = []) => {
+    server.respondWith(`/api/index_lifecycle_management/nodes/${nodeAttributes}/details`, [
+      200,
+      { 'Content-Type': 'application/json' },
+      JSON.stringify(response),
+    ]);
+  };
+
   return {
     setPoliciesResponse,
     setNodesListResponse,
+    setNodesDetailsResponse,
   };
 };