test(http_request): add integration test for http_request tool #113
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
zastrowm
requested changes
Jul 2, 2025
…0 unit test fail when user has faiss installed
JackYPCOnline
commented
Jul 3, 2025
zastrowm
approved these changes
Jul 7, 2025
jimbrub
added a commit
to jimbrub/tools
that referenced
this pull request
Jul 15, 2025
author James Brubaker <[email protected]> 1750079629 -0400 committer James Brubaker <[email protected]> 1752586952 -0400 * test(memory): add an end-to-end integration test * test: refactor code to be more readable --------- Co-authored-by: Jack Yuan <[email protected]> fix(generate_image): updating generate_image tool to support additional models in Amazon Bedrock (strands-agents#89) * switching region to us-east-1 as sdxl model is no longer available in us-west-2 * fix(image-gen): add support for Amazon Nova Canvas Fix image generation functionality by adding support for Amazon Nova Canvas model in addition to existing Stable Diffusion models. This enhancement allows users to generate images using both model types with appropriate parameters. * fix(gen_image): region selection and number of images parmaters * fix: updating toolspec to include region id and remove number of images parameter (defaulting the tool to generate one image always * fix(generate_image): update the tool to initial state where it only support stable diffusion models * Update src/strands_tools/generate_image.py Co-authored-by: Mackenzie Zastrow <[email protected]> * fix(generate_image): removing conditional for model specific * Update src/strands_tools/generate_image.py Add end to end test for read, write, edit tools (strands-agents#100) * test(read_write_edit): add end to end tests for these tools --------- Co-authored-by: Jack Yuan <[email protected]> test(generate_read_image): add end to end tests for image tools (strands-agents#107) Co-authored-by: Jack Yuan <[email protected]> feat: migrate editor tool from TOOL_SPEC to @tool decorator (strands-agents#111) Adding use_computer tool fix: Updating readme with fix for use_browser tool fix: replace colorama with rich's native styling in use_aws tool (strands-agents#118) * fix: replace colorama with rich's native styling in use_aws tool - Replace colorama Fore/Style with rich Table for better UI rendering - Add structured table layout for AWS operation details - Improve visual consistency with other rich-based tools - Remove colorama dependency from use_aws module - Fixes strands-agents#35: use_aws tool rich UI rendering issues * chore: remove colorama dependency from pyproject.toml - No longer needed after replacing colorama with Rich native styling - Fixes Rich panel border rendering issues when mixing ANSI sequences - Part of use-aws Rich UI rendering improvements --------- Co-authored-by: Strands Agent <[email protected]> Add stability tool (strands-agents#101) Adds a tool that will create images using the Stability AI Platform + related documentation feat: updating README.md with information about use_computer test(http_request): add integration test for http_request tool (strands-agents#113) * test(http_request): add integration test for http_request tool * test(http_request): use self host server to handle the tests, fix mem0 unit test fail when user has faiss installed --------- Co-authored-by: Jack Yuan <[email protected]> fix: Fixing optional dependencies for use_browser tool (strands-agents#119) * fix: Fixing optional dependencies for use_browser tool * fix: Fixing optional dependencies for use_browser tool * fix: Updating readme with fix for use_browser tool fix: fixing dependencies and merging with main branch feat(a2a): native client tool (strands-agents#108) Co-authored-by: jer <[email protected]> refactor(a2a): rename dependency target and update docs (strands-agents#125) Co-authored-by: jer <[email protected]> adding code to delete screenshots taken by use_computer adding pytesseract dependency as optional
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
[Provide a detailed description of the changes in this PR]
add serval tests for http_request tool.
Related Issues
[Link to related issues using #issue-number format]
Documentation PR
[Link to related associated PR in the agent-docs repo]
Type of Change
Testing
[How have you tested the change?]
hatch fmt --linterhatch fmt --formatterhatch test --allChecklist
[x ] I have read the CONTRIBUTING document
[x ] I have added tests that prove my fix is effective or my feature works
[x ] I have updated the documentation accordingly
[x ] I have added an appropriate example to the documentation to outline the feature
[x ] My changes generate no new warnings
[x ] Any dependent changes have been merged and published
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.