diff options
| author | Leo <[email protected]> | 2025-06-01 19:18:43 +0100 |
|---|---|---|
| committer | GitHub <[email protected]> | 2025-06-01 11:18:43 -0700 |
| commit | edc12e416d0b9daf24ede50cb18b012cb2b6e18a (patch) | |
| tree | 3c04e0685181948c016ed8663b980195fb05a788 /packages/core/src/tools/edit.test.ts | |
| parent | 9dae07784b82e9cc3306fa146de2956d574daa2d (diff) | |
Update edit tool validation function to override validateToolParams (#667)
Diffstat (limited to 'packages/core/src/tools/edit.test.ts')
| -rw-r--r-- | packages/core/src/tools/edit.test.ts | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/packages/core/src/tools/edit.test.ts b/packages/core/src/tools/edit.test.ts index 08d0860d..e0de98d5 100644 --- a/packages/core/src/tools/edit.test.ts +++ b/packages/core/src/tools/edit.test.ts @@ -178,14 +178,14 @@ describe('EditTool', () => { }); }); - describe('validateParams', () => { + describe('validateToolParams', () => { it('should return null for valid params', () => { const params: EditToolParams = { file_path: path.join(rootDir, 'test.txt'), old_string: 'old', new_string: 'new', }; - expect(tool.validateParams(params)).toBeNull(); + expect(tool.validateToolParams(params)).toBeNull(); }); it('should return error for relative path', () => { @@ -194,7 +194,9 @@ describe('EditTool', () => { old_string: 'old', new_string: 'new', }; - expect(tool.validateParams(params)).toMatch(/File path must be absolute/); + expect(tool.validateToolParams(params)).toMatch( + /File path must be absolute/, + ); }); it('should return error for path outside root', () => { @@ -203,7 +205,7 @@ describe('EditTool', () => { old_string: 'old', new_string: 'new', }; - expect(tool.validateParams(params)).toMatch( + expect(tool.validateToolParams(params)).toMatch( /File path must be within the root directory/, ); }); |
