feat: refactor tests

Author: vladislavkeblysh

src/courseware/course/sequence/sequence-navigation/SequenceNavigationTabs.test.jsx

@@ -54,7 +54,7 @@ describe('Sequence Navigation Tabs', () => {
     const booyah = render(<SequenceNavigationTabs {...mockData} />, { wrapWithRouter: true });
 
     // wait for links to appear so we aren't testing an empty div
-    await screen.findAllByRole('link');
+    await screen.findAllByRole('tabpanel');
 
     container = booyah.container;
 

src/generic/hooks.test.jsx

@@ -75,34 +75,15 @@ describe('Hooks', () => {
       expect(targetContent.focus).toHaveBeenCalled();
     });
 
-    test('should trigger on "Enter" key', async () => {
-      render(<TestComponent />);
-
-      const skipLink = screen.getByRole('link', { name: /skip to content/i });
-      const targetContent = screen.getByTestId('target-content');
-
-      targetContent.focus = jest.fn();
-
-      skipLink.focus();
-
-      await userEvent.keyboard('{Enter}');
-
-      await waitFor(() => {
-        expect(global.scrollTo).toHaveBeenCalledWith({
-          top: expect.any(Number), behavior: 'smooth',
-        });
-      });
-      expect(targetContent.focus).toHaveBeenCalled();
-    });
-
     test('should warn if element is not focusable', async () => {
       render(<TestComponent />);
 
-      const skipLink = screen.getByRole('link', { name: /skip to content/i });
+      const skipLink = screen.getByTestId('skip-link');
       const targetContent = screen.getByTestId('target-content');
 
-      jest.spyOn(targetContent, 'focus').mockImplementationOnce(() => {
-        throw new Error('focus is not a function');
+      Object.defineProperty(targetContent, 'focus', {
+        value: undefined,
+        configurable: true,
       });
 
       await userEvent.click(skipLink);
@@ -119,13 +100,20 @@ describe('Hooks', () => {
     });
 
     test('should warn if target element is not found', async () => {
-      render(<TestComponent />);
+      const ComponentWithoutTarget = () => {
+        useScrollToContent();
+        return (
+          <>
+            <a href="#main-content" data-testid="skip-link">Skip to content</a>
+            {/* Нет #main-content */}
+          </>
+        );
+      };
+
+      render(<ComponentWithoutTarget />);
 
       const skipLink = screen.getByRole('link', { name: /skip to content/i });
 
-      const targetElement = document.getElementById('main-content');
-      targetElement?.parentNode?.removeChild(targetElement);
-
       await userEvent.click(skipLink);
 
       await waitFor(() => {