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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enhance Android Instrumentation tests doc #1409

Merged
merged 2 commits into from Aug 29, 2022

Conversation

cmunaro
Copy link
Contributor

@cmunaro cmunaro commented Aug 12, 2022

I know that some people get frustrated when they try to make Instrumentation test with Koin for the lack of examples. (I was one of them 馃槄)
I propose a documentation update to make it easier for newcomers to create instrumentation tests with Koin. 馃敡
What do you think about it?

Related to #1318 and #850

I know that some people get frustrated when they try to make Instrumentation test with Koin for the lack of examples. 馃槄
I propose a documentation update to make it easier for newcomers to create instrumentation tests with Koin. 馃敡
What do you think about it?
@arnaudgiuliani arnaudgiuliani added this to the 3.2.1 milestone Aug 29, 2022
@arnaudgiuliani arnaudgiuliani self-assigned this Aug 29, 2022
@arnaudgiuliani arnaudgiuliani added the type:improvement Improving a current feature label Aug 29, 2022
@arnaudgiuliani
Copy link
Member

Good proposal that will help many people 馃憤

@sbarbour-93
Copy link

sbarbour-93 commented Sep 16, 2022

Thanks for updating the docs on this - it's a really useful update. However, I've noticed one issue in that if the module I try to override expects parameters to be injected in order for the instance to be built then my instrumentation tests fall over when using the new KoinTestRule approach with a DefinitionParameterException.

Is there any suggestions on this? If it needs a new issue to be opened in order to review please advise.

Thanks!

@arnaudgiuliani

@CoolMind
Copy link

See https://insert-koin.io/docs/reference/koin-android/instrumented-testing/ or commits here for an updated documentation.

@maeceloacm1998
Copy link

Muito obrigado, sua solu莽茫o me ajudou para fazer testes com espresso.

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.

None yet

5 participants