From 173e79bdfafb0b8c8eba93529c45d8da101f4fcb Mon Sep 17 00:00:00 2001
From: Jacob Herrington <jacobherringtondeveloper@gmail.com>
Date: Tue, 20 Oct 2020 17:49:17 -0500
Subject: [PATCH] Add a button to delete pages (#10971)

---
 app/views/admin/pages/_form.html.erb          |  6 ++++
 spec/system/admin/admin_manages_pages_spec.rb | 28 +++++++++++++------
 2 files changed, 26 insertions(+), 8 deletions(-)

diff --git a/app/views/admin/pages/_form.html.erb b/app/views/admin/pages/_form.html.erb
index 4763d0c21..69cc05aa4 100644
--- a/app/views/admin/pages/_form.html.erb
+++ b/app/views/admin/pages/_form.html.erb
@@ -60,6 +60,12 @@
     </div>
     <%= form.submit class: "btn btn-primary" %>
   <% end %>
+
+  <% if @page.persisted? %>
+    <%= form_with model: [:admin, @page], local: true, method: :delete, class: "mt-3" do |f| %>
+      <%= f.submit "Delete Page", class: "btn btn-danger", data: { confirm: "Are you sure?" } %>
+    <% end %>
+  <% end %>
 </div>
 
 <script>
diff --git a/spec/system/admin/admin_manages_pages_spec.rb b/spec/system/admin/admin_manages_pages_spec.rb
index 94bee3325..bb535377d 100644
--- a/spec/system/admin/admin_manages_pages_spec.rb
+++ b/spec/system/admin/admin_manages_pages_spec.rb
@@ -1,10 +1,15 @@
 require "rails_helper"
 
 RSpec.describe "Admin manages pages", type: :system do
-
   let(:admin) { create(:user, :super_admin) }
 
   before do
+    create(:page,
+           slug: "test-page",
+           body_html: "<div>hello there</div>",
+           title: "Test Page",
+           description: "A test page",
+           is_top_level_path: true)
     sign_in admin
     visit admin_pages_path
   end
@@ -43,11 +48,20 @@ RSpec.describe "Admin manages pages", type: :system do
 
     it "does not show any of the links in the pages table" do
       within(".pages__table") do
-        expect(page).to_not have_content("Terms of Use")
-        expect(page).to_not have_content("Code of Conduct")
-        expect(page).to_not have_content("Privacy Policy")
+        expect(page).not_to have_content("Terms of Use")
+        expect(page).not_to have_content("Code of Conduct")
+        expect(page).not_to have_content("Privacy Policy")
       end
     end
+
+    it "allows a page to be deleted" do
+      expect(page).to have_content("Test Page")
+      click_on("Edit")
+      expect(page).to have_selector("input[type=submit][value='Delete Page']")
+      click_on("Delete Page")
+      expect(page).to have_current_path(admin_pages_path)
+      expect(page).not_to have_content("Test Page")
+    end
   end
 
   describe "when the defaults are overridden" do
@@ -62,15 +76,13 @@ RSpec.describe "Admin manages pages", type: :system do
              slug: "privacy",
              body_html: "<div>Privacy Policy</div>",
              title: "Privacy Policy",
-             description: "A page that describes the privacy policy", is_top_level_path: true
-            )
+             description: "A page that describes the privacy policy", is_top_level_path: true)
       create(:page,
              slug: "terms",
              body_html: "<div>Terms of Use</div>",
              title: "Terms of Use",
              description: "A page that describes the terms of use for the application",
-             is_top_level_path: true
-            )
+             is_top_level_path: true)
       sign_in admin
       visit admin_pages_path
     end