5 Words on Test Organization

Instead of this:

void testCallListReportData() {
   controller.programService = programService
   def program = Program.findByName(Keys.PROGRAM_ABC)
   assertEquals Keys.PROGRAM_ABC, program.name
   def student = Student.findByLastName(Keys.STUDENT)
   assertEquals Keys.STUDENT, student.lastName
   mockParams.id = program.id
   def reportData = controller.callListReportData()
   assertEquals Keys.PROGRAM_ABC, mockParams['PROGRAM_NAME']        
   assertNotNull reportData
   def thisMap = reportData[0]
   assertEquals student.fullName(), thisMap['STUDENT_NAME']
   assertEquals Keys.CONTACT_EMAIL, thisMap['CONTACT_EMAIL']
} 

Prefer this:

void testCallListReportData() {
   controller.programService = programService
   def program = Program.findByName(Keys.PROGRAM_ABC)
   assertEquals Keys.PROGRAM_ABC, program.name
   def student = Student.findByLastName(Keys.STUDENT)
   assertEquals Keys.STUDENT, student.lastName
   mockParams.id = program.id
  
   // test
   def reportData = controller.callListReportData()
  
   assertEquals Keys.PROGRAM_ABC, mockParams['PROGRAM_NAME']        
   assertNotNull reportData
   def thisMap = reportData[0]
   assertEquals student.fullName(), thisMap['STUDENT_NAME']
   assertEquals Keys.CONTACT_EMAIL, thisMap['CONTACT_EMAIL']
}