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

@@ -98,11 +98,12 @@ describe('Hooks', () => {
     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 +120,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(() => {