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

Move tests to the correct module #4314

Merged
merged 2 commits into from
Nov 20, 2021
Merged

Move tests to the correct module #4314

merged 2 commits into from
Nov 20, 2021

Conversation

BraisGabin
Copy link
Member

We were checking that the detekt-default-config file were correct in detekt-core. But it is generated in detekt-generate. It have more sense to test that the file is created correctly in detekt-generate than in detekt-core. In any case these tests are a bit strange.

@BraisGabin BraisGabin added the housekeeping Marker for housekeeping tasks and refactorings label Nov 20, 2021
@BraisGabin BraisGabin force-pushed the improve-test-default-config branch from b9fcf28 to 317c8ae Compare November 20, 2021 17:39
@codecov
Copy link

codecov bot commented Nov 20, 2021

Codecov Report

Merging #4314 (701fd55) into main (5f9e4a5) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##               main    #4314   +/-   ##
=========================================
  Coverage     84.22%   84.22%           
  Complexity     3258     3258           
=========================================
  Files           472      472           
  Lines         10309    10309           
  Branches       1820     1820           
=========================================
  Hits           8683     8683           
  Misses          666      666           
  Partials        960      960           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5f9e4a5...701fd55. Read the comment docs.

@BraisGabin BraisGabin force-pushed the improve-test-default-config branch from 317c8ae to 701fd55 Compare November 20, 2021 19:50
@schalkms schalkms merged commit 7b8d273 into main Nov 20, 2021
@schalkms schalkms deleted the improve-test-default-config branch November 20, 2021 20:57
@cortinico cortinico added this to the 1.19.0 milestone Nov 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
housekeeping Marker for housekeeping tasks and refactorings
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants