-
-
Notifications
You must be signed in to change notification settings - Fork 794
ThrowingExceptionInMain: fix #2248 and add tests #2240
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
Conversation
This PR adds test cases for main functions inside classes w/o @JvmStatic annotation.
This PR adds test cases for main functions inside objects w/o @JvmStatic annotation.
Codecov Report
@@ Coverage Diff @@
## master #2240 +/- ##
============================================
+ Coverage 81.37% 81.41% +0.04%
- Complexity 2073 2077 +4
============================================
Files 343 345 +2
Lines 5981 5989 +8
Branches 1086 1084 -2
============================================
+ Hits 4867 4876 +9
Misses 532 532
+ Partials 582 581 -1
Continue to review full report at Codecov.
|
.../src/test/kotlin/io/gitlab/arturbosch/detekt/rules/exceptions/ThrowingExceptionInMainSpec.kt
Outdated
Show resolved
Hide resolved
The test |
This commit also fixes a codefactor warning.
Fixes #2248