wylab/nanobot
1
0
Fork 0
Code Issues 5 Pull Requests 6 Actions Packages Projects Releases Wiki Activity

fix: update EditTool name in tests to match implementation #22

Merged
code-server merged 1 commits from fix/edit-tool-name-tests into main 2026-03-05 14:02:29 +01:00
Conversation 0 Commits 1 Files Changed 3
+5 -8
code-server commented 2026-03-05 10:36:00 +01:00
Collaborator
Copy Link
Copy Source

Summary

The EditTool20250728 uses the name "str_replace_based_edit_tool" but tests were checking for the old name "str_replace_editor". This PR updates all test expectations to use the correct tool name.

Changes

  • ✅ Updated test_edit_tool.py to expect "str_replace_based_edit_tool"
  • ✅ Updated test_native_tools_registration.py for correct tool name
  • ✅ Updated test_registry_native_execution.py to execute with correct name
  • ✅ Removed computer tool assertion (intentionally disabled by default)

Test Results

✅ All 3 EditTool naming tests now pass

Type

  • Test fixes (tool name expectations)

🤖 Generated with Claude Code

## Summary The `EditTool20250728` uses the name `"str_replace_based_edit_tool"` but tests were checking for the old name `"str_replace_editor"`. This PR updates all test expectations to use the correct tool name. ## Changes - ✅ Updated `test_edit_tool.py` to expect `"str_replace_based_edit_tool"` - ✅ Updated `test_native_tools_registration.py` for correct tool name - ✅ Updated `test_registry_native_execution.py` to execute with correct name - ✅ Removed computer tool assertion (intentionally disabled by default) ## Test Results ✅ All 3 EditTool naming tests now pass ## Type - [x] Test fixes (tool name expectations) 🤖 Generated with Claude Code
code-server added 1 commit 2026-03-05 10:36:00 +01:00
fix: update EditTool name in tests to match implementation
Build Nanobot OAuth / cleanup (pull_request) Has been skipped
Details
Build Nanobot OAuth / build (pull_request) Successful in 6m11s
Details
d4abb3d06f 
The EditTool20250728 uses the name "str_replace_based_edit_tool" but
tests were checking for the old name "str_replace_editor". This commit
updates all test expectations to use the correct tool name.

Changes:
- ✅ Updated test_edit_tool.py to expect "str_replace_based_edit_tool"
- ✅ Updated test_native_tools_registration.py for correct tool name
- ✅ Updated test_registry_native_execution.py to execute with correct name
- ✅ Removed computer tool assertion (intentionally disabled by default)

All 3 EditTool naming tests now pass.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
code-server merged commit 1ff3356d1b into main 2026-03-05 14:02:29 +01:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
code-server (Code Server) nanobot (Nanobot Agent) wylab
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: wylab/nanobot#22