diff --git a/tests/test_auth_builder.py b/tests/test_auth_builder.py index 62ca1e5..74dab6b 100644 --- a/tests/test_auth_builder.py +++ b/tests/test_auth_builder.py @@ -51,7 +51,8 @@ def test_build_google_auth_omits_jwt_key_when_empty(monkeypatch): monkeypatch.setenv("GOOGLE_CLIENT_ID", "test-id.apps.googleusercontent.com") monkeypatch.setenv("GOOGLE_CLIENT_SECRET", "GOCSPX-test-secret") monkeypatch.setenv("UNRAID_MCP_BASE_URL", "http://10.1.0.2:6970") - monkeypatch.delenv("UNRAID_MCP_JWT_SIGNING_KEY", raising=False) + # Use setenv("") not delenv so dotenv reload can't re-inject from ~/.unraid-mcp/.env + monkeypatch.setenv("UNRAID_MCP_JWT_SIGNING_KEY", "") import unraid_mcp.config.settings as s diff --git a/tests/test_auth_settings.py b/tests/test_auth_settings.py index 3c696a6..2c40af6 100644 --- a/tests/test_auth_settings.py +++ b/tests/test_auth_settings.py @@ -16,11 +16,17 @@ def _reload_settings(monkeypatch, overrides: dict) -> Any: def test_google_auth_defaults_to_empty(monkeypatch): """Google auth vars default to empty string when not set.""" - monkeypatch.delenv("GOOGLE_CLIENT_ID", raising=False) - monkeypatch.delenv("GOOGLE_CLIENT_SECRET", raising=False) - monkeypatch.delenv("UNRAID_MCP_BASE_URL", raising=False) - monkeypatch.delenv("UNRAID_MCP_JWT_SIGNING_KEY", raising=False) - mod = _reload_settings(monkeypatch, {}) + # Use setenv("", "") rather than delenv so dotenv reload can't re-inject values + # from ~/.unraid-mcp/.env (load_dotenv won't override existing env vars). + mod = _reload_settings( + monkeypatch, + { + "GOOGLE_CLIENT_ID": "", + "GOOGLE_CLIENT_SECRET": "", + "UNRAID_MCP_BASE_URL": "", + "UNRAID_MCP_JWT_SIGNING_KEY": "", + }, + ) assert mod.GOOGLE_CLIENT_ID == "" assert mod.GOOGLE_CLIENT_SECRET == "" assert mod.UNRAID_MCP_BASE_URL == ""