fix(tooltip): solve failing test

dhis2 · Mar 19, 2024 · eeafeef · eeafeef
1 parent 72461f2
commit eeafeef
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 7 deletions.
diff --git a/components/tooltip/src/tooltip.js b/components/tooltip/src/tooltip.js
@@ -130,6 +130,7 @@ const Tooltip = ({
                         modifiers={[offsetModifier, flipModifier, hideModifier]}
                     >
                         <div
+                            className={className}
                             id="tooltipContenDhis2Ui"
                             onMouseOver={onMouseOver}
                             onMouseOut={onMouseOut}

diff --git a/components/tooltip/src/tooltip.test.js b/components/tooltip/src/tooltip.test.js
@@ -160,23 +160,20 @@ describe('Keyboard interaction', () => {
 
         expect(document.querySelector(tooltipContentSelector)).toBe(null)
 
-        // wait for 'open delay' to elapse to open tooltip
         act(() => {
             jest.advanceTimersByTime(200)
         })
 
-        // expect tooltip to be open after delay
         const res = document.querySelector(tooltipContentSelector)
         expect(res).not.toBe(null)
         expect(res.textContent).toBe(tooltipContent)
 
         wrapper.simulate('blur')
         act(() => {
-            jest.advanceTimersByTime(200) // Assuming default closeDelay is 200ms
+            jest.advanceTimersByTime(200)
         })
         expect(document.querySelector(tooltipContentSelector)).toBe(null)
-        // this last part clears a warning about "code should be wrapped in `act(...)`"
-        // and clears the tooltip
+
         act(() => {
             jest.runAllTimers()
         })
@@ -187,10 +184,27 @@ describe('Keyboard interaction', () => {
 
         // open tooltip
         wrapper.simulate('mouseover')
+
         act(() => {
-            jest.advanceTimersByTime(200) // open the tooltip
+            jest.advanceTimersByTime(200)
         })
-        document.dispatchEvent(new KeyboardEvent('keydown', { key: 'Escape' }))
+
+        // verify tooltip is open
+        expect(document.querySelector(tooltipContentSelector)).not.toBe(null)
+
+        //Press the Escape key
+        act(() => {
+            document.dispatchEvent(
+                new KeyboardEvent('keydown', { key: 'Escape' })
+            )
+        })
+
+        // wait for close delay
+        act(() => {
+            jest.advanceTimersByTime(200)
+        })
+
+        // expect tooltip to be closed
         expect(document.querySelector(tooltipContentSelector)).toBe(null)
     })
 })