Merge pull request #74 from github/jalafel/fix-panic

Add buffer for observed panic

Author: jalafel

pkg/classroom/classroom.go

@@ -100,6 +100,14 @@ type GitHubOrganization struct {
 }
 
 func NewAssignmentList(assignments []Assignment) AssignmentList {
+	if len(assignments) == 0 {
+		return AssignmentList{
+			Assignments: []Assignment{},
+			Classroom:   Classroom{},
+			Count:       0,
+		}
+	}
+
 	classroom := assignments[0].Classroom
 	count := len(assignments)
 
@@ -111,6 +119,15 @@ func NewAssignmentList(assignments []Assignment) AssignmentList {
 }
 
 func NewAcceptedAssignmentList(assignments []AcceptedAssignment) AcceptedAssignmentList {
+	if len(assignments) == 0 {
+		return AcceptedAssignmentList{
+			AcceptedAssignments: []AcceptedAssignment{},
+			Classroom:           Classroom{},
+			Assignment:          Assignment{},
+			Count:               0,
+		}
+	}
+
 	classroom := assignments[0].Assignment.Classroom
 	assignment := assignments[0].Assignment
 	count := len(assignments)

pkg/classroom/classroom_test.go

@@ -42,6 +42,11 @@ func TestNewAssignmentList(t *testing.T) {
 
 		assert.Equal(t, assignmentList.Classroom.Name, "The Classroom")
 	})
+
+	t.Run("returns empty list when there are no assignments", func(t *testing.T) {
+		assignmentList := NewAssignmentList([]Assignment{})
+		assert.Equal(t, assignmentList.Count, 0)
+	})
 }
 
 func TestNewAcceptedAssignmentList(t *testing.T) {
@@ -92,6 +97,11 @@ func TestNewAcceptedAssignmentList(t *testing.T) {
 
 		assert.Equal(t, assignmentList.Assignment.Title, "The Assignment")
 	})
+
+	t.Run("it returns an empty list when there are no accepted assignments", func(t *testing.T) {
+		assignmentList := NewAcceptedAssignmentList([]AcceptedAssignment{})
+		assert.Equal(t, assignmentList.Count, 0)
+	})
 }
 
 func TestAssignmentLists(t *testing.T) {