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(web): add Retrofit2ServiceFactoryTest to test the creation of retrofit2 client using Retrofit2ServiceFactory #1221

Merged

Conversation

kirangodishala
Copy link
Contributor

Adds test coverage to the following classes by creating a retrofit2 client: Retrofit2ServiceFactory, DefaultServiceClientProvider, OkHttpClientProvider.

@kirangodishala kirangodishala force-pushed the retrofit2service-factory-test branch 2 times, most recently from be469c5 to efdb5c2 Compare January 10, 2025 15:24
@kirangodishala kirangodishala force-pushed the retrofit2service-factory-test branch 2 times, most recently from 6785fc4 to ddb0bdb Compare January 13, 2025 17:18
…trofit2 client using Retrofit2ServiceFactory
@kirangodishala kirangodishala force-pushed the retrofit2service-factory-test branch from ddb0bdb to f79b5ec Compare January 15, 2025 08:06
@dbyron-sf dbyron-sf added the ready to merge Approved and ready for merge label Jan 15, 2025
@mergify mergify bot added the auto merged label Jan 15, 2025
@mergify mergify bot merged commit 1058c53 into spinnaker:master Jan 15, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants