diff --git a/app/views/locations/check_answers.html.erb b/app/views/locations/check_answers.html.erb index 642ec7f93..fdc9796a5 100644 --- a/app/views/locations/check_answers.html.erb +++ b/app/views/locations/check_answers.html.erb @@ -15,20 +15,22 @@ <%= render partial: "organisations/headings", locals: { main: "Check your answers", sub: "Add a location to #{@scheme.service_name}" } %>
-
- <%= govuk_summary_list do |summary_list| %> - <% display_location_attributes_for_check_answers(@location).each do |attr| %> - <%= summary_list.row do |row| %> - <% row.key { attr[:name] } %> - <% row.value { details_html(attr) } %> - <% row.action(text: action_text_helper(attr, @location), href: location_edit_path(@location, attr[:attribute])) %> - <% end %> - <% end %> +
+ <%= govuk_summary_list do |summary_list| %> + <% display_location_attributes_for_check_answers(@location).each do |attr| %> + <%= summary_list.row do |row| %> + <% row.key { attr[:name] } %> + <% row.value { details_html(attr) } %> + <% if LocationPolicy.new(current_user, @location).update? %> + <% row.action(text: action_text_helper(attr, @location), href: location_edit_path(@location, attr[:attribute])) %> + <% end %> <% end %> -
+ <% end %> + <% end %> +
-<% if LocationPolicy.new(current_user, Location).create? %> +<% if LocationPolicy.new(current_user, @location).create? %>
<%= govuk_button_to "Save and return to locations", scheme_location_confirm_path(@scheme, @location, route: params[:route]), method: :patch %> <%= govuk_button_link_to "Cancel", scheme_locations_path(@scheme), secondary: true %> diff --git a/spec/views/locations/check_answers.html.erb_spec.rb b/spec/views/locations/check_answers.html.erb_spec.rb index db0296027..3a5cedc15 100644 --- a/spec/views/locations/check_answers.html.erb_spec.rb +++ b/spec/views/locations/check_answers.html.erb_spec.rb @@ -54,5 +54,16 @@ RSpec.describe "locations/check_answers.html.erb" do expect(rendered).not_to have_content("Save and return to locations") end + + it "does not see change answer links" do + assign(:scheme, scheme) + assign(:location, location) + + allow(view).to receive(:current_user).and_return(user) + + render + + expect(rendered).not_to have_content("Change") + end end end