Can we discuss the approach & confirm on #8365 before proceeding with this PR?

There was a reason the modules were split in the first place, so we may not want to go down this path... the discussion hadn't reached a conclusion yet.

Codecov Report

❌ Patch coverage is 80.00000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 85.21%. Comparing base (3e5216b) to head (9994131).
⚠️ Report is 48 commits behind head on main.

@@             Coverage Diff              @@
##               main    #8898      +/-   ##
============================================
- Coverage     85.37%   85.21%   -0.17%     
- Complexity     4384     4392       +8     
============================================
  Files           570      574       +4     
  Lines         12567    12639      +72     
  Branches       2623     2633      +10     
============================================
+ Hits          10729    10770      +41     
- Misses          695      716      +21     
- Partials       1143     1153      +10     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Can we discuss the approach & confirm on #8365 before proceeding with this PR?

There was a reason the modules were split in the first place, so we may not want to go down this path... the discussion hadn't reached a conclusion yet.

Hi from the PR it seems that we just need to remove functions which are related to detekt testing to internal module(internal folder of the dev.detekt.test should suffice here)
For that can we review the detekt-test/api/detekt-test.api file?

My comment wasn't about the implementation of the change, it was about this comment.

There may be valid reasons to keep the split. We didn't fully resolve that question or make a decision, so that should be done on the original issue first.