Browse Source

checking the back functionality for secondary group confirm yes

pull/671/head
JG 3 years ago
parent
commit
e48a3944d1
  1. 2
      app/views/schemes/primary_client_group.html.erb
  2. 57
      spec/features/schemes_spec.rb

2
app/views/schemes/primary_client_group.html.erb

@ -3,7 +3,7 @@
<% content_for :before_content do %> <% content_for :before_content do %>
<%= govuk_back_link( <%= govuk_back_link(
text: "Back", text: "Back",
href: "/schemes/#{@scheme.id}/details", href: request.query_parameters["check_answers"] ? "/schemes/#{@scheme.id}/check-answers" : "/schemes/#{@scheme.id}/details",
) %> ) %>
<% end %> <% end %>

57
spec/features/schemes_spec.rb

@ -365,15 +365,15 @@ RSpec.describe "Schemes scheme Features" do
end end
context "when changing answers" do context "when changing answers" do
it "displays change links" do
assert_selector "a", text: "Change", count: 12
end
context "changing details" do context "changing details" do
before do before do
click_link("Change", href: "/schemes/#{scheme.id}/details?check_answers=true", match: :first) click_link("Change", href: "/schemes/#{scheme.id}/details?check_answers=true", match: :first)
end end
it "displays change links" do
assert_selector "a", text: "Change", count: 12
end
it "allows changing details questions" do it "allows changing details questions" do
expect(page).to have_current_path("/schemes/#{scheme.id}/details?check_answers=true") expect(page).to have_current_path("/schemes/#{scheme.id}/details?check_answers=true")
@ -400,8 +400,11 @@ RSpec.describe "Schemes scheme Features" do
end end
context "changing primary client group" do context "changing primary client group" do
it "allows changing primary-client-group question" do before do
click_link("Change", href: "/schemes/#{scheme.id}/primary-client-group?check_answers=true") click_link("Change", href: "/schemes/#{scheme.id}/primary-client-group?check_answers=true")
end
it "allows changing primary-client-group question" do
expect(page).to have_current_path("/schemes/#{scheme.id}/primary-client-group?check_answers=true") expect(page).to have_current_path("/schemes/#{scheme.id}/primary-client-group?check_answers=true")
choose "Older people with support needs" choose "Older people with support needs"
@ -423,19 +426,12 @@ RSpec.describe "Schemes scheme Features" do
end end
end end
it "allows changing secondary-client-group question" do context "changing confirm secondary group answer" do
click_link("Change", href: "/schemes/#{scheme.id}/secondary-client-group?check_answers=true") before do
expect(page).to have_current_path("/schemes/#{scheme.id}/secondary-client-group?check_answers=true") click_link("Change", href: "/schemes/#{scheme.id}/confirm-secondary-client-group?check_answers=true")
choose "People at risk of domestic violence"
click_button "Save and continue"
expect(page).to have_current_path("/schemes/#{scheme.id}/check-answers")
expect(page).to have_content "People at risk of domestic violence"
end end
it "allows changing confirm-secondary-client-group question to yes" do it "allows changing confirm-secondary-client-group question to yes" do
click_link("Change", href: "/schemes/#{scheme.id}/confirm-secondary-client-group?check_answers=true")
expect(page).to have_current_path("/schemes/#{scheme.id}/confirm-secondary-client-group?check_answers=true") expect(page).to have_current_path("/schemes/#{scheme.id}/confirm-secondary-client-group?check_answers=true")
choose "Yes" choose "Yes"
@ -450,8 +446,25 @@ RSpec.describe "Schemes scheme Features" do
expect(page).to have_content "People at risk of domestic violence" expect(page).to have_content "People at risk of domestic violence"
end end
it "allows changing confirm-secondary-client-group question to no" do context "when I press the back button" do
before do
click_link "Back"
end
it "lets me select the support answers" do
expect(page).to have_current_path("/schemes/#{scheme.id}/check-answers")
expect(page).to have_content "Check your changes before updating this scheme"
end
end
end
context "allows changing confirm-secondary-client-group question to no" do
before do
click_link("Change", href: "/schemes/#{scheme.id}/confirm-secondary-client-group?check_answers=true") click_link("Change", href: "/schemes/#{scheme.id}/confirm-secondary-client-group?check_answers=true")
end
it "allows changing confirm-secondary-client-group question to no" do
expect(page).to have_current_path("/schemes/#{scheme.id}/confirm-secondary-client-group?check_answers=true") expect(page).to have_current_path("/schemes/#{scheme.id}/confirm-secondary-client-group?check_answers=true")
choose "No" choose "No"
@ -460,6 +473,18 @@ RSpec.describe "Schemes scheme Features" do
expect(page).to have_current_path("/schemes/#{scheme.id}/check-answers") expect(page).to have_current_path("/schemes/#{scheme.id}/check-answers")
expect(page).to have_content "None" expect(page).to have_content "None"
end end
end
it "allows changing secondary-client-group question" do
click_link("Change", href: "/schemes/#{scheme.id}/secondary-client-group?check_answers=true")
expect(page).to have_current_path("/schemes/#{scheme.id}/secondary-client-group?check_answers=true")
choose "People at risk of domestic violence"
click_button "Save and continue"
expect(page).to have_current_path("/schemes/#{scheme.id}/check-answers")
expect(page).to have_content "People at risk of domestic violence"
end
it "allows changing support questions" do it "allows changing support questions" do
click_link("Change", href: "/schemes/#{scheme.id}/support?check_answers=true", match: :first) click_link("Change", href: "/schemes/#{scheme.id}/support?check_answers=true", match: :first)

Loading…
Cancel
Save