From 89082815bf8aa77cae07526323002c55f09d7fb1 Mon Sep 17 00:00:00 2001
From: Walter Rafelsberger <walter.rafelsberger@elastic.co>
Date: Tue, 18 Apr 2023 16:05:58 +0200
Subject: [PATCH] fix jest tests

---
 .../table_action_button.tsx                   | 15 +++++++++--
 .../use_copy_to_clipboard_action.test.tsx     | 26 ++++++++++++++-----
 2 files changed, 32 insertions(+), 9 deletions(-)

diff --git a/x-pack/plugins/aiops/public/components/spike_analysis_table/table_action_button.tsx b/x-pack/plugins/aiops/public/components/spike_analysis_table/table_action_button.tsx
index f77ed1581d8436..48efd37fe76a15 100644
--- a/x-pack/plugins/aiops/public/components/spike_analysis_table/table_action_button.tsx
+++ b/x-pack/plugins/aiops/public/components/spike_analysis_table/table_action_button.tsx
@@ -32,11 +32,22 @@ export const TableActionButton: FC<TableActionButtonProps> = ({
   );
 
   const unwrappedButton = !isDisabled ? (
-    <EuiLink onClick={onClick} color={'text'} aria-label={message}>
+    <EuiLink
+      data-test-subj="aiopsTableActionButtonEnabled"
+      onClick={onClick}
+      color={'text'}
+      aria-label={message}
+    >
       {buttonContent}
     </EuiLink>
   ) : (
-    <EuiText size="s" color={'subdued'} aria-label={message} css={{ fontWeight: 500 }}>
+    <EuiText
+      data-test-subj="aiopsTableActionButtonDisabled"
+      size="s"
+      color={'subdued'}
+      aria-label={message}
+      css={{ fontWeight: 500 }}
+    >
       {buttonContent}
     </EuiText>
   );
diff --git a/x-pack/plugins/aiops/public/components/spike_analysis_table/use_copy_to_clipboard_action.test.tsx b/x-pack/plugins/aiops/public/components/spike_analysis_table/use_copy_to_clipboard_action.test.tsx
index 82359b3d2b1aa5..966c4d1466afb3 100644
--- a/x-pack/plugins/aiops/public/components/spike_analysis_table/use_copy_to_clipboard_action.test.tsx
+++ b/x-pack/plugins/aiops/public/components/spike_analysis_table/use_copy_to_clipboard_action.test.tsx
@@ -30,11 +30,19 @@ describe('useCopyToClipboardAction', () => {
   it('renders the action for a single significant term', async () => {
     execCommandMock.mockImplementationOnce(() => true);
     const { result } = renderHook(() => useCopyToClipboardAction());
-    const { getByLabelText } = render((result.current as Action).render(significantTerms[0]));
+    const { findByText, getByTestId } = render(
+      (result.current as Action).render(significantTerms[0])
+    );
 
-    const button = getByLabelText('Copy field/value pair as KQL syntax to clipboard');
+    const button = getByTestId('aiopsTableActionButtonEnabled');
 
-    expect(button).toBeInTheDocument();
+    userEvent.hover(button);
+
+    // The tooltip from EUI takes 250ms to appear, so we must
+    // use a `find*` query to asynchronously poll for it.
+    expect(
+      await findByText('Copy field/value pair as KQL syntax to clipboard')
+    ).toBeInTheDocument();
 
     await act(async () => {
       await userEvent.click(button);
@@ -50,12 +58,16 @@ describe('useCopyToClipboardAction', () => {
   it('renders the action for a group of items', async () => {
     execCommandMock.mockImplementationOnce(() => true);
     const groupTableItems = getGroupTableItems(finalSignificantTermGroups);
-    const { result } = renderHook(() => useCopyToClipboardAction());
-    const { getByLabelText } = render((result.current as Action).render(groupTableItems[0]));
+    const { result } = renderHook(useCopyToClipboardAction);
+    const { findByText, getByText } = render((result.current as Action).render(groupTableItems[0]));
+
+    const button = getByText('Copy to clipboard');
 
-    const button = getByLabelText('Copy group items as KQL syntax to clipboard');
+    userEvent.hover(button);
 
-    expect(button).toBeInTheDocument();
+    // The tooltip from EUI takes 250ms to appear, so we must
+    // use a `find*` query to asynchronously poll for it.
+    expect(await findByText('Copy group items as KQL syntax to clipboard')).toBeInTheDocument();
 
     await act(async () => {
       await userEvent.click(button);