Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[1.0.0] Backport #4819 Fix journalist test_delete_one #4822

Merged
merged 1 commit into from Sep 16, 2019
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
Expand Up @@ -17,6 +17,7 @@
from selenium.webdriver.common.by import By
from selenium.webdriver.common.keys import Keys
from selenium.webdriver.support import expected_conditions
from selenium.webdriver.support.ui import WebDriverWait


# Number of times to try flaky clicks.
Expand Down Expand Up @@ -879,7 +880,12 @@ def _journalist_delete_all_confirmation(self):

def _journalist_delete_one(self):
self.safe_click_by_css_selector("[name=doc_names_selected]")
self.safe_click_by_id("delete-selected-link")

el = WebDriverWait(self.driver, self.timeout, self.poll_frequency).until(
expected_conditions.element_to_be_clickable((By.ID, "delete-selected-link"))
)
el.location_once_scrolled_into_view
ActionChains(self.driver).move_to_element(el).click().perform()

def _journalist_flags_source(self):
self.safe_click_by_id("flag-button")
Expand Down