Commit 70fff66e by Dillon Dumesnil

Responding to comments

parent a3a0459d
...@@ -116,16 +116,12 @@ class TestReviewXBlock(SharedModuleStoreTestCase, LoginEnrollmentTestCase): ...@@ -116,16 +116,12 @@ class TestReviewXBlock(SharedModuleStoreTestCase, LoginEnrollmentTestCase):
cls.course_actual_url = reverse( cls.course_actual_url = reverse(
'courseware_section', 'courseware_section',
kwargs={ kwargs={
'course_id': cls.course_actual.id.to_deprecated_string(), 'course_id': unicode(cls.course_actual.id),
'chapter': 'Overview', 'chapter': 'Overview',
'section': 'Welcome', 'section': 'Welcome',
} }
) )
# refresh the course from the modulestore so that it has children
# Not sure if this is actually needed or not
cls.course_actual = modulestore().get_course(cls.course_actual.id)
# Set up for the review course where the review problems are hosted # Set up for the review course where the review problems are hosted
cls.course_review = CourseFactory.create( cls.course_review = CourseFactory.create(
display_name='Review_Test_Course_REVIEW', display_name='Review_Test_Course_REVIEW',
...@@ -183,7 +179,7 @@ class TestReviewXBlock(SharedModuleStoreTestCase, LoginEnrollmentTestCase): ...@@ -183,7 +179,7 @@ class TestReviewXBlock(SharedModuleStoreTestCase, LoginEnrollmentTestCase):
cls.course_review_url = reverse( cls.course_review_url = reverse(
'courseware_section', 'courseware_section',
kwargs={ kwargs={
'course_id': cls.course_review.id.to_deprecated_string(), 'course_id': unicode(cls.course_review.id),
'chapter': 'Overview', 'chapter': 'Overview',
'section': 'Welcome', 'section': 'Welcome',
} }
......
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