code-server
471fd08fba
- Remove singular `token` field, keep only `tokens` dict - Simplify `resolve_token()` and `has_tokens` logic - Use finally block for correlation cleanup in server - Simplify `_resolve_auth()` to eliminate duplicate pattern - Remove redundant `has_tokens` check from CLI (server checks internally) - Update tests to remove backward-compat test cases Lines removed: ~30 Tests passing: 14/14 Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
25 lines
975 B
Python
25 lines
975 B
Python
"""Tests for HooksConfig with named tokens."""
|
|
|
|
from nanobot.config.schema import HooksConfig
|
|
|
|
|
|
def test_hooks_config_named_tokens():
|
|
"""Named tokens dict should work."""
|
|
config = HooksConfig(enabled=True, tokens={"gitea": "secret1", "ha": "secret2"})
|
|
assert config.tokens == {"gitea": "secret1", "ha": "secret2"}
|
|
|
|
|
|
def test_hooks_config_resolve_token():
|
|
"""resolve_token should return token name for a matching token."""
|
|
config = HooksConfig(enabled=True, tokens={"gitea": "secret1", "ha": "secret2"})
|
|
assert config.resolve_token("secret1") == "gitea"
|
|
assert config.resolve_token("secret2") == "ha"
|
|
assert config.resolve_token("unknown") is None
|
|
|
|
|
|
def test_hooks_config_has_tokens():
|
|
"""has_tokens should be True if tokens dict is non-empty."""
|
|
assert HooksConfig(enabled=True, tokens={"webhook": "secret"}).has_tokens
|
|
assert not HooksConfig(enabled=True).has_tokens
|
|
assert not HooksConfig(enabled=True, tokens={}).has_tokens
|