Commit 1ba1c213 by Peter Fogg Committed by Bill DeRusha

Fix broken certificates-related tests.

parent 8b97ca1b
......@@ -359,11 +359,14 @@ def certificates_list_handler(request, course_key_string):
course_id=course.id, include_expired=True
) if mode.slug != 'audit'
]
certificate_web_view_url = get_lms_link_for_certificate_web_view(
user_id=request.user.id,
course_key=course_key,
mode=course_modes[0] # CourseMode.modes_for_course returns default mode if doesn't find anyone.
)
if len(course_modes) > 0:
certificate_web_view_url = get_lms_link_for_certificate_web_view(
user_id=request.user.id,
course_key=course_key,
mode=course_modes[0] # CourseMode.modes_for_course returns default mode if doesn't find anyone.
)
else:
certificate_web_view_url = None
certificates = None
is_active = False
if settings.FEATURES.get('CERTIFICATES_HTML_VIEW', False):
......
......@@ -20,6 +20,7 @@ from courseware.access_response import (
StartDateError,
VisibilityError,
)
from course_modes.models import CourseMode
from student.models import CourseEnrollment
from util.milestones_helpers import (
set_prerequisite_courses,
......@@ -94,7 +95,7 @@ class TestUserEnrollmentApi(UrlResetMixin, MobileAPITestCase, MobileAuthUserTest
self.assertIn('course_info/{}/handouts'.format(self.course.id), found_course['course_handouts'])
self.assertIn('video_outlines/courses/{}'.format(self.course.id), found_course['video_outline'])
self.assertEqual(found_course['id'], unicode(self.course.id))
self.assertEqual(courses[0]['mode'], 'honor')
self.assertEqual(courses[0]['mode'], CourseMode.DEFAULT_MODE_SLUG)
self.assertEqual(courses[0]['course']['subscription_id'], self.course.clean_id(padding_char='_'))
def verify_failure(self, response, error_type=None):
......
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