From bc649ea8ed91d708029013085158bf9ffcf0250b Mon Sep 17 00:00:00 2001
From: Matthew Phelan <matthew.phelan@madetech.com>
Date: Fri, 26 Nov 2021 14:52:06 +0000
Subject: [PATCH] Navigation bar and specs added

---
 app/views/layouts/application.html.erb |  8 +++++++-
 spec/features/user_spec.rb             | 16 ++++++++++++++--
 2 files changed, 21 insertions(+), 3 deletions(-)

diff --git a/app/views/layouts/application.html.erb b/app/views/layouts/application.html.erb
index cf69b955d..6943d7ae2 100644
--- a/app/views/layouts/application.html.erb
+++ b/app/views/layouts/application.html.erb
@@ -38,7 +38,13 @@
             logotype: 'GOV.UK',
             service_name: 'Share Lettings and Sales for Social Housing in England Data with DLUHC',
             service_url: '/'
-          )
+          ) do |component|
+            unless current_user.nil?
+              component.navigation_item(text: 'Your account', href: '/users/account')
+              component.navigation_item(text: 'Sign out', href: destroy_user_session_path)
+              component.navigation_item(text: 'Page C', href: '/#page-c')
+            end
+          end
       %>
     </header>
     <aside class="govuk-width-container">
diff --git a/spec/features/user_spec.rb b/spec/features/user_spec.rb
index bc23ffc73..be8a0b9c4 100644
--- a/spec/features/user_spec.rb
+++ b/spec/features/user_spec.rb
@@ -58,8 +58,13 @@ RSpec.describe "User Features" do
     end
   end
 
-  context "If a not logged in user tries to access pages that need permissions" do
-    it "redirects to log in page" do
+  context "If user not logged in" do
+    it "'Your account' link does not display" do
+      visit("/case_logs")
+      expect(page).to have_no_link("Your account")
+    end
+
+    it "tries to access account page, redirects to log in page" do
       visit("/users/account")
       expect(page).to have_content("Sign in to your account to submit CORE data")
     end
@@ -73,6 +78,13 @@ RSpec.describe "User Features" do
       click_button("Sign in")
     end
 
+    it "shows 'Your account' link in navigation if logged in and redirect to correct page" do
+      visit("/case_logs")
+      expect(page).to have_link("Your account")
+      click_link("Your account")
+      expect(page).to have_current_path("/users/account")
+    end
+
     it "main page is present and accessible" do
       visit("/users/account")
       expect(page).to have_content("Your account")