Skip to content

fix(tests): fix KeyError when accessing tool name in test_tool_manage… - #2

Open
Novazeb wants to merge 1 commit into
IMPHNEN:mainfrom
Novazeb:fix/test-tool-manager
Open

Novazeb wants to merge 1 commit into
IMPHNEN:mainfrom
Novazeb:fix/test-tool-manager

Conversation

@Novazeb

@Novazeb Novazeb commented Sep 8, 2026 •

Copy link
Copy Markdown

Description

Fixes KeyError: 'function' in tests/test_basic.py (test_tool_manager_has_tools).

ToolManager.list_tools() returns {"name": ..., "description": ...}, but the test was accessing tool['function']['name'], causing unittest to fail.

Type of Change

  • Bug fix
  • Test improvements

Changes Made

  • Update test_tool_manager_has_tools in tests/test_basic.py to use tool['name'].

Testing

  • Existing tests pass (python -m unittest -> OK)

Test Configuration:

  • OS: Windows
  • Python version: 3.14

Checklist

  • My code follows the project's coding standards
  • I have performed a self-review of my own code
  • New and existing unit tests pass locally with my changes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant