Migrate detekt-gradle-plugin tests to JUnit

[3flex]

See #4450

[codecov]

Codecov Report

Merging #4529 (3e12b6e) into main (7126e4b) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main    #4529   +/-   ##
=========================================
  Coverage     84.45%   84.45%           
  Complexity     3330     3330           
=========================================
  Files           479      479           
  Lines         11143    11143           
  Branches       2040     2040           
=========================================
  Hits           9411     9411           
  Misses          699      699           
  Partials       1033     1033           

Impacted Files	Coverage Δ
...osch/detekt/rules/exceptions/SwallowedException.kt	77.04% <0.00%> (ø)

---

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 7126e4b...3e12b6e. Read the comment docs.

[BraisGabin]

Don't commit this! For sure it will not compile.

Suggested change

	fun scenarios(): List<Arguments> = listOf(
	arguments(groovy, groovyBuildFileContent),
	arguments(kotlin, kotlinBuildFileContent)
	)
	@ParameterizedTest(name = "Using {0}")
	@MethodSource("scenarios")
	class Scenarios : ArgumentsProvider {
	override fun provideArguments(context: ExtensionContext) = Stream.of(
	arguments(DslTestBuilder.groovy(), groovyBuildFileContent),
	arguments(DslTestBuilder.kotlin(), kotlinBuildFileContent),
	)
	}
	@ParameterizedTest(name = "Using {0}")
	@ArgumentsSource(DetektReportMergeSpec.Scenarios::class)

But what do you think of something like this? It avoid String references that are easier to break. The problem is that it's more verbose. It's just a suggestion.

[3flex]

I think the string reference is OK in this case, given the function sits directly above the test method. Also I just tried and IntelliJ will update the string in the annotation if the method is renamed using Refactor -> Rename in the IDE, and also if the string itself is renamed with Refactor -> Rename.

PR is complete so this is ready for a fresh review.

[cortinico]

Generally looks good to me 👍 Diff is quite big so I haven't looked into too much details as I believe most of the changes were auto-generated.

[3flex]

Diff is quite big

Just wait for detekt-rules-style migration :D

(With more to do)

[cortinico]

Just wait for detekt-rules-style migration :D

Ahahah my brain refuses to process whatever is above 1k loc 😅