Commit 833b22f
committed
test: add comprehensive ListFilesTool test suite (40 tests)
Add full test coverage for ListFilesTool including:
- Parameter validation (missing path)
- Basic and recursive file listing
- Approval flow and user cancellation
- Relative and absolute path resolution
- Error handling (nonexistent dirs, permission denied)
- Partial message streaming (handlePartial)
- Edge cases (empty dirs, large file sets, root/dot paths)
- Singleton instance verification
- RooIgnore/RooProtected controller passthrough
Fixes cross-platform compatibility for Windows CI by using
path.resolve() instead of hardcoded Unix paths, and removing
the unnecessary path module mock that caused failures on Windows.
Closes #2091 parent 79ed8b5 commit 833b22f
1 file changed
Lines changed: 590 additions & 0 deletions
0 commit comments