Skip to content

Commit faadb52

Browse files
OG-Drizzlesclaude
andcommitted
fix: set dummy auth env vars in comment guard test
test_comment_add_human_mode_warns_without_yes relied on TRELLO_API_KEY and TRELLO_TOKEN being present in the environment. CI runners don't have them, so TrelloAuth.from_env raised ValueError before the mock client was reached — exit code 1 instead of 0. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent b8d8a44 commit faadb52

1 file changed

Lines changed: 2 additions & 0 deletions

File tree

tests/test_cli.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -515,6 +515,8 @@ def test_comment_add_human_mode_warns_without_yes(
515515

516516
_setup_cli_cache(tmp_path, monkeypatch)
517517
# Human mode is already set by _setup_cli_cache
518+
monkeypatch.setenv("TRELLO_API_KEY", "dummy")
519+
monkeypatch.setenv("TRELLO_TOKEN", "dummy")
518520

519521
mock_comment = MagicMock()
520522
mock_comment.id = "comment_abc"

0 commit comments

Comments
 (0)