Browse Source

Update delete button to use delete controller

pull/1752/head
Kat 3 years ago
parent
commit
167b90bd9d
  1. 11
      app/views/logs/delete_duplicates.html.erb
  2. 17
      spec/features/lettings_log_spec.rb

11
app/views/logs/delete_duplicates.html.erb

@ -1,5 +1,5 @@
<% content_for :before_content do %> <% content_for :before_content do %>
<% content_for :title, "Are you sure you want to delete #{@duplicate_logs.count == 1 ? "this duplicate log" : "these duplicate logs"}?" %> <% content_for :title, "Are you sure you want to delete #{@duplicate_logs.count == 1 ? 'this duplicate log' : 'these duplicate logs'}?" %>
<%= govuk_back_link href: @log.lettings? ? lettings_logs_path(@log) : sales_logs_path(@log) %> <%= govuk_back_link href: @log.lettings? ? lettings_logs_path(@log) : sales_logs_path(@log) %>
<% end %> <% end %>
@ -16,13 +16,14 @@
</p> </p>
<% @duplicate_logs.each do |duplicate_log| %> <% @duplicate_logs.each do |duplicate_log| %>
<%= govuk_link_to "Log #{duplicate_log.id}", lettings_log_path(duplicate_log)%> <%= govuk_link_to "Log #{duplicate_log.id}", lettings_log_path(duplicate_log) %>
<% end %> <% end %>
<div class="govuk-button-group"> <div class="govuk-button-group">
<%= govuk_button_to( <%= govuk_button_to @duplicate_logs.count == 1 ? "Delete this log" : "Delete these logs",
@duplicate_logs.count == 1 ? "Delete this log" : "Delete these logs" delete_logs_lettings_logs_path,
) %> method: "delete",
params: { ids: @duplicate_logs.map(&:id) } %>
<%= govuk_button_link_to( <%= govuk_button_link_to(
"Cancel", "Cancel",
@log.lettings? ? lettings_log_path(@log) : sales_log_path(@log), @log.lettings? ? lettings_log_path(@log) : sales_log_path(@log),

17
spec/features/lettings_log_spec.rb

@ -389,5 +389,22 @@ RSpec.describe "Lettings Log Features" do
expect(page).to have_current_path("/lettings-logs") expect(page).to have_current_path("/lettings-logs")
end end
end end
context "when there are duplicate logs" do
let(:lettings_log) { create(:lettings_log, :completed, owning_organisation: user.organisation, created_by: user) }
let!(:duplicate_log) do
duplicate = lettings_log.dup
duplicate.id = nil
duplicate.save!
duplicate
end
it "is possible to delete duplicate logs" do
visit lettings_log_delete_duplicates_path(lettings_log)
click_button "Delete this log"
duplicate_log.reload
expect(duplicate_log.status).to eq("deleted")
end
end
end end
end end

Loading…
Cancel
Save