detekt / detekt

Static code analysis for Kotlin
https://detekt.dev
Apache License 2.0
6.07k stars 751 forks source link

Improve core tests #7232

Closed BraisGabin closed 2 weeks ago

BraisGabin commented 2 weeks ago

This PR include the 2 first commits of #7101. The idea is that this will simplify the review of that other PR.

In this PR I make that the custom rules that we have for tests on core report the correct location of the issue. This is done because later I want to test the suppression and it only works if the location if correct.

codecov[bot] commented 2 weeks ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 84.69%. Comparing base (78fa964) to head (0eb262c).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #7232 +/- ## ========================================= Coverage 84.68% 84.69% Complexity 3984 3984 ========================================= Files 578 578 Lines 12158 12158 Branches 2499 2499 ========================================= + Hits 10296 10297 +1 Misses 626 626 + Partials 1236 1235 -1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.