Skip to content

test(toolbox-adk): Implement parity integration tests #458

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
anubhav756 wants to merge 1 commit into
base: feature/tool_wrapper
Choose a base branch
from
Open

test(toolbox-adk): Implement parity integration tests #458

anubhav756 wants to merge 1 commit into from

Conversation

@anubhav756
Copy link
Contributor

@anubhav756 anubhav756 commented Dec 10, 2025 •
edited
Loading

Adds a comprehensive integration test suite for toolbox-adk.

  • Loading toolsets, running tools.
  • Bound parameters, Map parameters, Optional parameters.
  • Verifies auth_token_getters propagation and error handling.
  • Verifies correct error types for missing/invalid inputs.

@anubhav756 anubhav756 requested a review from a team as a code owner December 10, 2025 12:05
@anubhav756 anubhav756 marked this pull request as draft December 10, 2025 12:42
@anubhav756 anubhav756 force-pushed the feature/e2e_testing branch 3 times, most recently from f22f8e8 to bca0d6b Compare December 10, 2025 20:07
@anubhav756 anubhav756 force-pushed the feature/e2e_testing branch 5 times, most recently from c9a0500 to 8710e50 Compare December 10, 2025 23:28
@anubhav756 anubhav756 force-pushed the feature/e2e_testing branch 3 times, most recently from 3acda48 to b60d9b4 Compare December 12, 2025 14:40
@anubhav756 anubhav756 force-pushed the feature/tool_wrapper branch 2 times, most recently from 802328b to 61c1e52 Compare December 12, 2025 15:52
@anubhav756 anubhav756 force-pushed the feature/e2e_testing branch 3 times, most recently from 30ebfe2 to 606c00b Compare December 12, 2025 16:31
This was referenced Dec 12, 2025
Open
Open
Open
Open
@anubhav756 anubhav756 force-pushed the feature/e2e_testing branch 2 times, most recently from 75c8ed3 to 5b53949 Compare December 12, 2025 18:20
@anubhav756
test(toolbox-adk): Implement parity integration tests
4ff24fa 
Adds a comprehensive integration test suite for toolbox-adk matching toolbox-core coverage.

- Basic E2E: Loading toolsets, running tools.
- Parameters: Bound parameters, Map parameters, Optional parameters.
- Authentication: Verifies auth_token_getters propagation and error handling.
- Input Validation: Verifies correct error types for missing/invalid inputs.
@anubhav756 anubhav756 changed the title feat: Comprehensive e2e testing test(toolbox-adk): Implement parity integration tests Dec 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@averikitsch averikitsch Awaiting requested review from averikitsch

@kurtisvg kurtisvg Awaiting requested review from kurtisvg

Assignees

@anubhav756 anubhav756

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@anubhav756 @twishabansal