Skip to content

Commit bbb6f4b

Browse files
committed
Fix useFocusTrap test helper shadowing
Copilot-Session: 28a6bc72-61a7-4142-933a-df469ee7461e
1 parent ee8ccad commit bbb6f4b

1 file changed

Lines changed: 8 additions & 8 deletions

File tree

packages/react/src/hooks/useFocusTrap.test.tsx

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ jest.mock('@primer/behaviors', () => ({
1010
const mockFocusTrap = jest.mocked(focusTrap)
1111

1212
describe('useFocusTrap', () => {
13-
const TestComponent = ({
13+
const ToggleFocusTrapTestComponent = ({
1414
disabled,
1515
restoreFocusOnCleanUp = false,
1616
}: {
@@ -73,15 +73,15 @@ describe('useFocusTrap', () => {
7373
const abortSpy = jest.spyOn(abortController, 'abort')
7474
mockFocusTrap.mockReturnValue(abortController)
7575

76-
const {rerender} = render(<TestComponent disabled />)
76+
const {rerender} = render(<ToggleFocusTrapTestComponent disabled />)
7777

7878
expect(mockFocusTrap).not.toHaveBeenCalled()
7979

80-
rerender(<TestComponent disabled={false} />)
80+
rerender(<ToggleFocusTrapTestComponent disabled={false} />)
8181

8282
expect(mockFocusTrap).toHaveBeenCalledTimes(1)
8383

84-
rerender(<TestComponent disabled />)
84+
rerender(<ToggleFocusTrapTestComponent disabled />)
8585

8686
expect(abortSpy).toHaveBeenCalled()
8787
})
@@ -95,15 +95,15 @@ describe('useFocusTrap', () => {
9595
const nextFocusedElement = document.createElement('button')
9696
document.body.append(nextFocusedElement)
9797

98-
const {rerender} = render(<TestComponent disabled={false} restoreFocusOnCleanUp />)
98+
const {rerender} = render(<ToggleFocusTrapTestComponent disabled={false} restoreFocusOnCleanUp />)
9999

100-
rerender(<TestComponent disabled restoreFocusOnCleanUp />)
100+
rerender(<ToggleFocusTrapTestComponent disabled restoreFocusOnCleanUp />)
101101
nextFocusedElement.focus()
102102

103103
const focusSpy = jest.spyOn(nextFocusedElement, 'focus')
104104

105-
rerender(<TestComponent disabled={false} restoreFocusOnCleanUp />)
106-
rerender(<TestComponent disabled restoreFocusOnCleanUp />)
105+
rerender(<ToggleFocusTrapTestComponent disabled={false} restoreFocusOnCleanUp />)
106+
rerender(<ToggleFocusTrapTestComponent disabled restoreFocusOnCleanUp />)
107107

108108
expect(focusSpy).toHaveBeenCalled()
109109

0 commit comments

Comments
 (0)