5373b138e8
* Implement Assignment of Reports (#6967) * Change translation of admin.report.comment.label to "Report Comment" for clarity As we'll soon add the ability for reports to have comments on them, this clarification makes sense. * Implement notes for Reports This enables moderators to leave comments about a report whilst they work on it * Fix display of report moderation notes * Allow reports to be reopened / marked as unresolved * Redirect to reports listing upon resolution of report * Implement "resolve with note" functionality * Add inverse relationship for report notes * Remove additional database querying when loading report notes * Fix tests for reports * Fix localisations for report notes / reports
133 lines
3.9 KiB
Ruby
133 lines
3.9 KiB
Ruby
require 'rails_helper'
|
|
|
|
describe Admin::ReportsController do
|
|
render_views
|
|
|
|
let(:user) { Fabricate(:user, admin: true) }
|
|
before do
|
|
sign_in user, scope: :user
|
|
end
|
|
|
|
describe 'GET #index' do
|
|
it 'returns http success with no filters' do
|
|
specified = Fabricate(:report, action_taken: false)
|
|
Fabricate(:report, action_taken: true)
|
|
|
|
get :index
|
|
|
|
reports = assigns(:reports).to_a
|
|
expect(reports.size).to eq 1
|
|
expect(reports[0]).to eq specified
|
|
expect(response).to have_http_status(:success)
|
|
end
|
|
|
|
it 'returns http success with resolved filter' do
|
|
specified = Fabricate(:report, action_taken: true)
|
|
Fabricate(:report, action_taken: false)
|
|
|
|
get :index, params: { resolved: 1 }
|
|
|
|
reports = assigns(:reports).to_a
|
|
expect(reports.size).to eq 1
|
|
expect(reports[0]).to eq specified
|
|
|
|
expect(response).to have_http_status(:success)
|
|
end
|
|
end
|
|
|
|
describe 'GET #show' do
|
|
it 'renders report' do
|
|
report = Fabricate(:report)
|
|
|
|
get :show, params: { id: report }
|
|
|
|
expect(assigns(:report)).to eq report
|
|
expect(response).to have_http_status(:success)
|
|
end
|
|
end
|
|
|
|
describe 'PUT #update' do
|
|
describe 'with an unknown outcome' do
|
|
it 'rejects the change' do
|
|
report = Fabricate(:report)
|
|
put :update, params: { id: report, outcome: 'unknown' }
|
|
|
|
expect(response).to have_http_status(:missing)
|
|
end
|
|
end
|
|
|
|
describe 'with an outcome of `resolve`' do
|
|
it 'resolves the report' do
|
|
report = Fabricate(:report)
|
|
|
|
put :update, params: { id: report, outcome: 'resolve' }
|
|
expect(response).to redirect_to(admin_reports_path)
|
|
report.reload
|
|
expect(report.action_taken_by_account).to eq user.account
|
|
expect(report.action_taken).to eq true
|
|
end
|
|
end
|
|
|
|
describe 'with an outcome of `suspend`' do
|
|
it 'suspends the reported account' do
|
|
report = Fabricate(:report)
|
|
allow(Admin::SuspensionWorker).to receive(:perform_async)
|
|
|
|
put :update, params: { id: report, outcome: 'suspend' }
|
|
expect(response).to redirect_to(admin_reports_path)
|
|
report.reload
|
|
expect(report.action_taken_by_account).to eq user.account
|
|
expect(report.action_taken).to eq true
|
|
expect(Admin::SuspensionWorker).
|
|
to have_received(:perform_async).with(report.target_account_id)
|
|
end
|
|
end
|
|
|
|
describe 'with an outsome of `silence`' do
|
|
it 'silences the reported account' do
|
|
report = Fabricate(:report)
|
|
|
|
put :update, params: { id: report, outcome: 'silence' }
|
|
expect(response).to redirect_to(admin_reports_path)
|
|
report.reload
|
|
expect(report.action_taken_by_account).to eq user.account
|
|
expect(report.action_taken).to eq true
|
|
expect(report.target_account).to be_silenced
|
|
end
|
|
end
|
|
|
|
describe 'with an outsome of `reopen`' do
|
|
it 'reopens the report' do
|
|
report = Fabricate(:report)
|
|
|
|
put :update, params: { id: report, outcome: 'reopen' }
|
|
expect(response).to redirect_to(admin_report_path(report))
|
|
report.reload
|
|
expect(report.action_taken_by_account).to eq nil
|
|
expect(report.action_taken).to eq false
|
|
end
|
|
end
|
|
|
|
describe 'with an outsome of `assign_to_self`' do
|
|
it 'reopens the report' do
|
|
report = Fabricate(:report)
|
|
|
|
put :update, params: { id: report, outcome: 'assign_to_self' }
|
|
expect(response).to redirect_to(admin_report_path(report))
|
|
report.reload
|
|
expect(report.assigned_account).to eq user.account
|
|
end
|
|
end
|
|
|
|
describe 'with an outsome of `unassign`' do
|
|
it 'reopens the report' do
|
|
report = Fabricate(:report)
|
|
|
|
put :update, params: { id: report, outcome: 'unassign' }
|
|
expect(response).to redirect_to(admin_report_path(report))
|
|
report.reload
|
|
expect(report.assigned_account).to eq nil
|
|
end
|
|
end
|
|
end
|
|
end
|