Clarify method intent. Use BrokenPromise rather than assert.

This commit is contained in:
Ben Patterson
2016-02-02 06:48:30 -05:00
parent 07f8b8e9c3
commit 902126d55c
2 changed files with 5 additions and 5 deletions

View File

@@ -78,9 +78,11 @@ class CoursewarePage(CoursePage):
else:
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')):
ActionChains(self.browser).move_to_element(tab).perform()
@@ -89,8 +91,6 @@ class CoursewarePage(CoursePage):
'Tab {index} should appear'.format(index=index)
)
return True
@property
def course_license(self):
"""

View File

@@ -864,7 +864,7 @@ class TooltipTest(UniqueCourseTest):
self.course_info_page.visit()
self.tab_nav.go_to_tab('Courseware')
self.assertTrue(self.courseware_page.tooltips_displayed())
self.courseware_page.verify_tooltips_displayed()
@attr('shard_1')