From c3c08c46597c3347851b786b877469bf549102de Mon Sep 17 00:00:00 2001
From: Xavier Mouligneau <189600+XavierM@users.noreply.github.com>
Date: Sat, 19 Dec 2020 15:50:47 -0500
Subject: [PATCH] fix jest tests

---
 .../draggable_wrapper_hover_content.test.tsx         | 12 ++++++------
 .../__snapshots__/index.test.tsx.snap                |  7 +++++++
 2 files changed, 13 insertions(+), 6 deletions(-)

diff --git a/x-pack/plugins/security_solution/public/common/components/drag_and_drop/draggable_wrapper_hover_content.test.tsx b/x-pack/plugins/security_solution/public/common/components/drag_and_drop/draggable_wrapper_hover_content.test.tsx
index 25cc575e5bf26..9ce5778fb72e5 100644
--- a/x-pack/plugins/security_solution/public/common/components/drag_and_drop/draggable_wrapper_hover_content.test.tsx
+++ b/x-pack/plugins/security_solution/public/common/components/drag_and_drop/draggable_wrapper_hover_content.test.tsx
@@ -403,7 +403,7 @@ describe('DraggableWrapperHoverContent', () => {
   });
 
   describe('Top N', () => {
-    test(`it renders the 'Show top field' button when showTopN is false and an aggregatable string field is provided`, async () => {
+    test(`it renders the 'Show top field' button when showTopN is false and an aggregatable string field is provided`, () => {
       const aggregatableStringField = 'cloud.account.id';
       const wrapper = mount(
         <TestProviders>
@@ -421,7 +421,7 @@ describe('DraggableWrapperHoverContent', () => {
       expect(wrapper.find('[data-test-subj="show-top-field"]').first().exists()).toBe(true);
     });
 
-    test(`it renders the 'Show top field' button when showTopN is false and a allowlisted signal field is provided`, async () => {
+    test(`it renders the 'Show top field' button when showTopN is false and a allowlisted signal field is provided`, () => {
       const allowlistedField = 'signal.rule.name';
       const wrapper = mount(
         <TestProviders>
@@ -439,7 +439,7 @@ describe('DraggableWrapperHoverContent', () => {
       expect(wrapper.find('[data-test-subj="show-top-field"]').first().exists()).toBe(true);
     });
 
-    test(`it does NOT render the 'Show top field' button when showTopN is false and a field not known to BrowserFields is provided`, async () => {
+    test(`it does NOT render the 'Show top field' button when showTopN is false and a field not known to BrowserFields is provided`, () => {
       const notKnownToBrowserFields = 'unknown.field';
       const wrapper = mount(
         <TestProviders>
@@ -477,7 +477,7 @@ describe('DraggableWrapperHoverContent', () => {
       });
     });
 
-    test(`invokes the toggleTopN function when the 'Show top field' button is clicked`, async () => {
+    test(`invokes the toggleTopN function when the 'Show top field' button is clicked`, () => {
       const allowlistedField = 'signal.rule.name';
       const wrapper = mount(
         <TestProviders>
@@ -518,7 +518,7 @@ describe('DraggableWrapperHoverContent', () => {
       );
     });
 
-    test(`it does NOT render the 'Show top field' button when showTopN is true`, async () => {
+    test(`it does NOT render the 'Show top field' button when showTopN is true`, () => {
       const allowlistedField = 'signal.rule.name';
       const wrapper = mount(
         <TestProviders>
@@ -537,7 +537,7 @@ describe('DraggableWrapperHoverContent', () => {
       expect(wrapper.find('[data-test-subj="show-top-field"]').first().exists()).toBe(false);
     });
 
-    test(`it renders the Top N histogram when when showTopN is true`, async () => {
+    test(`it renders the Top N histogram when when showTopN is true`, () => {
       const allowlistedField = 'signal.rule.name';
       const wrapper = mount(
         <TestProviders>
diff --git a/x-pack/plugins/security_solution/public/timelines/components/timeline/body/data_driven_columns/__snapshots__/index.test.tsx.snap b/x-pack/plugins/security_solution/public/timelines/components/timeline/body/data_driven_columns/__snapshots__/index.test.tsx.snap
index 8f514ca49e848..d112a665d77c0 100644
--- a/x-pack/plugins/security_solution/public/timelines/components/timeline/body/data_driven_columns/__snapshots__/index.test.tsx.snap
+++ b/x-pack/plugins/security_solution/public/timelines/components/timeline/body/data_driven_columns/__snapshots__/index.test.tsx.snap
@@ -44,6 +44,7 @@ exports[`Columns it renders the expected columns 1`] = `
         truncate={true}
       />
     </styled.div>
+    0
   </styled.div>
   <styled.div
     $ariaColumnIndex={3}
@@ -74,6 +75,7 @@ exports[`Columns it renders the expected columns 1`] = `
         value="Access"
       />
     </styled.div>
+    0
   </styled.div>
   <styled.div
     $ariaColumnIndex={4}
@@ -104,6 +106,7 @@ exports[`Columns it renders the expected columns 1`] = `
         value="Action"
       />
     </styled.div>
+    0
   </styled.div>
   <styled.div
     $ariaColumnIndex={5}
@@ -134,6 +137,7 @@ exports[`Columns it renders the expected columns 1`] = `
         value="apache"
       />
     </styled.div>
+    0
   </styled.div>
   <styled.div
     $ariaColumnIndex={6}
@@ -164,6 +168,7 @@ exports[`Columns it renders the expected columns 1`] = `
         value="192.168.0.1"
       />
     </styled.div>
+    0
   </styled.div>
   <styled.div
     $ariaColumnIndex={7}
@@ -194,6 +199,7 @@ exports[`Columns it renders the expected columns 1`] = `
         value="192.168.0.3"
       />
     </styled.div>
+    0
   </styled.div>
   <styled.div
     $ariaColumnIndex={8}
@@ -224,6 +230,7 @@ exports[`Columns it renders the expected columns 1`] = `
         value="john.dee"
       />
     </styled.div>
+    0
   </styled.div>
 </styled.div>
 `;