Merge pull request #11387 from edx/benp/fix-test-tooltip
Correct flaky condition on tooltip.
This commit is contained in:
@@ -78,16 +78,18 @@ class CoursewarePage(CoursePage):
|
|||||||
else:
|
else:
|
||||||
return self.q(css=self.xblock_component_selector).attrs('innerHTML')[index].strip()
|
return self.q(css=self.xblock_component_selector).attrs('innerHTML')[index].strip()
|
||||||
|
|
||||||
def tooltips_displayed(self):
|
def verify_tooltips_displayed(self):
|
||||||
"""
|
"""
|
||||||
Verify if sequence navigation bar tooltips are being displayed upon mouse hover.
|
Verify that all sequence navigation bar tooltips are being displayed upon mouse hover.
|
||||||
|
|
||||||
|
If a tooltip does not appear, raise a BrokenPromise.
|
||||||
"""
|
"""
|
||||||
for index, tab in enumerate(self.q(css='#sequence-list > li')):
|
for index, tab in enumerate(self.q(css='#sequence-list > li')):
|
||||||
ActionChains(self.browser).move_to_element(tab).perform()
|
ActionChains(self.browser).move_to_element(tab).perform()
|
||||||
if not self.q(css='#tab_{index} > p'.format(index=index)).visible:
|
self.wait_for_element_visibility(
|
||||||
return False
|
'#tab_{index} > p'.format(index=index),
|
||||||
|
'Tab {index} should appear'.format(index=index)
|
||||||
return True
|
)
|
||||||
|
|
||||||
@property
|
@property
|
||||||
def course_license(self):
|
def course_license(self):
|
||||||
|
|||||||
@@ -864,7 +864,7 @@ class TooltipTest(UniqueCourseTest):
|
|||||||
self.course_info_page.visit()
|
self.course_info_page.visit()
|
||||||
self.tab_nav.go_to_tab('Courseware')
|
self.tab_nav.go_to_tab('Courseware')
|
||||||
|
|
||||||
self.assertTrue(self.courseware_page.tooltips_displayed())
|
self.courseware_page.verify_tooltips_displayed()
|
||||||
|
|
||||||
|
|
||||||
@attr('shard_1')
|
@attr('shard_1')
|
||||||
|
|||||||
Reference in New Issue
Block a user