-
Notifications
You must be signed in to change notification settings - Fork 12.7k
Closed
Labels
priority/p2Important but can be addressed in a future release.Important but can be addressed in a future release.
Description
What would you like to be added?
The test packages/core/src/code_assist/server.test.ts still contains a TODO in the loadCodeAssist test case (around lines 114‑130).
Replace the placeholder with a realistic LoadCodeAssistResponse object (e.g., currentTier, allowedTiers data) and assert against fields in this response.
Why is this needed?
Currently the test merely checks that the function returns the provided object.
Providing a proper mock response ensures we validate the expected fields of the API response and helps prevent regressions.
Additional context
See lines 114‑130 of packages/core/src/code_assist/server.test.ts where the TODO is located.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
priority/p2Important but can be addressed in a future release.Important but can be addressed in a future release.