Add roles to school dashboard

Make it easier to see who is a student and teacher for user support

Author: zetter-rpf

app/controllers/admin/schools_controller.rb

@@ -2,6 +2,8 @@
 
 module Admin
   class SchoolsController < Admin::ApplicationController
+    helper_method :school_role_users_by_id
+
     def verify
       service = SchoolVerificationService.new(requested_resource)
 
@@ -33,5 +35,19 @@ def default_sorting_attribute
     def default_sorting_direction
       :desc
     end
+
+    private
+
+    def school_role_users_by_id
+      @school_role_users_by_id ||= fetch_users_batch(school_role_user_ids)
+    end
+
+    def school_role_user_ids
+      requested_resource.roles.where(role: SchoolRolesField::DISPLAYED_ROLES).filter_map(&:user_id).uniq
+    end
+
+    def fetch_users_batch(user_ids)
+      User.from_userinfo(ids: user_ids).index_by(&:id)
+    end
   end
 end

app/dashboards/school_dashboard.rb

@@ -25,6 +25,7 @@ class SchoolDashboard < Administrate::BaseDashboard
     classes: Field::HasMany,
     lessons: Field::HasMany,
     projects: Field::HasMany,
+    roles: SchoolRolesField,
     reference: Field::String,
     district_name: Field::String,
     district_nces_id: Field::String,
@@ -59,6 +60,7 @@ class SchoolDashboard < Administrate::BaseDashboard
     name
     user_origin
     creator
+    roles
     creator_role
     creator_department
     reference

app/fields/school_roles_field.rb

@@ -0,0 +1,22 @@
+# frozen_string_literal: true
+
+require 'administrate/field/base'
+
+class SchoolRolesField < Administrate::Field::Base
+  DISPLAYED_ROLES = %w[owner teacher].freeze
+
+  def roles
+    @roles ||= data.where(role: DISPLAYED_ROLES).sort_by(&:created_at)
+  end
+
+  def user_display(role, users_by_id = {})
+    user = users_by_id[role.user_id]
+    user.present? ? user_dashboard.display_resource(user) : role.user_id
+  end
+
+  private
+
+  def user_dashboard
+    @user_dashboard ||= UserDashboard.new
+  end
+end

app/views/fields/school_roles_field/_show.html.erb

@@ -0,0 +1,26 @@
+<%#
+Administrate field partial for SchoolRolesField in show view
+%>
+
+<% if field.roles.any? %>
+  <table>
+    <thead>
+      <tr>
+        <th>Role</th>
+        <th>User</th>
+        <th>Created</th>
+      </tr>
+    </thead>
+    <tbody>
+      <% field.roles.each do |role| %>
+        <tr>
+          <td><%= role.role.humanize %></td>
+          <td><%= field.user_display(role, school_role_users_by_id) %></td>
+          <td><%= l(role.created_at) %></td>
+        </tr>
+      <% end %>
+    </tbody>
+  </table>
+<% else %>
+  <%= t("administrate.fields.has_many.none", default: "–") %>
+<% end %>

spec/features/admin/schools_spec.rb

@@ -77,6 +77,44 @@
     end
   end
 
+  describe 'GET #show with roles' do
+    let(:creator) { create(:user) }
+    let(:school) { create(:school, creator_id: creator.id) }
+    let(:owner) { create(:user, name: 'Olivia Owner', email: 'owner@example.com') }
+    let(:teacher) { create(:user, name: 'Tariq Teacher', email: 'teacher@example.com') }
+    let(:student) { create(:user, name: 'Sam Student', email: 'student@example.com') }
+    let(:role_users) do
+      [
+        owner,
+        teacher
+      ]
+    end
+
+    before do
+      create(:owner_role, school:, user_id: owner.id)
+      create(:teacher_role, school:, user_id: teacher.id)
+      create(:student_role, school:, user_id: student.id)
+
+      allow(User).to receive(:from_userinfo).with(ids: creator.id).and_return([creator])
+      allow(User).to receive(:from_userinfo).with(ids: contain_exactly(owner.id, teacher.id)).and_return(role_users)
+
+      get admin_school_path(school)
+    end
+
+    it 'lists owner and teacher roles with names and emails' do
+      expect(User).to have_received(:from_userinfo).with(ids: contain_exactly(owner.id, teacher.id)).once
+      expect(response.body).to include('Owner')
+      expect(response.body).to include('Olivia Owner (owner@example.com)')
+      expect(response.body).to include('Teacher')
+      expect(response.body).to include('Tariq Teacher (teacher@example.com)')
+    end
+
+    it 'does not list student roles' do
+      expect(response.body).not_to include('Sam Student')
+      expect(response.body).not_to include('student@example.com')
+    end
+  end
+
   describe 'POST #verify' do
     let(:creator) { create(:user) }
     let(:verified_at) { nil }