From 10f5713d8df3469e17f0adb7a6fbcdb7014dfe66 Mon Sep 17 00:00:00 2001 From: Daniel Date: Thu, 7 Dec 2023 19:34:52 +0100 Subject: [PATCH] set expect timeout in its own fixture with autouse to guarantee it will be set before use --- pytest_abra/pytest_abra.py | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/pytest_abra/pytest_abra.py b/pytest_abra/pytest_abra.py index 21350af..33a74ae 100644 --- a/pytest_abra/pytest_abra.py +++ b/pytest_abra/pytest_abra.py @@ -23,6 +23,12 @@ def pytest_addoption(parser: Parser): parser.addoption("--timeout", action="store", type=int, default=20_000) +@pytest.fixture(autouse=True) +def set_expect_timeout(request): + TIMEOUT = request.config.getoption("--timeout") + expect.set_options(timeout=TIMEOUT) + + @pytest.fixture def context(context: BrowserContext, request) -> BrowserContext: # note: because this has the existing context fixture as an argument, it is ensured @@ -33,7 +39,6 @@ def context(context: BrowserContext, request) -> BrowserContext: context.set_default_timeout(TIMEOUT) context.set_extra_http_headers(LOCALE) - expect.set_options(timeout=TIMEOUT) return context