Fix flagging/unflagging in the admin section

We weren't adding the HTML id our JavaScript expects, and so the page
didn't update the flag element.
This commit is contained in:
Javi Martín
2020-07-07 21:45:40 +02:00
parent 31b65679c3
commit 4f30720593
3 changed files with 14 additions and 3 deletions

View File

@@ -2,7 +2,7 @@
<%= t("admin.header.title") %> - <%= t("admin.menu.debates") %> - <%= @debate.title %> <%= t("admin.header.title") %> - <%= t("admin.menu.debates") %> - <%= @debate.title %>
<% end %> <% end %>
<div class="debate-show"> <div id="<%= dom_id(@debate) %>" class="debate-show">
<h2><%= @debate.title %></h2> <h2><%= @debate.title %></h2>
<% if @debate.conflictive? %> <% if @debate.conflictive? %>

View File

@@ -1,12 +1,21 @@
shared_examples "flaggable" do |factory_name| shared_examples "flaggable" do |factory_name, admin: false|
include ActionView::RecordIdentifier include ActionView::RecordIdentifier
let(:user) { create(:user, :level_two) }
let(:flaggable) { create(factory_name) } let(:flaggable) { create(factory_name) }
let(:user) do
if admin
create(:administrator).user
else
create(:user, :level_two)
end
end
let(:path) do let(:path) do
if flaggable.is_a?(Comment) if flaggable.is_a?(Comment)
polymorphic_path(flaggable.commentable) polymorphic_path(flaggable.commentable)
elsif admin
admin_polymorphic_path(flaggable)
else else
polymorphic_path(flaggable) polymorphic_path(flaggable)
end end

View File

@@ -6,6 +6,8 @@ describe "Admin debates" do
login_as(admin.user) login_as(admin.user)
end end
it_behaves_like "flaggable", :debate, admin: true
scenario "Index" do scenario "Index" do
create(:debate, title: "Best beaches") create(:debate, title: "Best beaches")