Skip to content

Commit d337499

Browse files
committed
test(extension): add setValues to the api-configuration spec provider mocks
API.setConfiguration now routes through ClineProvider.setValues (8576b18 "fix(api): route setConfiguration through ClineProvider.setValues"), but the pre-existing spec still shaped the provider mock around the old contextProxy.setValues call. Both tests in this file failed with "this.sidebarProvider.setValues is not a function" on the platform-unit-test runs (F3 #1560 windows unit, job 101863508169, run 34161293841). Add the provider-level setValues mock to both tests to match the new routing. Upstream: #1560 (vps2 F3)
1 parent 01982d5 commit d337499

1 file changed

Lines changed: 2 additions & 0 deletions

File tree

‎src/extension/__tests__/api-configuration.spec.ts‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@ describe("API - configuration", () => {
1717
const provider = {
1818
context: {},
1919
on: vi.fn(),
20+
setValues,
2021
contextProxy: { setValues },
2122
providerSettingsManager: { saveConfig, setModeConfig },
2223
postStateToWebview,
@@ -50,6 +51,7 @@ describe("API - configuration", () => {
5051
const provider = {
5152
context: {},
5253
on: vi.fn(),
54+
setValues,
5355
contextProxy: { setValues },
5456
providerSettingsManager: { saveConfig, setModeConfig },
5557
postStateToWebview,

0 commit comments

Comments
 (0)