Skip to content

Fix apply-diff test and add ntc apply-diff test #10185

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
dcbartlett wants to merge 14 commits into
base: main
Choose a base branch
from
Open

Fix apply-diff test and add ntc apply-diff test #10185

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
218ebbd
chore: fix apply-diff test and add ntc apply-diff test
dcbartlett Dec 18, 2025
238015a
fix: correct misleading comments about provider/model in native tool …
roomote Dec 18, 2025
134b231
Fix: Use testFile.name instead of testFile.content for Target File in…
roomote Dec 18, 2025
fe56578
fix: add hasNativeApiProtocol assertion in native tool calling tests
roomote Dec 18, 2025
8686643
fix: Add apiProvider and oepnRouterModelId to tests for consistency
dcbartlett Dec 19, 2025
0fbca04
fix: hard-code provider/model for all apply-diff tests
dcbartlett Dec 19, 2025
7eb0c34
fix: Update to models that can pass these tests.
dcbartlett Dec 20, 2025
3927d66
chore: Remove non-native e2e test for apply-diff tool
dcbartlett Dec 20, 2025
58be120
chore: Add more test suites. Add retry config
dcbartlett Dec 30, 2025
b901846
fix: add missing hasNativeApiProtocol assertion to native test files
roomote Dec 30, 2025
c0160dc
chore: Convert remaining tool tests to NTC tests
dcbartlett Dec 31, 2025
7055a62
Update apps/vscode-e2e/src/suite/tools/search-files-native.test.ts
dcbartlett Dec 31, 2025
4345009
Update apps/vscode-e2e/src/suite/tools/use-mcp-tool-native.test.ts
dcbartlett Dec 31, 2025
a0127d8
Update apps/vscode-e2e/src/suite/tools/write-to-file-native.test.ts
dcbartlett Dec 31, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
50 changes: 50 additions & 0 deletions .vscode/launch.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,56 @@
"group": "tasks",
"order": 1
}
},
{
"name": "Debug E2E Tests",
"type": "extensionHost",
"request": "launch",
"runtimeExecutable": "${execPath}",
"args": [
"${workspaceFolder}/apps/vscode-e2e/test-workspace",
"--extensionDevelopmentPath=${workspaceFolder}/src",
"--extensionTestsPath=${workspaceFolder}/apps/vscode-e2e/out/suite/index"
],
"sourceMaps": true,
"outFiles": ["${workspaceFolder}/src/dist/**/*.js", "${workspaceFolder}/apps/vscode-e2e/out/**/*.js"],
"preLaunchTask": "build-e2e-tests",
"envFile": "${workspaceFolder}/apps/vscode-e2e/.env.local",
"env": {
"NODE_ENV": "development",
"VSCODE_DEBUG_MODE": "true"
},
"resolveSourceMapLocations": ["${workspaceFolder}/**", "!**/node_modules/**"],
"presentation": {
"hidden": false,
"group": "tasks",
"order": 2
}
},
{
"name": "Debug E2E Tests (Quick - extension pre-built)",
"type": "extensionHost",
"request": "launch",
"runtimeExecutable": "${execPath}",
"args": [
"${workspaceFolder}/apps/vscode-e2e/test-workspace",
"--extensionDevelopmentPath=${workspaceFolder}/src",
"--extensionTestsPath=${workspaceFolder}/apps/vscode-e2e/out/suite/index"
],
"sourceMaps": true,
"outFiles": ["${workspaceFolder}/src/dist/**/*.js", "${workspaceFolder}/apps/vscode-e2e/out/**/*.js"],
"preLaunchTask": "compile-e2e-only",
"envFile": "${workspaceFolder}/apps/vscode-e2e/.env.local",
"env": {
"NODE_ENV": "development",
"VSCODE_DEBUG_MODE": "true"
},
"resolveSourceMapLocations": ["${workspaceFolder}/**", "!**/node_modules/**"],
"presentation": {
"hidden": false,
"group": "tasks",
"order": 4
}
}
]
}
57 changes: 57 additions & 0 deletions .vscode/tasks.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -69,6 +69,63 @@
"group": "watch",
"reveal": "always"
}
},
{
"label": "build-e2e-tests",
"dependsOn": ["build-e2e:bundle", "build-e2e:webview", "build-e2e:compile"],
"dependsOrder": "sequence",
"group": "build",
"problemMatcher": []
},
{
"label": "build-e2e:bundle",
"type": "shell",
"command": "pnpm -w bundle",
"group": "build",
"problemMatcher": [],
"presentation": {
"reveal": "always",
"panel": "shared"
}
},
{
"label": "build-e2e:webview",
"type": "shell",
"command": "pnpm --filter @roo-code/vscode-webview build",
"group": "build",
"problemMatcher": [],
"presentation": {
"reveal": "always",
"panel": "shared"
}
},
{
"label": "build-e2e:compile",
"type": "shell",
"command": "npx rimraf out; npx tsc -p tsconfig.json",
"options": {
"cwd": "${workspaceFolder}/apps/vscode-e2e"
},
"group": "build",
"problemMatcher": "$tsc",
"presentation": {
"reveal": "always",
"panel": "shared"
}
},
{
"label": "compile-e2e-only",
"type": "shell",
"command": "npx rimraf out; npx tsc -p tsconfig.json",
"options": {
"cwd": "${workspaceFolder}/apps/vscode-e2e"
},
"group": "build",
"problemMatcher": "$tsc",
"presentation": {
"reveal": "always",
"panel": "shared"
}
}
]
}
1 change: 1 addition & 0 deletions apps/vscode-e2e/src/suite/index.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,7 @@ export async function run() {
const mochaOptions: Mocha.MochaOptions = {
ui: "tdd",
timeout: 20 * 60 * 1_000, // 20m
retries: 3,
}

if (process.env.TEST_GREP) {
Expand Down
Loading
Loading