test(DefinitionTooltip): fix unit tests and add coverage for Definiti…

…onTooltip (#17679) * fix: definitiontooltip initial click * fix issue where initial click would open and immediately close tooltip * fix unit tests around this functionality * test(definitiontooltip): definitiontooltip tests * remove duplicate test case --------- Co-authored-by: Ariella Gilmore <[email protected]> Co-authored-by: Preeti Bansal <[email protected]>
carbon-design-system · Oct 14, 2024 · 85771f2 · 85771f2
1 parent 32c8bef
commit 85771f2
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/packages/react/src/components/Tooltip/__tests__/DefinitionTooltip-test.js b/packages/react/src/components/Tooltip/__tests__/DefinitionTooltip-test.js
@@ -11,11 +11,15 @@ import React from 'react';
 import { DefinitionTooltip } from '../DefinitionTooltip';
 
 describe('DefinitionTooltip', () => {
-  it('should display onClick a definition provided via prop', async () => {
+  it('should display on click a definition provided via prop', async () => {
     const definition = 'Uniform Resource Locator';
     render(<DefinitionTooltip definition={definition}>URL</DefinitionTooltip>);
+    expect(screen.getByRole('button')).toHaveAttribute(
+      'aria-expanded',
+      'false'
+    );
     await userEvent.click(screen.getByText('URL'));
-    expect(screen.getByText(definition)).toBeVisible();
+    expect(screen.getByRole('button')).toHaveAttribute('aria-expanded', 'true');
   });
 
   it('should have a visible tooltip if `defaultOpen` is set to true', () => {
@@ -25,7 +29,7 @@ describe('DefinitionTooltip', () => {
         term
       </DefinitionTooltip>
     );
-    expect(screen.getByText(definition)).toBeVisible();
+    expect(screen.getByRole('button')).toHaveAttribute('aria-expanded', 'true');
   });
 
   describe('Component API', () => {