scriptoriumadmin/Roo-Code
1
0
Fork 0
mirror of https://github.com/RooVetGit/Roo-Code.git synced 2026-09-07 08:26:51 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

refactor(tests): Update migrateSettings tests to use ContextProxy

Browse source
This commit is contained in:
Daniel Riccio 2025-07-02 13:20:21 -05:00
parent b4a8a45266
commit eb182d0ed0
No known key found for this signature in database
GPG key ID: FFD5FD825F8E8209
1 changed files with 19 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

26
src/__tests__/migrateSettings.spec.ts
View file

@ -4,6 +4,7 @@ import * as fs from "fs/promises"
import { fileExistsAtPath } from "../utils/fs"
import { GlobalFileNames } from "../shared/globalFileNames"
import { migrateSettings } from "../utils/migrateSettings"
import type { ContextProxy } from "../core/config/ContextProxy"
// Mock dependencies
vitest.mock("vscode")
@ -20,6 +21,7 @@ vitest.mock("../utils/fs")
describe("Settings Migration", () => {
let mockContext: vscode.ExtensionContext
let mockOutputChannel: vscode.OutputChannel
let mockContextProxy: ContextProxy
const mockStoragePath = "/mock/storage"
const mockSettingsDir = path.join(mockStoragePath, "settings")
@ -48,8 +50,18 @@ describe("Settings Migration", () => {
// Mock extension context
mockContext = {
globalStorageUri: { fsPath: mockStoragePath },
globalState: {
get: vitest.fn().mockReturnValue(undefined),
update: vitest.fn().mockResolvedValue(undefined),
},
} as unknown as vscode.ExtensionContext
// Mock ContextProxy
mockContextProxy = {
getWorkspaceSettings: vitest.fn().mockReturnValue({ taskHistory: [] }),
updateWorkspaceState: vitest.fn().mockResolvedValue(undefined),
} as unknown as ContextProxy
// Set global outputChannel for all tests
;(global as any).outputChannel = mockOutputChannel
})
@ -69,7 +81,7 @@ describe("Settings Migration", () => {
})
// Run the migration
await migrateSettings(mockContext, mockOutputChannel)
await migrateSettings(mockContext, mockOutputChannel, mockContextProxy)
// Verify expected rename call - cline_custom_modes.json should be renamed to custom_modes.json
expect(mockRename).toHaveBeenCalledWith(legacyClineCustomModesPath, legacyCustomModesJson)
@ -92,7 +104,7 @@ describe("Settings Migration", () => {
})
// Run the migration
await migrateSettings(mockContext, mockOutputChannel)
await migrateSettings(mockContext, mockOutputChannel, mockContextProxy)
// Verify expected rename call
expect(mockRename).toHaveBeenCalledWith(legacyMcpSettingsPath, newMcpSettingsPath)
@ -115,7 +127,7 @@ describe("Settings Migration", () => {
return false
})
await migrateSettings(mockContext, mockOutputChannel)
await migrateSettings(mockContext, mockOutputChannel, mockContextProxy)
// Verify rename was not called since destination files exist
expect(mockRename).not.toHaveBeenCalled()
@ -128,7 +140,7 @@ describe("Settings Migration", () => {
// Mock file existence to throw error
vitest.mocked(fileExistsAtPath).mockRejectedValue(new Error("Test error"))
await migrateSettings(mockContext, mockOutputChannel)
await migrateSettings(mockContext, mockOutputChannel, mockContextProxy)
// Verify error was logged
expect(mockOutputChannel.appendLine).toHaveBeenCalledWith(
@ -163,7 +175,7 @@ describe("Settings Migration", () => {
return false
})
await migrateSettings(mockContext, mockOutputChannel)
await migrateSettings(mockContext, mockOutputChannel, mockContextProxy)
// Verify file operations
expect(mockWrite).toHaveBeenCalledWith(newCustomModesYaml, expect.any(String), "utf-8")
@ -201,7 +213,7 @@ describe("Settings Migration", () => {
return false
})
await migrateSettings(mockContext, mockOutputChannel)
await migrateSettings(mockContext, mockOutputChannel, mockContextProxy)
// Verify error was logged
expect(mockOutputChannel.appendLine).toHaveBeenCalledWith(
@ -239,7 +251,7 @@ describe("Settings Migration", () => {
return false
})
await migrateSettings(mockContext, mockOutputChannel)
await migrateSettings(mockContext, mockOutputChannel, mockContextProxy)
// Verify skip message was logged
expect(mockOutputChannel.appendLine).toHaveBeenCalledWith(