maverick/docbrown
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

[deploy] Show reactable username for reactable_type == User (#7527)

Browse source 
* Show reactable username for reactable_type == User

* Add test for GET /internal/reports vomit reaction on user
This commit is contained in:
Andy Zhao 2020-04-27 11:27:45 -04:00 • committed by GitHub
parent 527c98fa01
commit ae4ba673c4
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 12 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
app/views/internal/feedback_messages/_abuse_reports.html.erb
View file

@ -33,7 +33,7 @@
</span>
<span>
<strong><%= reaction.reactable_type %>:</strong>
<a href="<%= reaction.reactable.path %>"><%= reaction.reactable.title %></a>
<a href="<%= reaction.reactable.path %>"><%= reaction.reactable_type == "User" ? reaction.reactable.username : reaction.reactable.title %></a>
<% if reaction.reactable_type == "User" && reaction.reactable.banned %>
<span class="badge badge-danger">Suspended</span>
<% end %>

11
spec/requests/internal/feedback_messages_spec.rb
View file

@ -3,6 +3,7 @@ require "rails_helper"
RSpec.describe "/internal/reports", type: :request do
let(:feedback_message) { create(:feedback_message, :abuse_report) }
let(:user) { create(:user) }
let(:trusted_user) { create(:user, :trusted) }
let(:admin) { create(:user, :super_admin) }
describe "GET /internal/reports" do
@ -15,6 +16,16 @@ RSpec.describe "/internal/reports", type: :request do
expect(response.status).to eq 200
end
end
context "when there is a vomit reaction on a user" do
it "renders with status 200" do
trusted_user
create(:reaction, category: "vomit", reactable: user, user: trusted_user)
sign_in admin
get internal_reports_path
expect(response.status).to eq 200
end
end
end
describe "POST /save_status" do