Add reporting and tests

Author: ArtOfCode-

app/controllers/complaints_controller.rb

@@ -2,7 +2,7 @@ class ComplaintsController < ApplicationController
   before_action :set_complaint, only: [:show, :comment, :self_assign, :update_status]
   before_action :access_check, only: [:show, :comment]
   before_action :write_access_check, only: [:self_assign, :update_status]
-  before_action :verify_staff, only: [:reports]
+  before_action :verify_staff, only: [:reports, :reporting]
 
   def index
     render layout: 'without_sidebar'
@@ -150,6 +150,17 @@ def update_status
     redirect_to complaint_path(@complaint.access_token)
   end
 
+  def reporting
+    @total = Complaint.recent(12.months.ago).count
+    @by_type = Complaint.recent(12.months.ago).group(:report_type).group_by_month(:created_at).count
+    @by_content_type = Complaint.recent(12.months.ago).where(report_type: 'illegal')
+                             .group(:content_type).group_by_month(:created_at).count
+    @by_outcome = Complaint.recent(12.months.ago).where.not(outcome: nil)
+                           .group(:outcome).group_by_month(:created_at).count
+    @type_totals = Complaint.recent(12.months.ago).group(:report_type).count
+    render layout: 'without_sidebar'
+  end
+
   private
 
   def access_check

app/models/complaint.rb

@@ -1,4 +1,6 @@
 class Complaint < ApplicationRecord
+  include Timestamped
+
   belongs_to :user, required: false
   belongs_to :assignee, required: false, class_name: 'User'
   has_many :comments, class_name: 'ComplaintComment', dependent: :destroy

app/views/complaints/index.html.erb

@@ -5,7 +5,7 @@
 </p>
 
 <div class="grid">
-  <div class="grid--cell is-12 is-6-md is-6-lg">
+  <div class="grid--cell is-12 is-6-lg">
     <div class="widget">
       <div class="widget--body">
         <h2 class="has-font-size-larger h-m-t-0">
@@ -23,7 +23,7 @@
 
 <% if user_signed_in? && current_user.staff? %>
   <div class="grid">
-    <div class="grid--cell is-12 is-6-md is-6-lg">
+    <div class="grid--cell is-12 is-6-lg">
       <div class="widget">
         <div class="widget--body">
           <h2 class="has-font-size-larger h-m-t-0">
@@ -37,5 +37,19 @@
         </div>
       </div>
     </div>
+    <div class="grid--cell is-12 is-6-lg">
+      <div class="widget">
+        <div class="widget--body">
+          <h2 class="has-font-size-larger h-m-t-0">
+            <%= link_to complaints_reporting_path do %>
+              Reporting &raquo;
+            <% end %>
+          </h2>
+          <p class="has-font-size-caption">
+            Charts and statistics reporting about user complaints.
+          </p>
+        </div>
+      </div>
+    </div>
   </div>
 <% end %>

app/views/complaints/reporting.html.erb

@@ -0,0 +1,28 @@
+<%= link_to safety_center_path do %>
+  &laquo; Back to Safety Center
+<% end %>
+
+<h1>Reporting</h1>
+
+<div class="flex-row">
+  <%= stat_panel 'total reports', @total %>
+</div>
+
+<div class="grid">
+  <div class="grid--cell is-12 is-6-lg">
+    <h2 class="has-font-size-base has-text-align-center">Report types by month, last 12 months</h2>
+    <%= line_chart @by_type %>
+  </div>
+  <div class="grid--cell is-12 is-6-lg">
+    <h2 class="has-font-size-base has-text-align-center">Content types by month, last 12 months</h2>
+    <%= line_chart @by_content_type %>
+  </div>
+  <div class="grid--cell is-12 is-6-lg">
+    <h2 class="has-font-size-base has-text-align-center">Outcomes by month, last 12 months</h2>
+    <%= line_chart @by_outcome %>
+  </div>
+  <div class="grid--cell is-12 is-6-lg">
+    <h2 class="has-font-size-base has-text-align-center">Totals by report type, last 12 months</h2>
+    <%= column_chart @type_totals %>
+  </div>
+</div>

config/routes.rb

@@ -383,6 +383,7 @@
     post   'report/:token/assign',         to: 'complaints#self_assign', as: :complaint_self_assign
     post   'report/:token/status',         to: 'complaints#update_status', as: :update_complaint_status
     get    'reports',                      to: 'complaints#reports', as: :complaints
+    get    'reporting',                    to: 'complaints#reporting', as: :complaints_reporting
   end
 
   get   '403',                             to: 'errors#forbidden'

test/controllers/complaints_controller_test.rb

@@ -288,6 +288,25 @@ class ComplaintsControllerTest < ActionDispatch::IntegrationTest
     assert_response(:not_found)
   end
 
+  test 'reporting should work for staff' do
+    sign_in users(:staff)
+    try_reporting
+    assert_response(:success)
+    assert_not_nil assigns(:by_type)
+    assert_not_nil assigns(:by_content_type)
+    assert_not_nil assigns(:by_outcome)
+  end
+
+  test 'reporting should not be accessible to anonymous' do
+    try_reporting
+    assert_response(:not_found)
+  end
+
+  test 'reporting should not be accessible to basic user' do
+    sign_in users(:basic_user)
+    try_reporting
+  end
+
   private
 
   def try_create_report(**params)
@@ -316,4 +335,8 @@ def try_update_status(complaint_sym, status, outcome: nil)
   def try_reports(status: nil, report_type: nil, outcome: nil)
     get complaints_path, params: { status: status, report_type: report_type, outcome: outcome }
   end
+
+  def try_reporting
+    get complaints_reporting_path
+  end
 end