From 4d22f25e203bb361a9fa3b70fedf57d1d85fbd09 Mon Sep 17 00:00:00 2001 From: Daniel Riccio Date: Sat, 26 Jul 2025 19:52:20 -0500 Subject: [PATCH] test: update list-files test to reflect fixed hidden files bug --- apps/vscode-e2e/src/suite/tools/list-files.test.ts | 14 +++++--------- 1 file changed, 5 insertions(+), 9 deletions(-) diff --git a/apps/vscode-e2e/src/suite/tools/list-files.test.ts b/apps/vscode-e2e/src/suite/tools/list-files.test.ts index 5340a13a161..c374e795159 100644 --- a/apps/vscode-e2e/src/suite/tools/list-files.test.ts +++ b/apps/vscode-e2e/src/suite/tools/list-files.test.ts @@ -242,8 +242,8 @@ This directory contains various files and subdirectories for testing the list_fi // Verify the tool returned the expected files (non-recursive) assert.ok(listResults, "Tool execution results should be captured") - // Check that expected root-level files are present (excluding hidden files due to current bug) - const expectedFiles = ["root-file-1.txt", "root-file-2.js", "config.yaml", "README.md"] + // Check that expected root-level files are present (including hidden files now that bug is fixed) + const expectedFiles = ["root-file-1.txt", "root-file-2.js", "config.yaml", "README.md", ".hidden-file"] const expectedDirs = ["nested/"] const results = listResults as string @@ -255,13 +255,9 @@ This directory contains various files and subdirectories for testing the list_fi assert.ok(results.includes(dir), `Tool results should include directory ${dir}`) } - // BUG: Hidden files are currently excluded in non-recursive mode - // This should be fixed - hidden files should be included when using --hidden flag - console.log("BUG DETECTED: Hidden files are excluded in non-recursive mode") - assert.ok( - !results.includes(".hidden-file"), - "KNOWN BUG: Hidden files are currently excluded in non-recursive mode", - ) + // Verify hidden files are now included (bug has been fixed) + console.log("Verifying hidden files are included in non-recursive mode") + assert.ok(results.includes(".hidden-file"), "Hidden files should be included in non-recursive mode") // Verify nested files are NOT included (non-recursive) const nestedFiles = ["nested-file-1.md", "nested-file-2.json", "deep-nested-file.ts"]