diff --git a/awxkit/test/api/pages/test_base.py b/awxkit/test/api/pages/test_base.py index 3126e333d8..137d78ab22 100644 --- a/awxkit/test/api/pages/test_base.py +++ b/awxkit/test/api/pages/test_base.py @@ -6,8 +6,8 @@ from awxkit.config import config @pytest.fixture(autouse=True) def setup_config(monkeypatch: pytest.MonkeyPatch) -> None: - monkeypatch.setattr(config, "credentials", {"default": {"username": "foo", "password": "bar"}}, raising=False) - monkeypatch.setattr(config, "base_url", "", raising=False) + monkeypatch.setitem(config, "credentials", {"default": {"username": "foo", "password": "bar"}}) + monkeypatch.setitem(config, "base_url", "") @pytest.fixture diff --git a/awxkit/test/cli/test_authentication.py b/awxkit/test/cli/test_authentication.py index f196f66043..c8428679d8 100644 --- a/awxkit/test/cli/test_authentication.py +++ b/awxkit/test/cli/test_authentication.py @@ -9,8 +9,8 @@ from awxkit import config @pytest.fixture(autouse=True) def reset_config_state(monkeypatch: pytest.MonkeyPatch) -> None: """Ensure clean config state for each test to prevent parallel test interference""" - monkeypatch.setattr(config, 'force_basic_auth', False, raising=False) - monkeypatch.setattr(config, 'use_sessions', False, raising=False) + monkeypatch.setitem(config, 'force_basic_auth', False) + monkeypatch.setitem(config, 'use_sessions', False) # ============================================================================ @@ -47,7 +47,7 @@ def setup_session_auth(cli_args: Optional[List[str]] = None) -> Tuple[CLI, Mock, def test_basic_auth_enabled(monkeypatch): """Test that AWXKIT_FORCE_BASIC_AUTH=true enables Basic authentication""" cli, mock_root, mock_connection = setup_basic_auth() - monkeypatch.setattr(config, 'force_basic_auth', True) + monkeypatch.setitem(config, 'force_basic_auth', True) cli.authenticate() mock_connection.login.assert_called_once_with('testuser', 'testpass') @@ -58,7 +58,7 @@ def test_basic_auth_enabled(monkeypatch): def test_session_auth_default(monkeypatch): """Test that session auth is used by default (backward compatibility)""" cli, mock_root, mock_load_session = setup_session_auth() - monkeypatch.setattr(config, 'force_basic_auth', False) + monkeypatch.setitem(config, 'force_basic_auth', False) cli.authenticate() mock_root.load_session.assert_called_once() @@ -71,7 +71,7 @@ def test_aap_gateway_scenario(monkeypatch): cli, mock_root, mock_connection = setup_basic_auth( ['awx', '--conf.host', 'https://aap-sbx.cambiahealth.com', '--conf.username', 'puretest', '--conf.password', 'testpass'] ) - monkeypatch.setattr(config, 'force_basic_auth', True) + monkeypatch.setitem(config, 'force_basic_auth', True) cli.authenticate() mock_connection.login.assert_called_once_with('puretest', 'testpass') @@ -82,7 +82,7 @@ def test_aap_gateway_scenario(monkeypatch): def test_empty_credentials_error(monkeypatch): """Test error handling for explicitly empty credentials""" cli, mock_root, mock_connection = setup_basic_auth(['awx', '--conf.username', '', '--conf.password', '']) - monkeypatch.setattr(config, 'force_basic_auth', True) + monkeypatch.setitem(config, 'force_basic_auth', True) with pytest.raises(ValueError, match="Basic authentication requires both username and password"): cli.authenticate() @@ -94,7 +94,7 @@ def test_connection_failure(monkeypatch): """Test error handling when Basic auth connection fails""" cli, mock_root, mock_connection = setup_basic_auth() mock_connection.login.side_effect = Exception("Connection failed") - monkeypatch.setattr(config, 'force_basic_auth', True) + monkeypatch.setitem(config, 'force_basic_auth', True) with pytest.raises(RuntimeError, match="Basic authentication failed: Connection failed"): cli.authenticate()