From 2dd3b1ef44c44decf5ec11c7abd4862112fbd2c4 Mon Sep 17 00:00:00 2001 From: Maximilian Wirtz Date: Fri, 20 Dec 2024 09:12:36 +0100 Subject: [PATCH] Do not use `store_automation_secret` in site fixture This parameter was not available in older versions so it caused problems in out update tests since we use the same fixture for older versions as well. Since we used only the default anyways, let's not set it... Change-Id: I3e76c2888b177d1903d38dc22956b8a92c7e2fd9 --- tests/integration/cmk/gui/test_login.py | 1 - tests/testlib/openapi_session.py | 2 -- tests/testlib/site.py | 1 - 3 files changed, 4 deletions(-) diff --git a/tests/integration/cmk/gui/test_login.py b/tests/integration/cmk/gui/test_login.py index c30dcfd0fdd..e86b77e598b 100644 --- a/tests/integration/cmk/gui/test_login.py +++ b/tests/integration/cmk/gui/test_login.py @@ -28,7 +28,6 @@ def create_and_delete_automation_user(site: Site) -> Iterator[tuple[str, str]]: contactgroups=[], roles=["user"], is_automation_user=True, - store_automation_secret=False, ) try: yield username, password diff --git a/tests/testlib/openapi_session.py b/tests/testlib/openapi_session.py index a4b1711fa27..5c7c3f0bdbd 100644 --- a/tests/testlib/openapi_session.py +++ b/tests/testlib/openapi_session.py @@ -325,13 +325,11 @@ def create( customer: None | str = None, roles: list[str] | None = None, is_automation_user: bool = False, - store_automation_secret: bool = False, ) -> None: if is_automation_user: auth_option = { "auth_type": "automation", "secret": password, - "store_automation_secret": store_automation_secret, } else: auth_option = { diff --git a/tests/testlib/site.py b/tests/testlib/site.py index 5d2640eedd3..b7a2bc931ef 100644 --- a/tests/testlib/site.py +++ b/tests/testlib/site.py @@ -1146,7 +1146,6 @@ def _create_automation_user(self) -> None: contactgroups=[], roles=["admin"], is_automation_user=True, - store_automation_secret=False, ) self.openapi.set_authentication_header(user=username, password=self._automation_secret.raw)