Skip to content

Commit 26ff2c0

Browse files
committed
test(e2e): address search-files review feedback
1 parent 706c4b7 commit 26ff2c0

4 files changed

Lines changed: 3 additions & 64 deletions

File tree

apps/vscode-e2e/fixtures/search-files.json

Lines changed: 0 additions & 60 deletions
This file was deleted.

apps/vscode-e2e/src/fixtures/list-files.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ export function addListFilesResultFixtures(mock: InstanceType<typeof LLMock>) {
2929
arguments: '{"path":"list-files-tool-fixture","recursive":true}',
3030
toolCallId: "call_list_files_recursive_001",
3131
expected: ["nested/", "nested/deep/", "deep-nested-file.ts"],
32-
result: "The recursive listing for `list-files-tool-fixture` reached the nested structure and includes `nested/`, `deep/`, and `deep-nested-file.ts`.",
32+
result: "The recursive listing for `list-files-tool-fixture` reached the nested structure and includes `nested/`, `nested/deep/`, and `deep-nested-file.ts`.",
3333
id: "call_list_files_recursive_002",
3434
},
3535
{

apps/vscode-e2e/src/fixtures/search-files.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ export function addSearchFilesResultFixtures(mock: InstanceType<typeof LLMock>)
2828
id: "call_search_files_functions_002",
2929
},
3030
{
31-
userMessagePattern: "matching TODO entries",
31+
userMessagePattern: "TODO comments using the regex TODO",
3232
toolName: "search_files",
3333
arguments: '{"path":"search-files-tool-fixture","regex":"TODO.*"}',
3434
toolCallId: "call_search_files_todo_001",

apps/vscode-e2e/src/suite/tools/list-files.test.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -274,8 +274,7 @@ This directory contains various files and subdirectories for testing the list_fi
274274
(m) =>
275275
m.type === "say" &&
276276
(m.say === "completion_result" || m.say === "text") &&
277-
m.text?.includes("nested/") &&
278-
m.text?.includes("deep/") &&
277+
m.text?.includes("nested/deep/") &&
279278
m.text?.includes("deep-nested-file.ts"),
280279
)
281280
assert.ok(completionMessage, "AI should have summarized the recursive directory contents")

0 commit comments

Comments
 (0)