Fix specs and missing features

app/components/moderation/shared/index_component.html.erb

@@ -21,18 +21,18 @@
   <%= submit_tag t("moderation.#{i18n_namespace}.index.block_authors"),
                  name: "block_authors",
                  class: "button hollow alert",
-                 data: { confirm: t("moderation.#{i18n_namespace}.index.confirm") } %>
+                 data: { confirm: t("moderation.#{i18n_namespace}.index.confirm_action", action: t("moderation.#{i18n_namespace}.index.block_authors")) } %>
 
   <div class="float-right">
     <%= submit_tag t("moderation.#{i18n_namespace}.index.hide"),
                    name: "hide_#{table_name}",
                    class: "button hollow alert",
-                   data: { confirm: t("moderation.#{i18n_namespace}.index.confirm") } %>
+                   data: { confirm: t("moderation.#{i18n_namespace}.index.confirm_action", action: t("moderation.#{i18n_namespace}.index.hide")) } %>
 
     <%= submit_tag t("moderation.#{i18n_namespace}.index.ignore_flags"),
                    name: "ignore_flags",
                    class: "button hollow",
-                   data: { confirm: t("moderation.#{i18n_namespace}.index.confirm") } %>
+                   data: { confirm: t("moderation.#{i18n_namespace}.index.confirm_action", action: t("moderation.#{i18n_namespace}.index.ignore_flags")) } %>
   </div>
 
   <%= paginate records %>

app/views/admin/poll/questions/answers/images/index.html.erb

@@ -18,6 +18,6 @@
                 class: "delete float-right",
                 method: :delete,
                 remote: true,
-                data: { confirm: t("admin.actions.confirm_action", action: t("images.remove_image")) } %>
+                data: { confirm: t("admin.actions.confirm_action", action: t("images.remove_image"), name: image.title) } %>
   </div>
 <% end %>

app/views/budgets/investments/_actions.html.erb

@@ -1,10 +1,10 @@
 <% if can? :hide, investment %>
   <%= link_to t("admin.actions.hide").capitalize, hide_moderation_budget_investment_path(investment),
-              method: :put, remote: true, data: { confirm: t("admin.actions.confirm_action", action: t("admin.actions.hide")) } %>
+              method: :put, remote: true, data: { confirm: t("admin.actions.confirm_action", action: t("admin.actions.hide"), name: investment.title) } %>
 <% end %>
 
 <% if can? :hide, investment.author %>
   &nbsp;|&nbsp;
   <%= link_to t("admin.actions.hide_author").capitalize, hide_moderation_user_path(investment.author_id),
-              method: :put, data: { confirm: t("admin.actions.confirm_action", action: t("admin.actions.hide_author")) } %>
+              method: :put, data: { confirm: t("admin.actions.confirm_action", action: t("admin.actions.hide_author"), name: investment.author.name) } %>
 <% end %>

app/views/comments/_actions.html.erb

@@ -12,13 +12,13 @@
                   data: { confirm: t("comments.actions.confirm_delete") } %>
     <% else %>
       <%= link_to t("admin.actions.hide").capitalize, hide_moderation_comment_path(comment),
-                  method: :put, remote: true, data: { confirm: t("admin.actions.confirm_action", action: t("admin.actions.hide")) } %>
+                  method: :put, remote: true, data: { confirm: t("admin.actions.confirm_action", action: t("admin.actions.hide"), name: comment.body.truncate(32)) } %>
     <% end %>
   <% end %>
 
   <% if can? :hide, comment.user %>
     <span class="divider">&nbsp;&bull;&nbsp;</span>
     <%= link_to t("admin.actions.hide_author").capitalize, hide_moderation_user_path(comment.user_id),
-                method: :put, data: { confirm: t("admin.actions.confirm_action", action: t("admin.actions.hide_author")) } %>
+                method: :put, data: { confirm: t("admin.actions.confirm_action", action: t("admin.actions.hide_author"), name: comment.author.name) } %>
   <% end %>
 </span>

app/views/debates/_actions.html.erb

@@ -1,21 +1,21 @@
 <% if can? :hide, debate %>
   <%= link_to t("admin.actions.hide").capitalize, hide_moderation_debate_path(debate),
-              method: :put, remote: true, data: { confirm: t("admin.actions.confirm_action", action: t("admin.actions.hide")) } %>
+              method: :put, remote: true, data: { confirm: t("admin.actions.confirm_action", action: t("admin.actions.hide"), name: debate.title) } %>
 <% end %>
 
 <% if can? :hide, debate.author %>
   &nbsp;|&nbsp;
   <%= link_to t("admin.actions.hide_author").capitalize, hide_moderation_user_path(debate.author_id),
-              method: :put, data: { confirm: t("admin.actions.confirm_action", action: t("admin.actions.hide_author")) } %>
+              method: :put, data: { confirm: t("admin.actions.confirm_action", action: t("admin.actions.hide_author"), name: debate.author.name) } %>
 <% end %>
 
 <% if can? :mark_featured, debate %>
   &nbsp;|&nbsp;
   <% if debate.featured? %>
     <%= link_to t("admin.actions.unmark_featured").capitalize, unmark_featured_debate_path(debate),
-              method: :put, data: { confirm: t("admin.actions.confirm_action", action: t("admin.actions.unmark_featured")) } %>
+              method: :put, data: { confirm: t("admin.actions.confirm_action", action: t("admin.actions.unmark_featured"), name: debate.title) } %>
   <% else %>
     <%= link_to t("admin.actions.mark_featured").capitalize, mark_featured_debate_path(debate),
-              method: :put, data: { confirm: t("admin.actions.confirm_action", action: t("admin.actions.mark_featured")) } %>
+              method: :put, data: { confirm: t("admin.actions.confirm_action", action: t("admin.actions.mark_featured"), name: debate.title) } %>
   <% end %>
 <% end %>

app/views/legislation/proposals/_actions.html.erb

@@ -1,10 +1,10 @@
 <% if can? :hide, proposal %>
   <%= link_to t("admin.actions.hide").capitalize, hide_moderation_legislation_proposal_path(proposal),
-              method: :put, remote: true, data: { confirm: t("admin.actions.confirm_action", action: t("admin.actions.hide")) } %>
+              method: :put, remote: true, data: { confirm: t("admin.actions.confirm_action", action: t("admin.actions.hide"), name: proposal.title) } %>
 <% end %>
 
 <% if can? :hide, proposal.author %>
   &nbsp;|&nbsp;
   <%= link_to t("admin.actions.hide_author").capitalize, hide_moderation_user_path(proposal.author_id),
-              method: :put, data: { confirm: t("admin.actions.confirm_action", action: t("admin.actions.hide_author")) } %>
+              method: :put, data: { confirm: t("admin.actions.confirm_action", action: t("admin.actions.hide_author"), name: proposal.author.name) } %>
 <% end %>

app/views/proposal_notifications/_actions.html.erb

@@ -3,13 +3,13 @@
     <span class="js-moderation-actions">
       <% if can? :hide, notification %>
         <%= link_to t("admin.actions.hide").capitalize, hide_moderation_proposal_notification_path(notification),
-                    method: :put, remote: true, data: { confirm: t("admin.actions.confirm_action", action: t("admin.actions.hide")) } %>
+                    method: :put, remote: true, data: { confirm: t("admin.actions.confirm_action", action: t("admin.actions.hide"), name: notification.notifiable_title) } %>
       <% end %>
 
       <% if can? :hide, notification.author %>
         <span class="divider">&nbsp;&bull;&nbsp;</span>
         <%= link_to t("admin.actions.hide_author").capitalize, hide_moderation_user_path(notification.author_id),
-                    method: :put, data: { confirm: t("admin.actions.confirm_action", action: t("admin.actions.hide_author")) } %>
+                    method: :put, data: { confirm: t("admin.actions.confirm_action", action: t("admin.actions.hide_author"), name: notification.author.name) } %>
       <% end %>
     </span>
   </div>

app/views/proposals/_actions.html.erb

@@ -1,10 +1,10 @@
 <% if can? :hide, proposal %>
   <%= link_to t("admin.actions.hide").capitalize, hide_moderation_proposal_path(proposal),
-              method: :put, remote: true, data: { confirm: t("admin.actions.confirm_action", action: t("admin.actions.hide")) } %>
+              method: :put, remote: true, data: { confirm: t("admin.actions.confirm_action", action: t("admin.actions.hide"), name: proposal.title) } %>
 <% end %>
 
 <% if can? :hide, proposal.author %>
   &nbsp;|&nbsp;
   <%= link_to t("admin.actions.hide_author").capitalize, hide_moderation_user_path(proposal.author_id),
-              method: :put, data: { confirm: t("admin.actions.confirm_action", action: t("admin.actions.hide_author")) } %>
+              method: :put, data: { confirm: t("admin.actions.confirm_action", action: t("admin.actions.hide_author"), name: proposal.author.name) } %>
 <% end %>

config/locales/en/moderation.yml

@@ -3,6 +3,7 @@ en:
     comments:
       index:
         block_authors: Block authors
+        confirm_action: "Are you sure? %{action}"
         filter: Filter
         filters:
           all: All
@@ -23,6 +24,7 @@ en:
     debates:
       index:
         block_authors: Block authors
+        confirm_action: "Are you sure? %{action}"
         filter: Filter
         filters:
           all: All
@@ -49,6 +51,7 @@ en:
     proposals:
       index:
         block_authors: Block authors
+        confirm_action: "Are you sure? %{action}"
         filter: Filter
         filters:
           all: All
@@ -66,6 +69,7 @@ en:
     budget_investments:
       index:
         block_authors: Block authors
+        confirm_action: "Are you sure? %{action}"
         filter: Filter
         filters:
           all: All
@@ -83,6 +87,7 @@ en:
     proposal_notifications:
       index:
         block_authors: Block authors
+        confirm_action: "Are you sure? %{action}"
         filter: Filter
         filters:
           all: All

spec/system/admin/activity_spec.rb

@@ -14,7 +14,7 @@
       visit proposal_path(proposal)
 
       within("#proposal_#{proposal.id}") do
-        accept_confirm("Hide: Are you sure?") { click_link "Hide" }
+        accept_confirm("Are you sure? Hide \"#{proposal.title}\"") { click_link "Hide" }
       end
       expect(page).to have_css("#proposal_#{proposal.id}.faded")
 
@@ -43,7 +43,7 @@
         check "proposal_#{proposal3.id}_check"
       end
 
-      accept_confirm("Hide proposals: Are you sure?") { click_button "Hide proposals" }
+      accept_confirm("Are you sure? Hide proposals") { click_button "Hide proposals" }
 
       expect(page).not_to have_content(proposal1.title)
 
@@ -60,7 +60,7 @@
       visit admin_hidden_proposals_path
 
       within("#proposal_#{proposal.id}") do
-        accept_confirm("Restore: Are you sure?") { click_link "Restore" }
+        accept_confirm("Are you sure? Restore") { click_button "Restore" }
       end
 
       expect(page).to have_content "There are no hidden proposals"
@@ -82,7 +82,7 @@
       visit debate_path(debate)
 
       within("#debate_#{debate.id}") do
-        accept_confirm("Hide: Are you sure?") { click_link "Hide" }
+        accept_confirm("Are you sure? Hide \"#{debate.title}\"") { click_link "Hide" }
       end
       expect(page).to have_css("#debate_#{debate.id}.faded")
 
@@ -110,7 +110,7 @@
         check "debate_#{debate3.id}_check"
       end
 
-      accept_confirm("Hide debates: Are you sure?") { click_button "Hide debates" }
+      accept_confirm("Are you sure? Hide debates") { click_button "Hide debates" }
 
       expect(page).not_to have_content(debate1.title)
 
@@ -127,7 +127,7 @@
       visit admin_hidden_debates_path
 
       within("#debate_#{debate.id}") do
-        accept_confirm("Restore: Are you sure?") { click_link "Restore" }
+        accept_confirm("Are you sure? Restore") { click_button "Restore" }
       end
 
       expect(page).to have_content "There are no hidden debates"
@@ -150,7 +150,7 @@
       visit debate_path(debate)
 
       within("#comment_#{comment.id}") do
-        accept_confirm("Hide: Are you sure?") { click_link "Hide" }
+        accept_confirm("Are you sure? Hide \"#{comment.body}\"") { click_link "Hide" }
         expect(page).to have_css(".faded")
       end
 
@@ -178,7 +178,7 @@
         check "comment_#{comment3.id}_check"
       end
 
-      accept_confirm("Hide comments: Are you sure?") { click_button "Hide comments" }
+      accept_confirm("Are you sure? Hide comments") { click_button "Hide comments" }
 
       expect(page).not_to have_content(comment1.body)
 
@@ -195,7 +195,7 @@
       visit admin_hidden_comments_path
 
       within("#comment_#{comment.id}") do
-        accept_confirm("Restore: Are you sure?") { click_link "Restore" }
+        accept_confirm("Are you sure? Restore") { click_button "Restore" }
       end
 
       expect(page).to have_content "There are no hidden comments"
@@ -217,7 +217,7 @@
       visit proposal_path(proposal)
 
       within("#proposal_#{proposal.id}") do
-        accept_confirm("Hide author: Are you sure?") { click_link "Hide author" }
+        accept_confirm("Are you sure? Hide author \"#{proposal.author.name}\"") { click_link "Hide author" }
 
         expect(page).to have_current_path(debates_path)
       end
@@ -266,7 +266,7 @@
         check "proposal_#{proposal3.id}_check"
       end
 
-      accept_confirm("Block authors: Are you sure?") { click_button "Block authors" }
+      accept_confirm("Are you sure? Block authors") { click_button "Block authors" }
 
       expect(page).not_to have_content(proposal1.author.username)
 
@@ -294,7 +294,7 @@
         check "debate_#{debate3.id}_check"
       end
 
-      accept_confirm("Block authors: Are you sure?") { click_button "Block authors" }
+      accept_confirm("Are you sure? Block authors") { click_button "Block authors" }
 
       expect(page).not_to have_content(debate1.author.username)
 
@@ -322,7 +322,7 @@
         check "comment_#{comment3.id}_check"
       end
 
-      accept_confirm("Block authors: Are you sure?") { click_button "Block authors" }
+      accept_confirm("Are you sure? Block authors") { click_button "Block authors" }
 
       expect(page).not_to have_content comment1.author.username
 
@@ -341,7 +341,7 @@
       visit admin_hidden_users_path
 
       within("#user_#{user.id}") do
-        accept_confirm("Restore: Are you sure?") { click_link "Restore" }
+        accept_confirm("Are you sure? Restore") { click_button "Restore" }
       end
 
       expect(page).to have_content "There are no hidden users"

spec/system/admin/admin_notifications_spec.rb

@@ -118,7 +118,7 @@
 
       visit admin_admin_notifications_path
       within("#admin_notification_#{notification.id}") do
-        accept_confirm("Delete: Are you sure?") { click_link "Delete" }
+        accept_confirm("Are you sure? This action will delete \"#{notification.title}\" and can't be undone.") { click_button "Delete" }
       end
 
       expect(page).to have_content "Notification deleted successfully"

spec/system/admin/administrators_spec.rb

@@ -36,7 +36,7 @@
     visit admin_administrators_path
 
     within "#administrator_#{user_administrator.id}" do
-      accept_confirm("Delete: Are you sure?") { click_link "Delete" }
+      accept_confirm("Are you sure? This action will delete \"#{user_administrator.name}\" and can't be undone.") { click_button "Delete" }
     end
 
     within("#administrators") do
@@ -48,7 +48,7 @@
     visit admin_administrators_path
 
     within "#administrator_#{admin.id}" do
-      accept_confirm("Delete: Are you sure?") { click_link "Delete" }
+      accept_confirm("Are you sure? This action will delete \"#{admin.name}\" and can't be undone.") { click_button "Delete" }
     end
 
     within("#error") do
@@ -112,7 +112,7 @@
       fill_in "Search user by name or email", with: administrator2.email
       click_button "Search"
 
-      accept_confirm("Delete: Are you sure?") { click_link "Delete" }
+      accept_confirm("Are you sure? This action will delete \"#{administrator2.name}\" and can't be undone.") { click_button "Delete" }
 
       expect(page).to have_content(administrator1.email)
       expect(page).not_to have_content(administrator2.email)

spec/system/admin/banners_spec.rb

@@ -161,7 +161,7 @@
   end
 
   scenario "Delete a banner" do
-    create(:banner, title: "Ugly banner",
+    banner = create(:banner, title: "Ugly banner",
                     description: "Bad text",
                     target_url:  "http://www.url.com",
                     post_started_at: (Date.current + 4.days),
@@ -173,7 +173,9 @@
 
     expect(page).to have_content "Ugly banner"
 
-    accept_confirm("Delete banner: Are you sure?") { click_link "Delete banner" }
+    accept_confirm("Are you sure? This action will delete \"#{banner.title}\" and can't be undone.") do
+      click_button "Delete"
+    end
 
     visit admin_root_path
     expect(page).not_to have_content "Ugly banner"

spec/system/admin/budget_groups_spec.rb

@@ -46,11 +46,13 @@
     end
 
     scenario "Delete a group without headings" do
-      create(:budget_group, budget: budget, name: "Nowhere")
+      group = create(:budget_group, budget: budget, name: "Nowhere")
 
       visit admin_budget_path(budget)
 
-      accept_confirm("Delete Nowhere: Are you sure?") { click_button "Delete Nowhere" }
+      accept_confirm("Are you sure? This action will delete \"#{group.name}\" and can't be undone.") do
+        click_button "Delete #{group.name}"
+      end
 
       expect(page).to have_content "Group deleted successfully"
       expect(page).not_to have_content "Nowhere"
@@ -62,8 +64,9 @@
 
       visit admin_budget_path(budget)
 
-      accept_confirm("Delete Everywhere: Are you sure?") { click_button "Delete Everywhere" }
-
+      accept_confirm("Are you sure? This action will delete \"#{group.name}\" and can't be undone.") do
+        click_button "Delete #{group.name}"
+      end
       expect(page).to have_content "You cannot delete a Group that has associated headings"
       expect(page).to have_content "Everywhere"
     end

spec/system/admin/budget_headings_spec.rb

@@ -51,7 +51,7 @@
 
       visit admin_budget_path(budget)
 
-      within("tr", text: "Lemuria") { accept_confirm("Delete: Are you sure?") { click_button "Delete" } }
+      within("tr", text: "Lemuria") { accept_confirm("Are you sure? This action will delete \"Lemuria\" and can't be undone.") { click_button "Delete" } }
 
       expect(page).to have_content "Heading deleted successfully"
       expect(page).not_to have_content "Lemuria"
@@ -62,7 +62,7 @@
       create(:budget_investment, heading: heading)
 
       visit admin_budget_path(budget)
-      within(".heading", text: "Atlantis") { accept_confirm("Delete: Are you sure?") { click_button "Delete" } }
+      within(".heading", text: "Atlantis") { accept_confirm("Are you sure? This action will delete \"Atlantis\" and can't be undone.") { click_button "Delete" } }
 
       expect(page).to have_content "You cannot delete a Heading that has associated investments"
       expect(page).to have_content "Atlantis"