From 0cc9426c6d3b73140499406bc62a7f36438d1839 Mon Sep 17 00:00:00 2001 From: Teknium <127238744+teknium1@users.noreply.github.com> Date: Thu, 16 Jul 2026 06:35:20 -0700 Subject: [PATCH] test: feishu port-binding report expects webhook mode after #52563 integration MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit With the mode-conditional check centralized, default (websocket) Feishu no longer counts as port-binding in the secondary batch report — pin the fixture to connection_mode=webhook so the test still exercises the multi-platform report path. --- tests/gateway/test_multiplex_adapter_registry.py | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/tests/gateway/test_multiplex_adapter_registry.py b/tests/gateway/test_multiplex_adapter_registry.py index 246d8602d2a06..ca744f80e17c2 100644 --- a/tests/gateway/test_multiplex_adapter_registry.py +++ b/tests/gateway/test_multiplex_adapter_registry.py @@ -182,7 +182,12 @@ class TestSecondaryProfileConfigHandling: reviewer_cfg = GatewayConfig(multiplex_profiles=True) reviewer_cfg.platforms = { - Platform.FEISHU: PlatformConfig(enabled=True), + # connection_mode=webhook: with #52563's conditional check merged, + # default (websocket) Feishu no longer binds a port — only webhook + # mode should be reported here. + Platform.FEISHU: PlatformConfig( + enabled=True, extra={"connection_mode": "webhook"} + ), Platform.WEBHOOK: PlatformConfig(enabled=True, extra={"port": 8644}), Platform.TELEGRAM: PlatformConfig(enabled=True, token="t"), }