From a51c7397b46dc41492b2bab424e6ce8a843dfef9 Mon Sep 17 00:00:00 2001
From: teknium1 <127238744+teknium1@users.noreply.github.com>
Date: Sun, 7 Jun 2026 06:43:50 -0700
Subject: [PATCH] test(discord): fix 2 more clarify-button auth expectations
 after fail-closed change (#f6f363662)

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.
---
 tests/gateway/test_discord_clarify_buttons.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tests/gateway/test_discord_clarify_buttons.py b/tests/gateway/test_discord_clarify_buttons.py
index d2157f2eb9b3..95c5bb5cc630 100644
--- a/tests/gateway/test_discord_clarify_buttons.py
+++ b/tests/gateway/test_discord_clarify_buttons.py
@@ -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()