Skip to content

Commit 529cb60

Browse files
committed
Re-enable and fix some tests
1 parent 994af7f commit 529cb60

File tree

5 files changed

+8
-5
lines changed

5 files changed

+8
-5
lines changed

tests/address_bar_and_search/test_default_search_provider_change_awesome_bar.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,11 @@
1+
from platform import system
2+
13
import pytest
24
from selenium.webdriver import Firefox
35

46
from modules.browser_object import Navigation
57
from modules.page_object import AboutPrefs
68

7-
WAIT_TIMEOUT = 10
89
SEARCH_ENGINE = "DuckDuckGo"
910
EXPECTED_PLACEHOLDER = f"Search with {SEARCH_ENGINE} or enter address"
1011

@@ -14,7 +15,7 @@ def test_case():
1415
return "3028795"
1516

1617

17-
@pytest.mark.unstable(reason="Bug 1983836")
18+
@pytest.mark.skipif(system().lower().startswith("win"), reason="Bug 1983836")
1819
@pytest.mark.ci
1920
def test_default_search_provider_change_awesome_bar(driver: Firefox):
2021
"""

tests/preferences/test_lang_pack_changed_from_about_prefs.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,6 @@ def add_to_prefs_list():
2828
return [("services.sync.prefs.sync-seen.intl.accept_languages", True)]
2929

3030

31-
@pytest.mark.unstable(reason="Bug 1978598")
3231
def test_lang_pack_changed_from_about_prefs(driver: Firefox):
3332
"""
3433
C1771617 - The language can be changed in about:preferences.

tests/security_and_privacy/test_downloads_from_private_not_leaked.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,6 @@ def delete_files_regex_string():
2828
return r"opm.*\.pdf"
2929

3030

31-
@pytest.mark.unstable(reason="Bug 1983849")
3231
@pytest.mark.slow
3332
@pytest.mark.audio
3433
def test_downloads_from_private_not_leaked(driver: Firefox, delete_files, screenshot):

tests/tabs/conftest.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,10 @@ def suite_id():
99
@pytest.fixture()
1010
def prefs_list(add_to_prefs_list: dict):
1111
"""List of prefs to send to main conftest.py driver fixture"""
12-
prefs = [("browser.tabs.delayHidingAudioPlayingIconMS", "200")]
12+
prefs = [
13+
("browser.tabs.delayHidingAudioPlayingIconMS", "200"),
14+
("browser.toolbars.bookmarks.visibility", "always"),
15+
]
1316
prefs.extend(add_to_prefs_list)
1417
return prefs
1518

tests/bookmarks_and_history/test_open_bookmark_in_new_tab.py renamed to tests/tabs/test_open_bookmark_in_new_tab.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,7 @@ def test_open_bookmark_in_new_tab(driver: Firefox):
3232

3333
# In a new tab, right-click the bookmarked page in the toolbar and select Open in New Tab from the context menu
3434
tabs.new_tab_by_button()
35+
tabs.wait_for_num_tabs(2)
3536
nav.open_bookmark_in_new_tab_via_context_menu(BOOKMARK_TITLE)
3637

3738
# Verify that the test page is opened in a new normal tab

0 commit comments

Comments
 (0)