Skip to content

Commit b8622fe

Browse files
review fix
1 parent e2cf378 commit b8622fe

File tree

2 files changed

+17
-3
lines changed

2 files changed

+17
-3
lines changed

tests/profile/conftest.py

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,7 @@
11
import pytest
2+
from selenium.webdriver import Firefox
3+
4+
from modules.page_object_about_pages import AboutProfiles
25

36

47
@pytest.fixture()
@@ -14,6 +17,12 @@ def prefs_list(add_to_prefs_list: dict):
1417
return prefs
1518

1619

20+
@pytest.fixture()
21+
def about_profiles(driver: Firefox):
22+
ap = AboutProfiles(driver)
23+
yield ap
24+
25+
1726
@pytest.fixture()
1827
def add_to_prefs_list():
1928
return []

tests/profile/test_set_default_profile.py

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -77,11 +77,16 @@ def create_profiles(profile_paths, sys_platform):
7777
os.remove(profile_file)
7878

7979

80-
def test_set_default_profile(driver: Firefox, opt_ci):
80+
def test_set_default_profile(
81+
driver: Firefox, opt_ci: bool, about_profiles: AboutProfiles
82+
):
8183
"""
82-
C130792: Set the default profile through the firefox browser
84+
C130792: Set the default profile through the firefox
85+
86+
Arguments:
87+
about_profiles: AboutProfiles page object model.
88+
opt_ci: Boolean to verify if test is run in ci.
8389
"""
84-
about_profiles = AboutProfiles(driver)
8590

8691
# Get the profiles container, extract all relevant children under it.
8792
about_profiles.open()

0 commit comments

Comments
 (0)