test(agent-orchestrator): use vitest mock for available agents provider - #13297
Conversation
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Summary
Fixes the
available-agents.test.tsharness after #13289 introducedimport { mock } from "bun:test", which the package's Vitest command cannot resolve.The test still installs the framework-state mock before dynamically importing
availableAgentsProvider, but usesvi.mock(...)so it runs under the existing Vitest harness.Verification
bun run --cwd plugins/plugin-agent-orchestrator test -- __tests__/unit/available-agents.test.tsResult: 1 test file passed, 5 tests passed.