From ea1243ea282d5581ede1bfb7658406d9ffd97b95 Mon Sep 17 00:00:00 2001
From: Eric Olkowski <thatblindgeye@gmail.com>
Date: Thu, 7 Sep 2023 13:14:32 -0400
Subject: [PATCH] Updated onTooltipHidden test to integration

---
 .../__tests__/ClipboardCopyButton.test.tsx           | 10 ++--------
 .../__snapshots__/ClipboardCopyButton.test.tsx.snap  | 12 +-----------
 .../cypress/integration/clipboardcopy.spec.ts        | 10 +++++++++-
 3 files changed, 12 insertions(+), 20 deletions(-)

diff --git a/packages/react-core/src/components/ClipboardCopy/__tests__/ClipboardCopyButton.test.tsx b/packages/react-core/src/components/ClipboardCopy/__tests__/ClipboardCopyButton.test.tsx
index 64193765b82..00b705e020c 100644
--- a/packages/react-core/src/components/ClipboardCopy/__tests__/ClipboardCopyButton.test.tsx
+++ b/packages/react-core/src/components/ClipboardCopy/__tests__/ClipboardCopyButton.test.tsx
@@ -6,14 +6,13 @@ import { ClipboardCopyButton } from '../ClipboardCopyButton';
 import buttonStyles from '@patternfly/react-styles/css/components/Button/button';
 
 jest.mock('../../Tooltip', () => ({
-  Tooltip: ({ content, children, exitDelay, entryDelay, maxWidth, position, onTooltipHidden, ...props }) => (
-    <div data-testid="tooltip-mock" {...props}>
+  Tooltip: ({ content, children, exitDelay, entryDelay, maxWidth, position }) => (
+    <div data-testid="tooltip-mock">
       <div data-testid="tooltip-mock-content">{content}</div>
       <p>{`exitDelay: ${exitDelay}`}</p>
       <p>{`entryDelay: ${entryDelay}`}</p>
       <p>{`maxWidth: ${maxWidth}`}</p>
       <p>{`position: ${position}`}</p>
-      <p>{`onTooltipHidden: ${onTooltipHidden}`}</p>
       {children}
     </div>
   )
@@ -121,11 +120,6 @@ test('Passes position to Tooltip', () => {
 
   expect(screen.getByText('position: bottom')).toBeVisible();
 });
-test('Passes onTooltipHidden to Tooltip', () => {
-  render(<ClipboardCopyButton onTooltipHidden={() => {}} {...requiredProps} />);
-
-  expect(screen.getByText('onTooltipHidden: () => {}')).toBeVisible();
-});
 
 test('Matches snapshot', () => {
   const { asFragment } = render(<ClipboardCopyButton {...requiredProps} />);
diff --git a/packages/react-core/src/components/ClipboardCopy/__tests__/__snapshots__/ClipboardCopyButton.test.tsx.snap b/packages/react-core/src/components/ClipboardCopy/__tests__/__snapshots__/ClipboardCopyButton.test.tsx.snap
index 4311fe4d86e..37358735497 100644
--- a/packages/react-core/src/components/ClipboardCopy/__tests__/__snapshots__/ClipboardCopyButton.test.tsx.snap
+++ b/packages/react-core/src/components/ClipboardCopy/__tests__/__snapshots__/ClipboardCopyButton.test.tsx.snap
@@ -3,11 +3,7 @@
 exports[`Matches snapshot 1`] = `
 <DocumentFragment>
   <div
-    aria="none"
-    aria-live="polite"
     data-testid="tooltip-mock"
-    trigger="mouseenter focus click"
-    triggerref="[object Object]"
   >
     <div
       data-testid="tooltip-mock-content"
@@ -28,18 +24,12 @@ exports[`Matches snapshot 1`] = `
     <p>
       position: top
     </p>
-    <p>
-      onTooltipHidden: () =&gt; {
-        /* istanbul ignore next */
-        cov_274vctcufi().f[1]++;
-      }
-    </p>
     <button
       aria-disabled="false"
       aria-label="Copyable input"
       aria-labelledby="button-id text-id"
       class="pf-v5-c-button pf-m-control"
-      data-ouia-component-id="OUIA-Generated-Button-control-14"
+      data-ouia-component-id="OUIA-Generated-Button-control-13"
       data-ouia-component-type="PF5/Button"
       data-ouia-safe="true"
       id="button-id"
diff --git a/packages/react-integration/cypress/integration/clipboardcopy.spec.ts b/packages/react-integration/cypress/integration/clipboardcopy.spec.ts
index 8a19d362223..ea5f9cd51c4 100644
--- a/packages/react-integration/cypress/integration/clipboardcopy.spec.ts
+++ b/packages/react-integration/cypress/integration/clipboardcopy.spec.ts
@@ -17,12 +17,20 @@ describe('Clipboard Copy Demo Test', () => {
     cy.get('#inline-copy').should('have.class', 'pf-m-block');
   });
 
-  it('Verify clickTip renders', () => {
+  it('Verify hoverTip and clickTip render and onTooltipHidden called', () => {
     cy.get('#tooltip-test .pf-v5-c-button.pf-m-control').focus();
     // Wait for default entryDelay of 300ms
     cy.wait(500);
     cy.get('.pf-v5-c-tooltip').should('have.text', 'Copy button not clicked');
     cy.get('#tooltip-test .pf-v5-c-button.pf-m-control').click();
     cy.get('.pf-v5-c-tooltip').should('have.text', 'Copy button clicked');
+    cy.get('#tooltip-test input').focus();
+
+    // By default onTooltipHidden is called to set internal copied state to false, which
+    // would reset the tooltip content to the hoverTip prop value
+    cy.wait(500);
+    cy.get('#tooltip-test .pf-v5-c-button.pf-m-control').focus();
+    cy.wait(500);
+    cy.get('.pf-v5-c-tooltip').should('have.text', 'Copy button not clicked');
   });
 });