Fix tests

[DISCO-139]

Author: RoyEJohnson

test/src/components/book-selector.test.js

@@ -1,6 +1,8 @@
 import React from 'react';
 import {render, screen} from '@testing-library/preact';
 import BookSelector from '~/components/book-selector/book-selector';
+import {MemoryRouter} from 'react-router-dom';
+import {it, expect} from '@jest/globals';
 
 const props = {
     prompt: 'Which textbook(s) are you currently using?',
@@ -12,8 +14,12 @@ const props = {
     }
 };
 
-it ('lists the books', async () => {
-    render(<BookSelector {...props} />);
+it('lists the books', async () => {
+    render(
+        <MemoryRouter>
+            <BookSelector {...props} />
+        </MemoryRouter>
+    );
     const checkboxes = await screen.findAllByRole('checkbox');
 
     expect(checkboxes).toHaveLength(24);

test/src/pages/confirmation.test.js

@@ -1,6 +1,8 @@
 import React from 'react';
 import {render, screen} from '@testing-library/preact';
 import Confirmation from '~/pages/confirmation/confirmation';
+import {MemoryRouter} from 'react-router-dom';
+import {test, expect} from '@jest/globals';
 
 const referrers = {
     contact: 'Thanks for contacting us',
@@ -10,7 +12,11 @@ const referrers = {
 Reflect.ownKeys(referrers).forEach((ref) => {
     test(`does ${ref} thanks`, () => {
         window.history.pushState({}, 'confirmation', `/confirmation/${ref}`);
-        render(<Confirmation />)
+        render(
+            <MemoryRouter>
+                <Confirmation />
+            </MemoryRouter>
+        );
         expect(screen.getByRole('heading').textContent).toBe(referrers[ref]);
     });
 });

test/src/pages/details/book-details-context.js

@@ -2,6 +2,7 @@ import React from 'react';
 import LoaderPage from '~/components/jsx-helpers/loader-page';
 import ShellContextProvider from '../../../helpers/shell-context';
 import {DetailsContextProvider} from '~/pages/details/context';
+import {MemoryRouter} from 'react-router-dom';
 
 function BookDetailsWithContext({data, children}) {
     return (
@@ -15,6 +16,8 @@ function BookDetailsWithContext({data, children}) {
 
 export default function BookDetailsLoader({slug, children}) {
     return (
-        <LoaderPage slug={slug} Child={BookDetailsWithContext} doDocumentSetup props={{children}} />
+        <MemoryRouter initialEntries={[slug]}>
+            <LoaderPage slug={slug} Child={BookDetailsWithContext} doDocumentSetup props={{children}} />
+        </MemoryRouter>
     );
 }

test/src/pages/details/let-us-know.test.js

@@ -8,15 +8,15 @@ const polishTitle = 'Fizyka dla szkół wyższych. Tom 1';
 
 test('handles English title', async () => {
     render(
-        <BookDetailsLoader slug={'books/college-algebra'}>
+        <BookDetailsLoader slug='books/college-algebra'>
             <LetUsKnow title={englishTitle} />
         </BookDetailsLoader>
     );
     await screen.findByText('Using this book? Let us know.');
 });
 test('handles Polish title', async () => {
     render(
-        <BookDetailsLoader slug={'books/college-algebra'}>
+        <BookDetailsLoader slug='books/college-algebra'>
             <LetUsKnow title={polishTitle} />
         </BookDetailsLoader>
     );

test/src/pages/details/resource-boxes.test.js

@@ -1,12 +1,12 @@
 import React from 'react';
 import {render, screen} from '@testing-library/preact';
 import BookDetailsLoader from './book-details-context';
-import {MemoryRouter} from 'react-router-dom';
 import ResourceBoxes from '~/pages/details/common/resource-box/resource-boxes';
 import {
     instructorResourceBoxPermissions,
     studentResourceBoxPermissions
 } from '~/pages/details/common/resource-box/resource-box';
+import {test, expect} from '@jest/globals';
 
 // Test all the conditions in here:
 // userStatus: isInstructor: true|false
@@ -25,17 +25,15 @@ const userStatus = {
 };
 const payload = {
     heading: 'This is the heading',
-    description: 'This is <b>a description</b> in HTML',
-}
+    description: 'This is <b>a description</b> in HTML'
+};
 
 function LangWrapResourceBoxes({models}) {
     // console.info('*** MODELS', models);
     return (
-        <MemoryRouter initialEntries={['/details/books/sometitle?Instructor%20resources']}>
-            <BookDetailsLoader slug='books/college-algebra'>
-                <ResourceBoxes models={models} />
-            </BookDetailsLoader>
-        </MemoryRouter>
+        <BookDetailsLoader slug='books/college-algebra'>
+            <ResourceBoxes models={models} />
+        </BookDetailsLoader>
     );
 }
 

test/src/pages/errata/errata-summary.test.js

@@ -3,7 +3,8 @@ import {render, screen} from '@testing-library/preact';
 import {within} from '@testing-library/dom';
 import userEvent from '@testing-library/user-event';
 import ErrataSummaryLoader from '~/pages/errata-summary/errata-summary';
-import Table from '~/pages/errata-summary/table/table';
+import {MemoryRouter} from 'react-router-dom';
+import {test, expect} from '@jest/globals';
 
 const searchStr = '/errata/?book=Anatomy%20and%20Physiology';
 
@@ -21,11 +22,19 @@ async function getTableRows() {
 }
 
 test('shows all items in table', async () => {
-    render(<ErrataSummaryLoader />)
+    render(
+        <MemoryRouter>
+            <ErrataSummaryLoader />
+        </MemoryRouter>
+    );
     expect(await getTableRows()).toHaveLength(54);
 });
 test('filters', async () => {
-    render(<ErrataSummaryLoader />)
+    render(
+        <MemoryRouter>
+            <ErrataSummaryLoader />
+        </MemoryRouter>
+    );
     const filters = await screen.findByRole('radiogroup');
     const user = userEvent.setup({delay: null});
     const reviewButton = within(filters).queryByText('In Review');

test/src/pages/institutional-partnership/institutional-partnership.test.js

@@ -1,11 +1,15 @@
 import React from 'react';
 import InstitutionalPartnership from '~/pages/institutional-partnership/institutional-partnership';
 import {render, screen} from '@testing-library/preact';
+import {MemoryRouter} from 'react-router-dom';
+import {describe, it, expect} from '@jest/globals';
 
 describe('InstitutionalPartnership', () => {
     it('creates', async () => {
         render(
-            <InstitutionalPartnership />
+            <MemoryRouter>
+                <InstitutionalPartnership />
+            </MemoryRouter>
         );
 
         await screen.findByText('About the program');

test/src/pages/team.test.js

@@ -1,9 +1,15 @@
 import React from 'react';
 import {render, screen} from '@testing-library/preact';
 import TeamLoader from '~/pages/team/team';
+import {MemoryRouter} from 'react-router-dom';
+import {it, expect} from '@jest/globals';
 
 it('creates with a big chunk of data', async () => {
-    render(<TeamLoader />);
+    render(
+        <MemoryRouter>
+            <TeamLoader />
+        </MemoryRouter>
+    );
     expect(await screen.findByRole('navigation'));
     expect(screen.queryAllByRole('heading').length).toBeGreaterThan(3);
 });