Skip to content

test: Fixes adk cli options and method parameters mismatching and adds a unit test for future proof checking #2339

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.

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

Merged
merged 1 commit into from
Aug 5, 2025

Conversation

copybara-service[bot]
Copy link

@copybara-service copybara-service bot commented Aug 4, 2025

test: Fixes adk cli options and method parameters mismatching and adds a unit test for future proof checking

The test will fail if @option list and method parameter don't match.

Future proof test for #2328

@copybara-service copybara-service bot added the google-contributor [Bot] This PR is created by Google label Aug 4, 2025
@copybara-service copybara-service bot changed the title test: Adds a unit test to check whether a click option is missing in the method parameter list test: Fixes adk cli options and method parameters mismatching and adds a unit test for future proof checking Aug 5, 2025
@copybara-service copybara-service bot force-pushed the copybara/790939220 branch 2 times, most recently from a06ac21 to 00a2e01 Compare August 5, 2025 04:05
…s a unit test for future proof checking

The test will fail if `@option` list and method parameter don't match.

Future proof test for #2328

PiperOrigin-RevId: 791022512
@copybara-service copybara-service bot merged commit 8ef2177 into main Aug 5, 2025
@copybara-service copybara-service bot deleted the copybara/790939220 branch August 5, 2025 04:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
google-contributor [Bot] This PR is created by Google
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant