Commit 5839e1ca by Amir Qayyum Khan

rearrage test code to avoid confusion

parent c0fc8669
...@@ -1157,10 +1157,10 @@ class CCXCoachTabTestCase(CcxTestCase): ...@@ -1157,10 +1157,10 @@ class CCXCoachTabTestCase(CcxTestCase):
""" """
Staff can access coach dashboard on ccx course. Staff can access coach dashboard on ccx course.
""" """
staff = self.make_staff()
self.make_coach() self.make_coach()
ccx = self.make_ccx() ccx = self.make_ccx()
ccx_key = CCXLocator.from_course_locator(self.course.id, unicode(ccx.id)) ccx_key = CCXLocator.from_course_locator(self.course.id, unicode(ccx.id))
staff = self.make_staff()
with ccx_course(ccx_key) as course_ccx: with ccx_course(ccx_key) as course_ccx:
allow_access(course_ccx, staff, 'staff') allow_access(course_ccx, staff, 'staff')
...@@ -1186,10 +1186,10 @@ class CCXCoachTabTestCase(CcxTestCase): ...@@ -1186,10 +1186,10 @@ class CCXCoachTabTestCase(CcxTestCase):
""" """
Instructor can access coach dashboard on ccx course. Instructor can access coach dashboard on ccx course.
""" """
instructor = self.make_instructor()
self.make_coach() self.make_coach()
ccx = self.make_ccx() ccx = self.make_ccx()
ccx_key = CCXLocator.from_course_locator(self.course.id, unicode(ccx.id)) ccx_key = CCXLocator.from_course_locator(self.course.id, unicode(ccx.id))
instructor = self.make_instructor()
with ccx_course(ccx_key) as course_ccx: with ccx_course(ccx_key) as course_ccx:
allow_access(course_ccx, instructor, 'instructor') allow_access(course_ccx, instructor, 'instructor')
......
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