test: align commands.test.ts with acceptsArgs field in getPluginCommandSpecs#33362
test: align commands.test.ts with acceptsArgs field in getPluginCommandSpecs#33362lewiswigmore wants to merge 1 commit intoopenclaw:mainfrom
Conversation
getPluginCommandSpecs() now returns acceptsArgs for each command spec, but the test was not updated to expect the new field, causing CI failures.
Greptile SummaryThis PR fixes a failing CI test by adding the missing The one-line test fix is correct: the test command is registered without an explicit Confidence Score: 5/5
Last reviewed commit: 9f4e469 |
|
Please don't make PRs for test failures on main. The team is aware of those and will handle them directly on the codebase, not only fixing the tests but also investigating what the root cause is. Having to sift through test-fix-PRs (including some that have been out of date for weeks...) on top of that doesn't help. There are already way too many PRs for humans to manage; please don't make the flood worse. Thank you. |
Summary
getPluginCommandSpecs()now returns anacceptsArgsboolean for each command spec, but the unit test assertion was not updated to include the new field, causing CI failures across all open PRs.Changes
acceptsArgs: falseto the expected output in thegetPluginCommandSpecs()test assertionTest
This fixes the failing
src/plugins/commands.test.tstest onmain.