Commit 516b1dff authored by Jan Reimes's avatar Jan Reimes
Browse files

test(02-02): Fix credentials tests for environment isolation 

- Add monkeypatch to clear TDC_EOL_USERNAME/PASSWORD in tests
- Fix test_resolve_returns_none_when_no_creds to clear env vars
- Fix test_resolve_with_cli_only to clear env vars

Tests now properly isolate environment for credential resolution testing.
parent bd5dc5e2

tests/test_credentials.py

+10 −2
Original line number Diff line number Diff line
@@ -58,8 +58,12 @@ class TestCredentialsWithConfig: 
        assert result.username == "env_user"

        assert result.password == "env_pass"



    def test_resolve_returns_none_when_no_creds(self) -> None:

    def test_resolve_returns_none_when_no_creds(self, monkeypatch) -> None:

        """Returns None when no credentials found anywhere."""

        # Clear any existing env vars that might interfere

        monkeypatch.delenv("TDC_EOL_USERNAME", raising=False)

        monkeypatch.delenv("TDC_EOL_PASSWORD", raising=False)



        result = resolve_credentials()



        assert result is None
@@ -134,8 +138,12 @@ class TestCredentialsBackwardCompatibility: 
        assert result.username == "legacy_user"

        assert result.password == "legacy_pass"



    def test_resolve_with_cli_only(self) -> None:

    def test_resolve_with_cli_only(self, monkeypatch) -> None:

        """CLI-only credentials still work."""

        # Clear any existing env vars

        monkeypatch.delenv("TDC_EOL_USERNAME", raising=False)

        monkeypatch.delenv("TDC_EOL_PASSWORD", raising=False)



        result = resolve_credentials(username="direct_user", password="direct_pass")



        assert result is not None