Add total attendance column to workshop dashboard for local summer workshops

apps/src/code-studio/pd/workshop_dashboard/components/workshop_enrollment.jsx

@@ -14,8 +14,9 @@ export default class WorkshopEnrollment extends React.Component {
     enrollments: PropTypes.arrayOf(enrollmentShape).isRequired,
     workshopId: PropTypes.string.isRequired,
     workshopCourse: PropTypes.string.isRequired,
-    workshopSubject: PropTypes.string,
+    workshopSubject: PropTypes.string.isRequired,
     workshopDate: PropTypes.string.isRequired,
+    numSessions: PropTypes.number.isRequired,
     accountRequiredForAttendance: PropTypes.bool.isRequired,
     onDelete: PropTypes.func.isRequired,
     location: PropTypes.object,
@@ -59,6 +60,7 @@ export default class WorkshopEnrollment extends React.Component {
         onDelete={this.props.onDelete}
         workshopCourse={this.props.workshopCourse}
         workshopSubject={this.props.workshopSubject}
+        numSessions={this.props.numSessions}
       />
     );
 

apps/src/code-studio/pd/workshop_dashboard/components/workshop_enrollment_school_info.jsx

@@ -8,6 +8,7 @@ import {workshopEnrollmentStyles as styles} from '../workshop_enrollment_styles'
 const CSF = 'CS Fundamentals';
 const DEEP_DIVE = 'Deep Dive';
 const NA = 'N/A';
+const LOCAL_SUMMER = '5-day Summer';
 
 export default class WorkshopEnrollmentSchoolInfo extends React.Component {
   constructor(props) {
@@ -87,6 +88,11 @@ export default class WorkshopEnrollmentSchoolInfo extends React.Component {
           <td>{enrollment.email}</td>
           <td>{enrollment.district_name}</td>
           <td>{enrollment.school}</td>
+          {this.props.workshopSubject === LOCAL_SUMMER && (
+            <td>
+              {enrollment.attendances} / {this.props.numSessions}
+            </td>
+          )}
           {this.props.workshopCourse === CSF && (
             <td>{enrollment.role ? enrollment.role : NA}</td>
           )}
@@ -160,6 +166,9 @@ export default class WorkshopEnrollmentSchoolInfo extends React.Component {
             <th style={styles.th}>Email</th>
             <th style={styles.th}>District</th>
             <th style={styles.th}>School</th>
+            {this.props.workshopSubject === LOCAL_SUMMER && (
+              <th style={styles.th}>Total Attendance</th>
+            )}
             {this.props.workshopCourse === CSF && (
               <th style={styles.th}>Current Role</th>
             )}
@@ -198,5 +207,6 @@ WorkshopEnrollmentSchoolInfo.propTypes = {
   accountRequiredForAttendance: PropTypes.bool.isRequired,
   onDelete: PropTypes.func.isRequired,
   workshopCourse: PropTypes.string.isRequired,
-  workshopSubject: PropTypes.string
+  workshopSubject: PropTypes.string.isRequired,
+  numSessions: PropTypes.number.isRequired
 };

apps/src/code-studio/pd/workshop_dashboard/workshop.jsx

@@ -760,6 +760,7 @@ export class Workshop extends React.Component {
           workshopCourse={this.state.workshop.course}
           workshopSubject={this.state.workshop.subject}
           workshopDate={firstSessionDate}
+          numSessions={this.state.workshop.sessions.length}
           enrollments={this.state.enrollments}
           onDelete={this.handleDeleteEnrollment}
           accountRequiredForAttendance={

apps/src/code-studio/soundLibrary.json

@@ -3724,4 +3724,4 @@
       "category_male_voiceover/you_win_male"
     ]
   }
-}
+}

apps/test/unit/code-studio/pd/workshop_dashboard/components/workshop_enrollment_school_infoTest.js

@@ -0,0 +1,54 @@
+import WorkshopEnrollmentSchoolInfo from '@cdo/apps/code-studio/pd/workshop_dashboard/components/workshop_enrollment_school_info';
+import React from 'react';
+import {expect} from 'chai';
+import {shallow} from 'enzyme';
+
+describe('Workshop Enrollment School Info', () => {
+  const fakeRouter = {
+    createHref() {}
+  };
+
+  const context = {
+    router: fakeRouter
+  };
+
+  it('shows Total Attendance column for local summer workshop', () => {
+    let workshopEnrollmentSchoolInfo = shallow(
+      <WorkshopEnrollmentSchoolInfo
+        enrollments={[]}
+        accountRequiredForAttendance={true}
+        onDelete={() => {}}
+        workshopCourse="CS Principles"
+        workshopSubject="5-day Summer"
+        numSessions={5}
+      />,
+      {context}
+    );
+
+    expect(
+      workshopEnrollmentSchoolInfo
+        .find('th')
+        .filterWhere(col => col.text().includes('Total Attendance'))
+    ).to.have.length(1);
+  });
+
+  it('does not show Total Attendance column for non-local-summer workshop', () => {
+    let workshopEnrollmentSchoolInfo = shallow(
+      <WorkshopEnrollmentSchoolInfo
+        enrollments={[]}
+        accountRequiredForAttendance={true}
+        onDelete={() => {}}
+        workshopCourse="CS Discoveries"
+        workshopSubject="1-day Academic Year, Units 1 and 2"
+        numSessions={5}
+      />,
+      {context}
+    );
+
+    expect(
+      workshopEnrollmentSchoolInfo
+        .find('th')
+        .filterWhere(col => col.text().includes('Total Attendance'))
+    ).to.have.length(0);
+  });
+});

dashboard/app/serializers/api/v1/pd/workshop_enrollment_serializer.rb

@@ -1,5 +1,8 @@
 class Api::V1::Pd::WorkshopEnrollmentSerializer < ActiveModel::Serializer
-  attributes :id, :first_name, :last_name, :email, :district_name, :school, :role, :grades_teaching, :attended_csf_intro_workshop, :csf_course_experience, :csf_courses_planned, :csf_has_physical_curriculum_guide, :user_id, :attended, :pre_workshop_survey
+  attributes :id, :first_name, :last_name, :email, :district_name, :school, :role,
+    :grades_teaching, :attended_csf_intro_workshop, :csf_course_experience,
+    :csf_courses_planned, :csf_has_physical_curriculum_guide, :user_id, :attended,
+    :pre_workshop_survey, :attendances
 
   def user_id
     user = object.resolve_user
@@ -25,4 +28,8 @@ def pre_workshop_survey
       )
     end
   end
+
+  def attendances
+    object.attendances.count
+  end
 end

dashboard/test/factories/pd_factories.rb

@@ -565,6 +565,7 @@
   factory :pd_attendance, class: 'Pd::Attendance' do
     association :session, factory: :pd_session
     association :teacher
+    association :enrollment
   end
 
   factory :pd_attendance_no_account, class: 'Pd::Attendance' do

dashboard/test/serializers/api/v1/pd/workshop_enrollment_serializer_test.rb

@@ -0,0 +1,23 @@
+require 'test_helper'
+
+class Api::V1::Pd::WorkshopEnrollmentSerializerTest < ::ActionController::TestCase
+  test 'serialized workshop enrollment has expected attributes' do
+    enrollment = create :pd_enrollment
+    expected_attributes = [
+      :id, :first_name, :last_name, :email, :district_name, :school, :role,
+      :grades_teaching, :user_id, :attended, :pre_workshop_survey, :attendances
+    ]
+
+    serialized = ::Api::V1::Pd::WorkshopEnrollmentSerializer.new(enrollment).attributes
+    assert_equal expected_attributes, serialized.keys
+  end
+
+  test 'attendances' do
+    workshop = create :pd_workshop, num_sessions: 5
+    enrollment = create :pd_enrollment, workshop: workshop
+    create :pd_attendance, session: workshop.sessions.first, enrollment: enrollment
+
+    serialized = ::Api::V1::Pd::WorkshopEnrollmentSerializer.new(workshop.enrollments.first).attributes
+    assert_equal enrollment.attendances.count, serialized[:attendances]
+  end
+end