add tests

src/router/aboutRoutes.tsx

@@ -46,7 +46,7 @@ export default function AboutRoutes({
       {...(sitemap && sitemapDefaults)}
     />,
     <Route
-      key="about-not-found"
+      key="AboutNotFound"
       path="/about/*"
       render={(): ReactNode => <NotFoundPage />}
     />,

src/router/notFoundRoutes.test.tsx

@@ -1,9 +1,15 @@
 import React from 'react'
 import { render, screen } from '@testing-library/react'
 import { Router, Switch } from 'react-router-dom'
-import FragmentariumRoutes from './fragmentariumRoutes'
 import { getServices } from 'test-support/AppDriver'
 import { createMemoryHistory } from 'history'
+import AboutRoutes from './aboutRoutes'
+import FragmentariumRoutes from './fragmentariumRoutes'
+import BibliographyRoutes from './bibliographyRoutes'
+import CorpusRoutes from './corpusRoutes'
+import DictionaryRoutes from './dictionaryRoutes'
+import SignRoutes from './signRoutes'
+import ToolsRoutes from './toolsRoutes'
 
 describe('NotFoundPage rendering in FragmentariumRoutes', () => {
   const nonExistentRoutes = [
@@ -28,3 +34,144 @@ describe('NotFoundPage rendering in FragmentariumRoutes', () => {
     })
   })
 })
+
+describe('NotFoundPage rendering in AboutRoutes', () => {
+  const nonExistentAboutRoutes = [
+    '/about/non-existent-page',
+    '/about/invalid-section',
+    '/about/undefined-route',
+  ]
+  nonExistentAboutRoutes.forEach((path) => {
+    const history = createMemoryHistory({ initialEntries: [path] })
+    test(`renders NotFoundPage for "${path}"`, () => {
+      render(
+        <Router history={history}>
+          <Switch>
+            {[...AboutRoutes({ ...getServices(), sitemap: false })]}
+          </Switch>
+        </Router>
+      )
+      expect(
+        screen.getByText(/The page you are looking for does not exist./i)
+      ).toBeInTheDocument()
+    })
+  })
+})
+
+describe('NotFoundPage rendering in BibliographyRoutes', () => {
+  const nonExistentAboutRoutes = [
+    '/bibliography/search/non-existent',
+    '/bibliography/afo-register/non-existent-page',
+    '/bibliography/afo-register/invalid-section',
+    '/bibliography/afo-register/undefined-route',
+  ]
+  nonExistentAboutRoutes.forEach((path) => {
+    const history = createMemoryHistory({ initialEntries: [path] })
+    test(`renders NotFoundPage for "${path}"`, () => {
+      render(
+        <Router history={history}>
+          <Switch>
+            {[...BibliographyRoutes({ ...getServices(), sitemap: false })]}
+          </Switch>
+        </Router>
+      )
+      expect(
+        screen.getByText(/The page you are looking for does not exist./i)
+      ).toBeInTheDocument()
+    })
+  })
+})
+
+describe('NotFoundPage rendering in CorpusRoutes', () => {
+  const nonExistentAboutRoutes = [
+    '/corpus/Corpus.12345/non-existent-page',
+    '/corpus/Corpus.12345/invalid-section',
+    '/corpus/Corpus.12345/undefined-route',
+  ]
+  nonExistentAboutRoutes.forEach((path) => {
+    const history = createMemoryHistory({ initialEntries: [path] })
+    test(`renders NotFoundPage for "${path}"`, () => {
+      render(
+        <Router history={history}>
+          <Switch>
+            {[...CorpusRoutes({ ...getServices(), sitemap: false })]}
+          </Switch>
+        </Router>
+      )
+      expect(
+        screen.getByText(/The page you are looking for does not exist./i)
+      ).toBeInTheDocument()
+    })
+  })
+})
+
+describe('NotFoundPage rendering in DictionaryRoutes', () => {
+  const nonExistentAboutRoutes = [
+    '/dictionary/search/non-existent',
+    '/dictionary/Dictionary.12345/non-existent-page',
+    '/dictionary/Dictionary.12345/invalid-section',
+    '/dictionary/Dictionary.12345/undefined-route',
+  ]
+  nonExistentAboutRoutes.forEach((path) => {
+    const history = createMemoryHistory({ initialEntries: [path] })
+    test(`renders NotFoundPage for "${path}"`, () => {
+      render(
+        <Router history={history}>
+          <Switch>
+            {[...DictionaryRoutes({ ...getServices(), sitemap: false })]}
+          </Switch>
+        </Router>
+      )
+      expect(
+        screen.getByText(/The page you are looking for does not exist./i)
+      ).toBeInTheDocument()
+    })
+  })
+})
+
+describe('NotFoundPage rendering in SignRoutes', () => {
+  const nonExistentAboutRoutes = [
+    '/signs/search/non-existent',
+    '/signs/Signs.12345/non-existent-page',
+    '/signs/Signs.12345/invalid-section',
+    '/signs/Signs.12345/undefined-route',
+  ]
+  nonExistentAboutRoutes.forEach((path) => {
+    const history = createMemoryHistory({ initialEntries: [path] })
+    test(`renders NotFoundPage for "${path}"`, () => {
+      render(
+        <Router history={history}>
+          <Switch>
+            {[...SignRoutes({ ...getServices(), sitemap: false })]}
+          </Switch>
+        </Router>
+      )
+      expect(
+        screen.getByText(/The page you are looking for does not exist./i)
+      ).toBeInTheDocument()
+    })
+  })
+})
+
+describe('NotFoundPage rendering in ToolsRoutes', () => {
+  const nonExistentAboutRoutes = [
+    '/tools/date-converter/non-existent-page',
+    '/tools/date-converter/invalid-section',
+    '/tools/date-converter/undefined-route',
+  ]
+  nonExistentAboutRoutes.forEach((path) => {
+    const history = createMemoryHistory({ initialEntries: [path] })
+    test(`renders NotFoundPage for "${path}"`, () => {
+      render(
+        <Router history={history}>
+          <Switch>
+            {[...ToolsRoutes({ ...getServices(), sitemap: false })]}
+          </Switch>
+        </Router>
+      )
+      expect(
+        screen.getByText(/The page you are looking for does not exist./i)
+      ).toBeInTheDocument()
+    })
+  })
+})

src/router/toolsRoutes.tsx

@@ -88,7 +88,7 @@ export default function ToolsRoutes({
       {...(sitemap && sitemapDefaults)}
     />,
     <Route
-      key="tools-not-found"
+      key="ToolsNotFound"
       path="/tools/*"
       exact
       render={(): ReactNode => <NotFoundPage />}