Merge pull request #10776 from edx/raees/chrome-bok-choy-test-split-test-lms-staff-view
Fixed test studio split test failing on chrome
This commit is contained in:
@@ -333,9 +333,9 @@ class GroupConfigurationsTest(ContainerBase, SplitTestMixin):
|
||||
return config, split_test
|
||||
return config
|
||||
|
||||
def publish_unit_in_LMS_and_view(self, courseware_page):
|
||||
def publish_unit_in_lms_and_view(self, courseware_page, publish=True):
|
||||
"""
|
||||
Given course outline page, publish first unit and view it in LMS
|
||||
Given course outline page, publish first unit and view it in LMS when publish is false, it will only view
|
||||
"""
|
||||
self.outline_page.visit()
|
||||
self.outline_page.expand_all_subsections()
|
||||
@@ -343,7 +343,8 @@ class GroupConfigurationsTest(ContainerBase, SplitTestMixin):
|
||||
unit = section.subsection_at(0).unit_at(0).go_to()
|
||||
|
||||
# I publish and view in LMS and it is rendered correctly
|
||||
unit.publish_action.click()
|
||||
if publish:
|
||||
unit.publish_action.click()
|
||||
unit.view_published_version()
|
||||
self.assertEqual(len(self.browser.window_handles), 2)
|
||||
courseware_page.wait_for_page()
|
||||
@@ -1033,11 +1034,11 @@ class GroupConfigurationsTest(ContainerBase, SplitTestMixin):
|
||||
config.edit_warning_message_text
|
||||
)
|
||||
|
||||
def publish_unit_and_verify_groups_in_LMS(self, courseware_page, group_names):
|
||||
def publish_unit_and_verify_groups_in_lms(self, courseware_page, group_names, publish=True):
|
||||
"""
|
||||
Publish first unit in LMS and verify that Courseware page has given Groups
|
||||
"""
|
||||
self.publish_unit_in_LMS_and_view(courseware_page)
|
||||
self.publish_unit_in_lms_and_view(courseware_page, publish)
|
||||
self.assertEqual(u'split_test', courseware_page.xblock_component_type())
|
||||
self.assertTrue(courseware_page.q(css=".split-test-select").is_present())
|
||||
rendered_group_names = self.get_select_options(page=courseware_page, selector=".split-test-select")
|
||||
@@ -1064,7 +1065,7 @@ class GroupConfigurationsTest(ContainerBase, SplitTestMixin):
|
||||
|
||||
# render in LMS correctly
|
||||
courseware_page = CoursewarePage(self.browser, self.course_id)
|
||||
self.publish_unit_and_verify_groups_in_LMS(courseware_page, [u'Group A', u'Group B', u'Group C'])
|
||||
self.publish_unit_and_verify_groups_in_lms(courseware_page, [u'Group A', u'Group B', u'Group C'])
|
||||
|
||||
# I go to group configuration and delete group
|
||||
self.page.visit()
|
||||
@@ -1078,7 +1079,11 @@ class GroupConfigurationsTest(ContainerBase, SplitTestMixin):
|
||||
self.browser.switch_to_window(self.browser.window_handles[0])
|
||||
|
||||
# render in LMS to see how inactive vertical is rendered
|
||||
self.publish_unit_and_verify_groups_in_LMS(courseware_page, [u'Group A', u'Group B', u'Group ID 2 (inactive)'])
|
||||
self.publish_unit_and_verify_groups_in_lms(
|
||||
courseware_page,
|
||||
[u'Group A', u'Group B', u'Group ID 2 (inactive)'],
|
||||
publish=False
|
||||
)
|
||||
|
||||
self.browser.close()
|
||||
self.browser.switch_to_window(self.browser.window_handles[0])
|
||||
@@ -1088,4 +1093,4 @@ class GroupConfigurationsTest(ContainerBase, SplitTestMixin):
|
||||
container.delete(0)
|
||||
|
||||
# render in LMS again
|
||||
self.publish_unit_and_verify_groups_in_LMS(courseware_page, [u'Group A', u'Group B'])
|
||||
self.publish_unit_and_verify_groups_in_lms(courseware_page, [u'Group A', u'Group B'])
|
||||
|
||||
Reference in New Issue
Block a user