mirrors/hermes-agent
1
0
Fork 0
mirror of https://github.com/NousResearch/hermes-agent.git synced 2026-10-20 05:15:31 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

test(discord): fix 2 more clarify-button auth expectations after fail-closed change (#f6f363662)

Browse source 
Same pre-existing main breakage as the model-picker fix: f6f363662 made
component-button auth fail closed but only updated test_discord_component_auth.py.
test_discord_clarify_buttons' test_choice_falls_back_to_label_text_when_entry_missing
and test_other_flips_entry_to_awaiting_text drove the authorized-proceed path
with an empty allowlist, which now correctly rejects. Give the interacting
user (id 42) an allowlist entry so the proceed path runs (the dedicated
*_unauthorized_* tests still cover rejection). Verified green under the
hermetic per-file runner.
This commit is contained in:
teknium1 2026-06-07 06:43:50 -07:00
parent c977436cbd
commit a51c7397b4
No known key found for this signature in database
1 changed files with 2 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

4
tests/gateway/test_discord_clarify_buttons.py
View file

@ -174,7 +174,7 @@ class TestClarifyChoiceResolve:
view = ClarifyChoiceView(
choices=["alpha"],
clarify_id="cidGone",
allowed_user_ids=set(),
allowed_user_ids={"42"},
)
interaction = _make_interaction()
# Doesn't raise; resolve_gateway_clarify returns False quietly
@ -245,7 +245,7 @@ class TestClarifyOtherButton:
view = ClarifyChoiceView(
choices=["x", "y"],
clarify_id="cidD",
allowed_user_ids=set(),
allowed_user_ids={"42"},
)
interaction = _make_interaction()