Merge pull request #17285 from edx/sstudent/hide_cert_setting
deprecate html setting for web certs and set to true by default
This commit is contained in:
@@ -31,6 +31,8 @@ class TestHeaderMenu(CourseTestCase, UrlResetMixin):
|
||||
Tests course header menu should not have `Certificates` menu item
|
||||
if course has not web/HTML certificates enabled.
|
||||
"""
|
||||
self.course.cert_html_view_enabled = False
|
||||
self.save_course()
|
||||
outline_url = reverse_course_url('course_handler', self.course.id)
|
||||
resp = self.client.get(outline_url, HTTP_ACCEPT='text/html')
|
||||
self.assertEqual(resp.status_code, 200)
|
||||
@@ -41,8 +43,6 @@ class TestHeaderMenu(CourseTestCase, UrlResetMixin):
|
||||
Tests course header menu should have `Certificates` menu item
|
||||
if course has web/HTML certificates enabled.
|
||||
"""
|
||||
self.course.cert_html_view_enabled = True
|
||||
self.save_course()
|
||||
outline_url = reverse_course_url('course_handler', self.course.id)
|
||||
resp = self.client.get(outline_url, HTTP_ACCEPT='text/html')
|
||||
self.assertEqual(resp.status_code, 200)
|
||||
|
||||
Reference in New Issue
Block a user