Skip to content
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

test: minor test cleanup #5679

Merged
merged 1 commit into from
Oct 3, 2022
Merged

test: minor test cleanup #5679

merged 1 commit into from
Oct 3, 2022

Conversation

dfawley
Copy link
Member

@dfawley dfawley commented Sep 30, 2022

Cleans up a minor nit I discovered while reviewing #5678. Removes magic setting of dialTargetWant and set it explicitly in all test case definitions instead.

RELEASE NOTES: n/a

@dfawley dfawley added this to the 1.50 Release milestone Sep 30, 2022
@dfawley dfawley requested a review from easwars September 30, 2022 17:04
@easwars easwars assigned dfawley and unassigned easwars Sep 30, 2022
@dfawley dfawley merged commit be4b63b into grpc:master Oct 3, 2022
@dfawley dfawley deleted the test_cleanup branch October 3, 2022 15:37
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 2, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants