Skip to content

Commit fa17c97

Browse files
committed
Fix integration tests: include eca-info skill in commands list
The built-in eca-info skill (added in 0.132.0) auto-loads as a /command, so the query-commands integration test now sees a 20th entry the expected list didn't include. Add it to the expected vector to match the actual response. The :query "co" sub-test is unaffected because "eca-info" and its description don't contain "co". This test has been red on master since b7ae244 (May 6); only the "Tests" workflow runs integration tests, so the failure was easy to miss in the "Docs" / "Bump downstreams" workflow noise.
1 parent 507c5db commit fa17c97

1 file changed

Lines changed: 2 additions & 1 deletion

File tree

integration-test/integration/chat/commands_test.clj

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,8 @@
4040
{:name "subagents" :arguments []}
4141
{:name "plugins" :arguments []}
4242
{:name "plugin-install" :arguments [{:name "plugin"}]}
43-
{:name "plugin-uninstall" :arguments [{:name "plugin"}]}]}
43+
{:name "plugin-uninstall" :arguments [{:name "plugin"}]}
44+
{:name "eca-info" :arguments nil}]}
4445
resp))))
4546

4647
(testing "We query specific commands"

0 commit comments

Comments
 (0)