KrustyPlanet/hermes-agent
6
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 2

test: feishu port-binding report expects webhook mode after #52563 integration

Browse source 
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.
This commit is contained in:
Teknium 2026-07-16 06:35:20 -07:00
parent 9cb3569e97
commit 0cc9426c6d
1 changed files with 6 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

7
tests/gateway/test_multiplex_adapter_registry.py
View file

@ -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"),
}