Skip to content

feat: add comprehensive test coverage for tools module - #172

Open
maslinedwin wants to merge 3 commits into
xai-org:mainfrom
maslinedwin:fix/issue-149
Open

feat: add comprehensive test coverage for tools module#172
maslinedwin wants to merge 3 commits into
xai-org:mainfrom
maslinedwin:fix/issue-149

Conversation

@maslinedwin

Copy link
Copy Markdown

Fixes #149

This PR adds comprehensive unit tests for the tools module, covering:

  • web_search() with various configurations (domains, image options, location)
  • x_search() with date ranges, handles, and media options
  • code_execution() tool creation
  • collections_search() with limits and instructions
  • mcp() tool with labels, descriptions, authorization, and headers
  • get_tool_call_type() for all supported tool types

Provides ~200 lines of test coverage for critical tool creation functions.

@maslinedwin
maslinedwin requested a review from a team as a code owner July 10, 2026 02:31
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