mirror of
https://github.com/galaxyproject/galaxy.git
synced 2026-09-24 16:30:27 +08:00
Retry failed clicks more aggressively for Selenium tests.
Would have prevented from this (https://jenkins.galaxyproject.org/job/selenium/238/testReport/junit/selenium_tests.test_history_panel/HistoryPanelTestCase/test_refresh_preserves_state/) failure in retrospect. Also refactor that common pattern into a helper and use throughout. Add missing @selenium_test decorator on that test also - it would have retried if this had been there and we would have a screenshot.
This commit is contained in:
@@ -59,8 +59,8 @@ class HistoryPanelTestCase(SeleniumTestCase):
|
||||
tag_area_selector = self.test_data["historyPanel"]["selectors"]["history"]["tagArea"]
|
||||
anno_area_selector = self.test_data["historyPanel"]["selectors"]["history"]["annoArea"]
|
||||
|
||||
tag_icon = self.wait_for_selector(tag_icon_selector)
|
||||
annon_icon = self.wait_for_selector(anno_icon_selector)
|
||||
tag_icon = self.wait_for_selector_clickable(tag_icon_selector)
|
||||
annon_icon = self.wait_for_selector_clickable(anno_icon_selector)
|
||||
|
||||
self.assert_selector_absent_or_hidden(tag_area_selector)
|
||||
self.assert_selector_absent_or_hidden(anno_area_selector)
|
||||
@@ -83,6 +83,7 @@ class HistoryPanelTestCase(SeleniumTestCase):
|
||||
self.assert_selector_absent_or_hidden(tag_area_selector)
|
||||
self.assert_selector_absent_or_hidden(anno_area_selector)
|
||||
|
||||
@selenium_test
|
||||
def test_refresh_preserves_state(self):
|
||||
self.register()
|
||||
self.perform_upload(self.get_filename("1.txt"))
|
||||
@@ -107,8 +108,7 @@ class HistoryPanelTestCase(SeleniumTestCase):
|
||||
self.assert_selector_absent_or_hidden(hda_body_selector)
|
||||
|
||||
def click_history_refresh(self):
|
||||
refresh_button_element = self.wait_for_selector('a#history-refresh-button')
|
||||
refresh_button_element.click()
|
||||
self.wait_for_and_click_selector('a#history-refresh-button')
|
||||
|
||||
def click_to_rename_history(self):
|
||||
self.history_panel_name_element().click()
|
||||
|
||||
Reference in New Issue
Block a user