diff --git a/tests/hermes_cli/test_commands.py b/tests/hermes_cli/test_commands.py index b6e11e4c517..562f54a5e0a 100644 --- a/tests/hermes_cli/test_commands.py +++ b/tests/hermes_cli/test_commands.py @@ -336,13 +336,19 @@ class TestSlackNativeSlashes: ) def test_includes_aliases_as_first_class_slashes(self): - """Aliases (/btw, /bg, /reset, /q) must be registered as standalone - slashes — this is the whole point of native-slashes parity.""" + """Aliases (/btw, /bg, /reset) must be registered as standalone + slashes — this is the whole point of native-slashes parity. + + Note: Slack's manifest hard-caps slash commands at 50 + (``_SLACK_MAX_SLASH_COMMANDS``). Canonical names win slots first, + then aliases, so the lowest-priority aliases can be clamped off + once the registry fills the cap (e.g. ``/q`` once ``/version`` + landed). The surviving aliases below still prove alias parity; + anything dropped remains reachable via ``/hermes ``.""" names = {n for n, _d, _h in slack_native_slashes()} assert "btw" in names assert "bg" in names assert "reset" in names - assert "q" in names def test_telegram_parity(self): """Every Telegram bot command must be registerable on Slack too.