Commit 98eb7645 by ichuang

cleaned up instructor dashboard CSV unit test code, per https://github.com/MITx/mitx/pull/615

parent 0c3d0542
...@@ -65,22 +65,20 @@ class TestInstructorDashboardGradeDownloadCSV(ct.PageLoader): ...@@ -65,22 +65,20 @@ class TestInstructorDashboardGradeDownloadCSV(ct.PageLoader):
course = self.toy course = self.toy
url = reverse('instructor_dashboard', kwargs={'course_id': course.id}) url = reverse('instructor_dashboard', kwargs={'course_id': course.id})
msg = "url = %s\n" % url msg = "url = %s\n" % url
resp = self.client.post(url, { response = self.client.post(url, {'action': 'Download CSV of all student grades for this course',
'action': 'Download CSV of all student grades for this course', })
}) msg += "instructor dashboard download csv grades: response = '%s'\n" % response
msg += "instructor dashboard download csv grades: resp = '%s'" % resp
respstr = str(resp).replace('\r','') self.assertEqual(response['Content-Type'],'text/csv',msg)
respstr = respstr.replace('TT_2012','2012') # jenkins course_id is TT_2012_Fall instead of 2012_Fall?
#open('idtest.out','w').write(respstr)
expected_resp = '''Vary: Cookie cdisp = response['Content-Disposition'].replace('TT_2012','2012') # jenkins course_id is TT_2012_Fall instead of 2012_Fall?
Content-Type: text/csv msg += "cdisp = '%s'\n" % cdisp
Content-Disposition: attachment; filename=grades_edX/toy/2012_Fall.csv self.assertEqual(cdisp,'attachment; filename=grades_edX/toy/2012_Fall.csv',msg)
Cache-Control: no-cache, no-store, must-revalidate
"ID","Username","Full Name","edX email","External email","HW 01","HW 02","HW 03","HW 04","HW 05","HW 06","HW 07","HW 08","HW 09","HW 10","HW 11","HW 12","HW Avg","Lab 01","Lab 02","Lab 03","Lab 04","Lab 05","Lab 06","Lab 07","Lab 08","Lab 09","Lab 10","Lab 11","Lab 12","Lab Avg","Midterm","Final" body = response.content.replace('\r','')
msg += "body = '%s'\n" % body
expected_body = '''"ID","Username","Full Name","edX email","External email","HW 01","HW 02","HW 03","HW 04","HW 05","HW 06","HW 07","HW 08","HW 09","HW 10","HW 11","HW 12","HW Avg","Lab 01","Lab 02","Lab 03","Lab 04","Lab 05","Lab 06","Lab 07","Lab 08","Lab 09","Lab 10","Lab 11","Lab 12","Lab Avg","Midterm","Final"
"2","u2","Fred Weasley","view2@test.com","","0","0","0","0","0","0","0","0","0","0","0","0","0","0","0","0","0","0","0","0","0","0","0","0","0","0","0.0","0.0" "2","u2","Fred Weasley","view2@test.com","","0","0","0","0","0","0","0","0","0","0","0","0","0","0","0","0","0","0","0","0","0","0","0","0","0","0","0.0","0.0"
''' '''
self.assertEqual(body, expected_body, msg)
self.assertEqual(respstr, expected_resp, msg)
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment