From 8215318c4fd693cf35aa9abaa6fecf440cfdf3e4 Mon Sep 17 00:00:00 2001
From: Yasa Akbulut <yasakbulut@gmail.com>
Date: Wed, 19 Feb 2020 14:20:10 +0100
Subject: [PATCH] Update nested NavItem tests

---
 .../shared/components/layout/NavItem.test.js  | 19 +++++++++----------
 1 file changed, 9 insertions(+), 10 deletions(-)

diff --git a/frontend/packages/shared/components/layout/NavItem.test.js b/frontend/packages/shared/components/layout/NavItem.test.js
index 997b765a12..9c892e2812 100644
--- a/frontend/packages/shared/components/layout/NavItem.test.js
+++ b/frontend/packages/shared/components/layout/NavItem.test.js
@@ -15,11 +15,6 @@ const minProps = {
   href: '/mocked',
 };
 
-const minChildProps = {
-  title: 'A child title',
-  href: '/child',
-};
-
 describe('<NavLink />', () => {
   it('renders without exploding', () => {
     const rendered = render(wrapInThemedTestApp(<NavItem {...minProps} />));
@@ -59,7 +54,8 @@ describe('<NavLink />', () => {
     const { getByText, getByTestId } = render(
       wrapInThemedTestApp(
         <NavItem {...minProps}>
-          <NavItem {...minChildProps} />
+          <NavItem title="Child 1 title" href="/child" />
+          <NavItem title="Child 2 title" href="/second-child" />
         </NavItem>,
       ),
     );
@@ -71,13 +67,15 @@ describe('<NavLink />', () => {
     const { getByText, getByTestId, queryByText } = render(
       wrapInThemedTestApp(
         <NavItem {...minProps}>
-          <NavItem {...minChildProps} />
+          <NavItem title="Child 1 title" href="/child" />
+          <NavItem title="Child 2 title" href="/second-child" />
         </NavItem>,
       ),
     );
     expect(queryByText('A child link')).not.toBeInTheDocument();
     fireEvent.click(getByTestId('expand-toggle'));
-    expect(getByText('A child title')).toBeInTheDocument();
+    expect(getByText('Child 1 title')).toBeInTheDocument();
+    expect(getByText('Child 2 title')).toBeInTheDocument();
   });
 
   it('shows nested navigation automatically if the child is active', () => {
@@ -85,11 +83,12 @@ describe('<NavLink />', () => {
     const { getByText } = render(
       wrapInThemedTestApp(
         <NavItem {...minProps}>
-          <NavItem {...minChildProps} />
+          <NavItem title="Child 1 title" href="/child" />
+          <NavItem title="Child 2 title" href="/second-child" />
         </NavItem>,
       ),
     );
     expect(getByText('A link title')).toBeInTheDocument();
-    expect(getByText('A child title')).toBeInTheDocument();
+    expect(getByText('Child 1 title')).toBeInTheDocument();
   });
 });