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

Fix KoinTestRule test exception handling. Fixes gh-803 #808

Merged
merged 8 commits into from
May 28, 2020

Conversation

mottljan
Copy link

resolve #803

Fixes a bug in the KoinTestRule related to the thrown exception in the test. When the exception was thrown in the test, KoinTestRule did not handle that properly and did not stop Koin which caused failure of next tests.

@arnaudgiuliani arnaudgiuliani changed the base branch from master to 2.1.x May 25, 2020 07:27
@arnaudgiuliani arnaudgiuliani added test status:accepted accepted to be developed status:checking currently in analysis - discussion or need more detailed specs labels May 25, 2020
@arnaudgiuliani arnaudgiuliani added this to the 2.1.6 milestone May 25, 2020
@arnaudgiuliani arnaudgiuliani removed the status:checking currently in analysis - discussion or need more detailed specs label May 28, 2020
@arnaudgiuliani arnaudgiuliani merged commit da6c19e into InsertKoinIO:2.1.x May 28, 2020
@mottljan mottljan deleted the fix/803-koin-test-rule branch May 28, 2020 16:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status:accepted accepted to be developed test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

KointTestRule doesn't catch potential exceptions in tests
7 participants