Commit 6a6ef0fd by Toby Lawrence

Fix some broken tests.

parent ad9498ec
...@@ -17,54 +17,51 @@ from xmodule.modulestore.django import modulestore ...@@ -17,54 +17,51 @@ from xmodule.modulestore.django import modulestore
@attr('shard_1') @attr('shard_1')
class TestNavigation(SharedModuleStoreTestCase, LoginEnrollmentTestCase): class TestNavigation(ModuleStoreTestCase, LoginEnrollmentTestCase):
""" """
Check that navigation state is saved properly. Check that navigation state is saved properly.
""" """
STUDENT_INFO = [('view@test.com', 'foo'), ('view2@test.com', 'foo')] STUDENT_INFO = [('view@test.com', 'foo'), ('view2@test.com', 'foo')]
@classmethod def setUp(self):
def setUpClass(cls): super(TestNavigation, self).setUp()
super(TestNavigation, cls).setUpClass()
cls.test_course = CourseFactory.create() self.test_course = CourseFactory.create()
cls.course = CourseFactory.create() self.course = CourseFactory.create()
cls.chapter0 = ItemFactory.create(parent=cls.course, self.chapter0 = ItemFactory.create(parent=self.course,
display_name='Overview') display_name='Overview')
cls.chapter9 = ItemFactory.create(parent=cls.course, self.chapter9 = ItemFactory.create(parent=self.course,
display_name='factory_chapter') display_name='factory_chapter')
cls.section0 = ItemFactory.create(parent=cls.chapter0, self.section0 = ItemFactory.create(parent=self.chapter0,
display_name='Welcome') display_name='Welcome')
cls.section9 = ItemFactory.create(parent=cls.chapter9, self.section9 = ItemFactory.create(parent=self.chapter9,
display_name='factory_section') display_name='factory_section')
cls.unit0 = ItemFactory.create(parent=cls.section0, self.unit0 = ItemFactory.create(parent=self.section0,
display_name='New Unit') display_name='New Unit')
cls.chapterchrome = ItemFactory.create(parent=cls.course, self.chapterchrome = ItemFactory.create(parent=self.course,
display_name='Chrome') display_name='Chrome')
cls.chromelesssection = ItemFactory.create(parent=cls.chapterchrome, self.chromelesssection = ItemFactory.create(parent=self.chapterchrome,
display_name='chromeless', display_name='chromeless',
chrome='none') chrome='none')
cls.accordionsection = ItemFactory.create(parent=cls.chapterchrome, self.accordionsection = ItemFactory.create(parent=self.chapterchrome,
display_name='accordion', display_name='accordion',
chrome='accordion') chrome='accordion')
cls.tabssection = ItemFactory.create(parent=cls.chapterchrome, self.tabssection = ItemFactory.create(parent=self.chapterchrome,
display_name='tabs', display_name='tabs',
chrome='tabs') chrome='tabs')
cls.defaultchromesection = ItemFactory.create( self.defaultchromesection = ItemFactory.create(
parent=cls.chapterchrome, parent=self.chapterchrome,
display_name='defaultchrome', display_name='defaultchrome',
) )
cls.fullchromesection = ItemFactory.create(parent=cls.chapterchrome, self.fullchromesection = ItemFactory.create(parent=self.chapterchrome,
display_name='fullchrome', display_name='fullchrome',
chrome='accordion,tabs') chrome='accordion,tabs')
cls.tabtest = ItemFactory.create(parent=cls.chapterchrome, self.tabtest = ItemFactory.create(parent=self.chapterchrome,
display_name='progress_tab', display_name='progress_tab',
default_tab='progress') default_tab='progress')
def setUp(self):
super(TestNavigation, self).setUp()
# Create student accounts and activate them. # Create student accounts and activate them.
for i in range(len(self.STUDENT_INFO)): for i in range(len(self.STUDENT_INFO)):
email, password = self.STUDENT_INFO[i] email, password = self.STUDENT_INFO[i]
......
...@@ -393,7 +393,7 @@ class TestMasqueradedGroup(StaffMasqueradeTestCase): ...@@ -393,7 +393,7 @@ class TestMasqueradedGroup(StaffMasqueradeTestCase):
group. group.
""" """
self.course.cohort_config = {'cohorted': True} self.course.cohort_config = {'cohorted': True}
self.update_course(self.course, self.test_user.id) modulestore().update_item(self.course, self.test_user.id)
cohort = CohortFactory.create(course_id=self.course.id, users=[self.test_user]) cohort = CohortFactory.create(course_id=self.course.id, users=[self.test_user])
CourseUserGroupPartitionGroup( CourseUserGroupPartitionGroup(
course_user_group=cohort, course_user_group=cohort,
......
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