Follow-up #6572: Re-order test classpath to load correct Logger as workaround for KT-60813

[TWiStErRob]

• Follow-up on https://github.com/detekt/detekt/pull/6572/files#r1386261829

This solution replaces the slf4j Logger with the correct one by classpath ordering: c4ef737

I ran into #4140/#4170 not being implemented fully (current implementation is based on assumptions). The assumption is that the / resource on the classpath points to the current module's test sourceSet. By changing the order I changed this and now the slf4j.jar is on the classpath first, which triggers #4140. I was able to workaround this by looking for a specific file that's in the correct folder (ba150db), but it's not scalable, so I instead changed the logic to insert the new dependency in the middle of the classpath: 68352ce

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (933be47) 85.13% compared to head (68352ce) 85.13%.

Additional details and impacted files

@@            Coverage Diff            @@
##               main    #6636   +/-   ##
=========================================
  Coverage     85.13%   85.13%           
  Complexity     4079     4079           
=========================================
  Files           570      570           
  Lines         13363    13363           
  Branches       2402     2402           
=========================================
  Hits          11376    11376           
  Misses          791      791           
  Partials       1196     1196           

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

[detekt-ci]

	Warnings
⚠️	It looks like this PR contains functional changes without a corresponding test.

Generated by 🚫 dangerJS against 68352ce

[3flex]

Kotlin 1.9.21 fixes KT-60813 so I've opened #6646 to remove the workaround introduced in #6636. This PR isn't required anymore.