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

fix: Move nested describe block outside test function in CodeIndexPopover tests

Browse source
This commit is contained in:
Roo Code 2025-12-29 01:44:11 +00:00
parent e06c5fadcf
commit 51bd2361a8
2 changed files with 54 additions and 54 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/services/code-index/__tests__/orchestrator.spec.ts
View file

@ -217,7 +217,7 @@ describe("CodeIndexOrchestrator - error path cleanup gating", () => {
getProviderSettings: vi.fn().mockReturnValue({}),
setProviderSettings: vi.fn(),
}
const mgr = new configManagerModule.CodeIndexConfigManager(mockContextProxy)
const mgr = new configManagerModule.CodeIndexConfigManager(mockContextProxy as any)
await mgr.loadConfiguration()
const requiresRestart = mgr.doesConfigChangeRequireRestart(prev)
expect(requiresRestart).toBe(true)

106
webview-ui/src/components/chat/__tests__/CodeIndexPopover.auto-populate.spec.tsx
View file

@ -309,59 +309,6 @@ describe("CodeIndexPopover - Auto-population Feature Logic", () => {
awsProfile: "default",
}
/**
* UI/component tests for codebaseIndexOpenRouterEmbedderBaseUrl field
*/
describe("CodeIndexPopover - OpenRouter Embedder Base URL Field", () => {
/**
* Test 1: Renders VSCodeTextField for codebaseIndexOpenRouterEmbedderBaseUrl
*/
test("renders VSCodeTextField for codebaseIndexOpenRouterEmbedderBaseUrl", () => {
const settings = {
codebaseIndexEmbedderProvider: "openrouter",
codebaseIndexOpenRouterEmbedderBaseUrl: "https://openrouter.ai/api/v1",
}
const rendered = `<VSCodeTextField data-testid="openrouter-base-url" value="${settings.codebaseIndexOpenRouterEmbedderBaseUrl}" />`
expect(rendered).toContain('data-testid="openrouter-base-url"')
expect(rendered).toContain(settings.codebaseIndexOpenRouterEmbedderBaseUrl)
})
/**
* Test 2: Calls updateSetting on user input
*/
test("calls updateSetting when user changes base URL", () => {
const mockUpdateSetting = vi.fn()
const event = { target: { value: "https://custom.openrouter.ai/api/v1" } }
mockUpdateSetting("codebaseIndexOpenRouterEmbedderBaseUrl", event.target.value)
expect(mockUpdateSetting).toHaveBeenCalledWith(
"codebaseIndexOpenRouterEmbedderBaseUrl",
"https://custom.openrouter.ai/api/v1",
)
})
/**
* Test 3: Validation error shown for invalid URL
*/
test("shows validation error for invalid OpenRouter base URL", () => {
const invalidUrl = "not-a-url"
const formErrors = { codebaseIndexOpenRouterEmbedderBaseUrl: "Invalid URL" }
const rendered = `<VSCodeTextField data-testid="openrouter-base-url" value="${invalidUrl}" className="border-red-500" />`
expect(rendered).toContain("border-red-500")
expect(formErrors.codebaseIndexOpenRouterEmbedderBaseUrl).toBe("Invalid URL")
})
/**
* Test 4: No error for valid URL
*/
test("does not show error for valid OpenRouter base URL", () => {
const validUrl = "https://openrouter.ai/api/v1"
const formErrors: Record<string, string> = {}
const rendered = `<VSCodeTextField data-testid="openrouter-base-url" value="${validUrl}" />`
expect(rendered).not.toContain("border-red-500")
expect(formErrors.codebaseIndexOpenRouterEmbedderBaseUrl).toBeUndefined()
})
})
// Simulate the onValueChange logic
const value = "bedrock"
@ -383,3 +330,56 @@ describe("CodeIndexPopover - Auto-population Feature Logic", () => {
expect(mockUpdateSetting).toHaveBeenCalledTimes(1)
})
})
/**
* UI/component tests for codebaseIndexOpenRouterEmbedderBaseUrl field
*/
describe("CodeIndexPopover - OpenRouter Embedder Base URL Field", () => {
/**
* Test 1: Renders VSCodeTextField for codebaseIndexOpenRouterEmbedderBaseUrl
*/
test("renders VSCodeTextField for codebaseIndexOpenRouterEmbedderBaseUrl", () => {
const settings = {
codebaseIndexEmbedderProvider: "openrouter",
codebaseIndexOpenRouterEmbedderBaseUrl: "https://openrouter.ai/api/v1",
}
const rendered = `<VSCodeTextField data-testid="openrouter-base-url" value="${settings.codebaseIndexOpenRouterEmbedderBaseUrl}" />`
expect(rendered).toContain('data-testid="openrouter-base-url"')
expect(rendered).toContain(settings.codebaseIndexOpenRouterEmbedderBaseUrl)
})
/**
* Test 2: Calls updateSetting on user input
*/
test("calls updateSetting when user changes base URL", () => {
const mockUpdateSetting = vi.fn()
const event = { target: { value: "https://custom.openrouter.ai/api/v1" } }
mockUpdateSetting("codebaseIndexOpenRouterEmbedderBaseUrl", event.target.value)
expect(mockUpdateSetting).toHaveBeenCalledWith(
"codebaseIndexOpenRouterEmbedderBaseUrl",
"https://custom.openrouter.ai/api/v1",
)
})
/**
* Test 3: Validation error shown for invalid URL
*/
test("shows validation error for invalid OpenRouter base URL", () => {
const invalidUrl = "not-a-url"
const formErrors = { codebaseIndexOpenRouterEmbedderBaseUrl: "Invalid URL" }
const rendered = `<VSCodeTextField data-testid="openrouter-base-url" value="${invalidUrl}" className="border-red-500" />`
expect(rendered).toContain("border-red-500")
expect(formErrors.codebaseIndexOpenRouterEmbedderBaseUrl).toBe("Invalid URL")
})
/**
* Test 4: No error for valid URL
*/
test("does not show error for valid OpenRouter base URL", () => {
const validUrl = "https://openrouter.ai/api/v1"
const formErrors: Record<string, string> = {}
const rendered = `<VSCodeTextField data-testid="openrouter-base-url" value="${validUrl}" />`
expect(rendered).not.toContain("border-red-500")
expect(formErrors.codebaseIndexOpenRouterEmbedderBaseUrl).toBeUndefined()
})
})