Merge branch 'develop'

Author: george

app/components/Footer/__tests__/Footer.spec.js

@@ -5,8 +5,27 @@ jest.mock("react-feather/dist/icons/twitter", () => global.simpleMock("Twitter")
 jest.mock("react-feather/dist/icons/github", () => global.simpleMock("Github"))
 
 describe("<Footer />", () => {
+  const baseProps = {
+    handleHeaderFocusClick: jest.fn(),
+  }
+
+  const mountComponent = () => {
+    return mount(<Footer {...baseProps} />)
+  }
+
   it("renders", () => {
-    const wrapper = mount(<Footer handleHeaderFocusClick={jest.fn()} />)
+    // Given
+    const wrapper = mountComponent()
+    // Then
     expect(wrapper).toMatchSnapshot()
   })
+
+  it("calls handleHeaderFocusClick prop when hidden button is clicked", () => {
+    // Given
+    const wrapper = mountComponent()
+    // When
+    wrapper.find(".is-visually-hidden-focusable").simulate("click")
+    // Then
+    expect(baseProps.handleHeaderFocusClick).toHaveBeenCalled()
+  })
 })

app/components/GlobalNav/__tests__/GlobalNav.spec.js

@@ -2,17 +2,32 @@ import React from "react"
 import GlobalNav from "../GlobalNav"
 import { BrowserRouter as Router } from "react-router-dom"
 
-function NavComponent() {
-  return mount(
-    <Router>
-      <GlobalNav handleMainFocusClick={jest.fn()} />
-    </Router>
-  )
-}
-
 describe("<Nav />", () => {
+  const baseProps = {
+    handleMainFocusClick: jest.fn(),
+  }
+
+  const mountComponent = () => {
+    return mount(
+      <Router>
+        <GlobalNav {...baseProps} />
+      </Router>
+    )
+  }
+
   it("renders", () => {
-    const wrapper = NavComponent()
+    // Given
+    const wrapper = mountComponent()
+    // Then
     expect(wrapper).toMatchSnapshot()
   })
+
+  it("calls handleMainFocusClick prop when hidden button is clicked", () => {
+    // Given
+    const wrapper = mountComponent()
+    // When
+    wrapper.find(".is-visually-hidden-focusable").simulate("click")
+    // Then
+    expect(baseProps.handleMainFocusClick).toHaveBeenCalled()
+  })
 })