Merge pull request #420 from DEFRA/unit-test-coverage

Committing Unit test coverage code change

src/server/accessibility/controller.test.js

@@ -1,4 +1,4 @@
-import { accessibilityHandler } from './controller'
+import { accessibilityController } from './controller'
 
 describe('Accessibility Handler', () => {
   const mockRequest = {
@@ -26,14 +26,22 @@ describe('Accessibility Handler', () => {
 
   it('should redirect to the Welsh version if the language is "cy"', () => {
     mockRequest.query.lang = 'cy'
-    const result = accessibilityHandler(mockRequest, mockH, mockContent)
+    const result = accessibilityController.handler(
+      mockRequest,
+      mockH,
+      mockContent
+    )
     expect(result).toBe('redirected')
     expect(mockH.redirect).toHaveBeenCalledWith('/hygyrchedd/cy?lang=cy')
   })
 
   it('should render the accessibility page with the necessary data', () => {
     mockRequest.query.lang = 'en'
-    const result = accessibilityHandler(mockRequest, mockH, mockContent)
+    const result = accessibilityController.handler(
+      mockRequest,
+      mockH,
+      mockContent
+    )
     expect(result).toBe('view rendered')
     expect(mockH.view).toHaveBeenCalledWith('accessibility/index', {
       pageTitle: 'mock pageTitle',

src/server/accessibility/cy/controller.test.js

@@ -1,4 +1,4 @@
-import { accessibilityHandler } from './controller'
+import { accessibilityController } from './controller'
 
 describe('Accessibility Handler', () => {
   const mockRequest = {
@@ -27,14 +27,46 @@ describe('Accessibility Handler', () => {
 
   it('should redirect to the English version if the language is "en"', () => {
     mockRequest.query.lang = 'en'
-    const result = accessibilityHandler(mockRequest, mockH, mockContent)
+    const result = accessibilityController.handler(
+      mockRequest,
+      mockH,
+      mockContent
+    )
     expect(result).toBe('redirected')
     expect(mockH.redirect).toHaveBeenCalledWith('/accessibility?lang=en')
   })
 
   it('should render the accessibility page with the necessary data', () => {
     mockRequest.query.lang = 'cy'
-    const result = accessibilityHandler(mockRequest, mockH, mockContent)
+    const result = accessibilityController.handler(
+      mockRequest,
+      mockH,
+      mockContent
+    )
+    expect(result).toBe('view rendered')
+    expect(mockH.view).toHaveBeenCalledWith('accessibility/index', {
+      pageTitle: 'mock pageTitle',
+      title: 'mock title',
+      heading: 'mock heading',
+      headings: 'mock headings',
+      paragraphs: 'mock paragraphs',
+      displayBacklink: false,
+      phaseBanner: 'mock phaseBanner',
+      footerTxt: 'mock footerTxt',
+      serviceName: 'mock serviceName',
+      cookieBanner: 'mock cookieBanner',
+      lang: mockRequest.query.lang
+    })
+  })
+
+  it('should render the accessibility page by Default to Welsh if language is not cy and en', () => {
+    mockRequest.query.lang = 'test'
+    mockRequest.path = '/preifatrwydd/cy'
+    const result = accessibilityController.handler(
+      mockRequest,
+      mockH,
+      mockContent
+    )
     expect(result).toBe('view rendered')
     expect(mockH.view).toHaveBeenCalledWith('accessibility/index', {
       pageTitle: 'mock pageTitle',

src/server/accessibility/cy/index.test.js

@@ -0,0 +1,12 @@
+import { accessibilityCy } from './index'
+
+describe('accessibility index plugin - cy', () => {
+  const server = {
+    route: jest.fn()
+  }
+
+  test('should register accessibility route - cy', () => {
+    accessibilityCy.plugin.register(server)
+    expect(server.route).toHaveBeenCalled()
+  })
+})

src/server/accessibility/cy/routes.test.js

@@ -0,0 +1,16 @@
+import { routes } from './routes'
+import { accessibilityController } from './controller'
+
+jest.mock('./controller')
+
+describe('Accessibility cy Routes', () => {
+  it('should define the accessibility route - cy', () => {
+    expect(routes).toEqual([
+      {
+        method: 'GET',
+        path: '/hygyrchedd/cy',
+        ...accessibilityController
+      }
+    ])
+  })
+})

src/server/accessibility/index.test.js

@@ -0,0 +1,12 @@
+import { accessibility } from './index'
+
+describe('accessibility index plugin', () => {
+  const server = {
+    route: jest.fn()
+  }
+
+  test('should register accessibility route', () => {
+    accessibility.plugin.register(server)
+    expect(server.route).toHaveBeenCalled()
+  })
+})

src/server/check-local-air-quality/controller.test.js

@@ -1,4 +1,4 @@
-import { handleHomeRequest } from '~/src/server/home/controller'
+import { homeController } from './controller'
 import { english } from '~/src/server/data/en/en.js'
 
 describe('Home Controller', () => {
@@ -18,14 +18,14 @@ describe('Home Controller', () => {
 
   it('should redirect to the Welsh version if the language is "cy"', () => {
     mockRequest.query.lang = 'cy'
-    const result = handleHomeRequest(mockRequest, mockH, mockContent)
+    const result = homeController.handler(mockRequest, mockH, mockContent)
     expect(result).toBe('redirected')
     expect(mockH.redirect).toHaveBeenCalledWith('cy')
   })
 
   it('should render the home page with the necessary data', () => {
     mockRequest.query.lang = 'en'
-    const result = handleHomeRequest(mockRequest, mockH, mockContent)
+    const result = homeController.handler(mockRequest, mockH, mockContent)
     expect(result).toBe('view rendered')
     expect(mockH.view).toHaveBeenCalledWith('home/index', {
       pageTitle: mockContent.home.pageTitle,

src/server/check-local-air-quality/cy/index.test.js

@@ -0,0 +1,12 @@
+import { homeCy } from './index'
+
+describe('homeCy index plugin - cy', () => {
+  const server = {
+    route: jest.fn()
+  }
+
+  test('should register homeCy route - cy', () => {
+    homeCy.plugin.register(server)
+    expect(server.route).toHaveBeenCalled()
+  })
+})

src/server/check-local-air-quality/index.test.js

@@ -0,0 +1,12 @@
+import { home } from './index'
+
+describe('home index plugin', () => {
+  const server = {
+    route: jest.fn()
+  }
+
+  test('should register home route', () => {
+    home.plugin.register(server)
+    expect(server.route).toHaveBeenCalled()
+  })
+})

src/server/check-local-air-quality/routes.test.js

@@ -1,5 +1,5 @@
-import { configureRoutes } from '~/src/server/home/routes'
-import { homeController } from '~/src/server/home/controller'
+import { configureRoutes } from './routes'
+import { homeController } from './controller'
 
 describe('Route Configuration', () => {
   it('should configure the routes correctly', () => {

src/server/cookies/controller.test.js

@@ -1,4 +1,4 @@
-import { cookiesHandler } from '~/src/server/cookies/controller'
+import { cookiesController } from './controller'
 import { welsh } from '~/src/server/data/cy/cy.js'
 
 describe('Cookies Handler', () => {
@@ -19,14 +19,14 @@ describe('Cookies Handler', () => {
 
   it('should redirect to the Welsh version if the language is "cy"', () => {
     mockRequest.query.lang = 'cy'
-    const result = cookiesHandler(mockRequest, mockH, mockContent)
+    const result = cookiesController.handler(mockRequest, mockH, mockContent)
     expect(result).toBe('redirected')
     expect(mockH.redirect).toHaveBeenCalledWith('/briwsion/cy?lang=cy')
   })
 
   it('should render the cookies page with the necessary data', () => {
     mockRequest.query.lang = 'en'
-    const result = cookiesHandler(mockRequest, mockH, mockContent)
+    const result = cookiesController.handler(mockRequest, mockH, mockContent)
     expect(result).toBe('view rendered')
     expect(mockH.view).toHaveBeenCalledWith('cookies/index', {
       pageTitle: mockContent.footer.cookies.pageTitle,
@@ -41,14 +41,14 @@ describe('Cookies Handler', () => {
       footerTxt: mockContent.footerTxt,
       serviceName: mockContent.multipleLocations.serviceName,
       cookieBanner: mockContent.cookieBanner,
-      lang: 'en'
+      lang: mockRequest.query.lang
     })
   })
 
   it('should default to English if language is not "cy" or "en" and path is "/cookies"', () => {
     mockRequest.query.lang = 'fr'
     mockRequest.path = '/cookies'
-    const result = cookiesHandler(mockRequest, mockH, mockContent)
+    const result = cookiesController.handler(mockRequest, mockH, mockContent)
     expect(result).toBe('view rendered')
     expect(mockH.view).toHaveBeenCalledWith('cookies/index', {
       pageTitle: mockContent.footer.cookies.pageTitle,

src/server/cookies/cy/controller.test.js

@@ -1,4 +1,4 @@
-import { cookiesHandler } from '~/src/server/cookies/controller'
+import { cookiesController } from './controller'
 import { welsh } from '~/src/server/data/cy/cy.js'
 
 describe('Cookies Handler', () => {
@@ -18,14 +18,15 @@ describe('Cookies Handler', () => {
   })
 
   it('should redirect to the English version if the language is "en"', () => {
-    mockRequest.query.lang = 'cy'
-    const result = cookiesHandler(mockRequest, mockH, mockContent)
+    mockRequest.query.lang = 'en'
+    const result = cookiesController.handler(mockRequest, mockH, mockContent)
     expect(result).toBe('redirected')
-    expect(mockH.redirect).toHaveBeenCalledWith('/briwsion/cy?lang=cy')
+    expect(mockH.redirect).toHaveBeenCalledWith('/cookies?lang=en')
   })
 
   it('should render the cookies page with the necessary data', () => {
-    const result = cookiesHandler(mockRequest, mockH, mockContent)
+    mockRequest.query.lang = 'cy'
+    const result = cookiesController.handler(mockRequest, mockH, mockContent)
     expect(result).toBe('view rendered')
     expect(mockH.view).toHaveBeenCalledWith('cookies/index', {
       pageTitle: mockContent.footer.cookies.pageTitle,
@@ -39,14 +40,15 @@ describe('Cookies Handler', () => {
       phaseBanner: mockContent.phaseBanner,
       footerTxt: mockContent.footerTxt,
       serviceName: mockContent.multipleLocations.serviceName,
-      cookieBanner: mockContent.cookieBanner
+      cookieBanner: mockContent.cookieBanner,
+      lang: mockRequest.query.lang
     })
   })
 
   it('should default to Welsh if language is not "cy" or "en" and path is "/preifatrwydd/cy"', () => {
     mockRequest.query.lang = 'fr'
     mockRequest.path = '/preifatrwydd/cy'
-    const result = cookiesHandler(mockRequest, mockH, mockContent)
+    const result = cookiesController.handler(mockRequest, mockH, mockContent)
     expect(result).toBe('view rendered')
     expect(mockH.view).toHaveBeenCalledWith('cookies/index', {
       pageTitle: mockContent.footer.cookies.pageTitle,
@@ -61,7 +63,7 @@ describe('Cookies Handler', () => {
       footerTxt: mockContent.footerTxt,
       serviceName: mockContent.multipleLocations.serviceName,
       cookieBanner: mockContent.cookieBanner,
-      lang: 'fr'
+      lang: 'cy'
     })
   })
 })

src/server/home/controller.test.js

@@ -1,4 +1,4 @@
-import { handleHomeRequest } from '~/src/server/home/controller'
+import { homeController } from './controller'
 import { english } from '~/src/server/data/en/en.js'
 
 describe('Home Controller', () => {
@@ -18,14 +18,14 @@ describe('Home Controller', () => {
 
   it('should redirect to the Welsh version if the language is "cy"', () => {
     mockRequest.query.lang = 'cy'
-    const result = handleHomeRequest(mockRequest, mockH, mockContent)
+    const result = homeController.handler(mockRequest, mockH, mockContent)
     expect(result).toBe('redirected')
     expect(mockH.redirect).toHaveBeenCalledWith('cy')
   })
 
   it('should render the home page with the necessary data', () => {
     mockRequest.query.lang = 'en'
-    const result = handleHomeRequest(mockRequest, mockH, mockContent)
+    const result = homeController.handler(mockRequest, mockH, mockContent)
     expect(result).toBe('view rendered')
     expect(mockH.view).toHaveBeenCalledWith('home/index', {
       pageTitle: mockContent.home.pageTitle,