Skip to content

Commit 1aea076

Browse files
committed
[py] make tests more robust
1 parent d3e4f50 commit 1aea076

File tree

1 file changed

+10
-10
lines changed

1 file changed

+10
-10
lines changed

py/test/selenium/webdriver/remote/remote_downloads_tests.py

+10-10
Original file line numberDiff line numberDiff line change
@@ -22,10 +22,7 @@
2222

2323

2424
def test_get_downloadable_files(driver, pages):
25-
pages.load("downloads/download.html")
26-
driver.find_element(By.ID, "file-1").click()
27-
driver.find_element(By.ID, "file-2").click()
28-
WebDriverWait(driver, 3).until(lambda d: len(d.get_downloadable_files()) == 2)
25+
_browser_downloads(driver, pages)
2926

3027
file_names = driver.get_downloadable_files()
3128

@@ -34,9 +31,7 @@ def test_get_downloadable_files(driver, pages):
3431

3532

3633
def test_download_file(driver, pages):
37-
pages.load("downloads/download.html")
38-
driver.find_element(By.ID, "file-1").click()
39-
WebDriverWait(driver, 3).until(lambda d: d.get_downloadable_files())
34+
_browser_downloads(driver, pages)
4035

4136
file_name = driver.get_downloadable_files()[0]
4237
with tempfile.TemporaryDirectory() as target_directory:
@@ -48,9 +43,14 @@ def test_download_file(driver, pages):
4843

4944

5045
def test_delete_downloadable_files(driver, pages):
51-
pages.load("downloads/download.html")
52-
driver.find_element(By.ID, "file-1").click()
53-
WebDriverWait(driver, 3).until(lambda d: d.get_downloadable_files())
46+
_browser_downloads(driver, pages)
5447

5548
driver.delete_downloadable_files()
5649
assert not driver.get_downloadable_files()
50+
51+
52+
def _browser_downloads(driver, pages):
53+
pages.load("downloads/download.html")
54+
driver.find_element(By.ID, "file-1").click()
55+
driver.find_element(By.ID, "file-2").click()
56+
WebDriverWait(driver, 3).until(lambda d: "file_2.jpg" in d.get_downloadable_files())

0 commit comments

Comments
 (0)