diff --git a/lib/galaxy/selenium/navigates_galaxy.py b/lib/galaxy/selenium/navigates_galaxy.py index 03def6f1b19..4e7535fb7bb 100644 --- a/lib/galaxy/selenium/navigates_galaxy.py +++ b/lib/galaxy/selenium/navigates_galaxy.py @@ -596,7 +596,9 @@ class NavigatesGalaxy(HasDriverProxy[WaitType]): def history_has_hid(driver=None): hids = get_hids() - return any(h == hid for h in hids) + # Return True or None (not False) so wait_on keeps polling + # wait_on treats any non-None value as success, including False + return True if any(h == hid for h in hids) else None timeout = self.wait_length(wait_type=WAIT_TYPES.JOB_COMPLETION) try: diff --git a/lib/galaxy_test/selenium/test_history_storage.py b/lib/galaxy_test/selenium/test_history_storage.py index 90e0cb090e9..cb542294404 100644 --- a/lib/galaxy_test/selenium/test_history_storage.py +++ b/lib/galaxy_test/selenium/test_history_storage.py @@ -1,6 +1,5 @@ from .framework import ( managed_history, - selenium_only, selenium_test, SeleniumTestCase, ) @@ -30,7 +29,6 @@ UPLOAD_DATA_3 = { class TestHistoryStorage(SeleniumTestCase): ensure_registered = True - @selenium_only("Not yet migrated to support Playwright backend") @selenium_test @managed_history def test_history_storage_accessibility(self): @@ -57,7 +55,6 @@ class TestHistoryStorage(SeleniumTestCase): self.screenshot("storage_dashboard_manage_explore_usage_landing") self.assert_baseline_accessibility() - @selenium_only("Not yet migrated to support Playwright backend") @selenium_test @managed_history def test_delete_dataset_from_storage_view(self):