Fix issues with Elvis operator in UnconditionalJumpStatementInLoop

[MiSikora]

It was hard to determine which features I could break during development, so I split the tests into more granular ones.

I tried my best to cover the rule. Still, suppose anyone has ideas for adding more tests for false-positive or false-negative checks. In that case, I'll gladly add them as this rule is quite tricky, in my opinion: especially given nesting and such.

Closes #4134.

[codecov]

Codecov Report

Merging #4150 (e5ec18e) into main (4e265df) will increase coverage by 0.00%.
The diff coverage is 83.33%.

@@            Coverage Diff            @@
##               main    #4150   +/-   ##
=========================================
  Coverage     83.45%   83.45%           
+ Complexity     3185     3181    -4     
=========================================
  Files           463      465    +2     
  Lines          9095     9104    +9     
  Branches       1768     1774    +6     
=========================================
+ Hits           7590     7598    +8     
- Misses          571      572    +1     
  Partials        934      934           

Impacted Files	Coverage Δ
...ekt/rules/bugs/UnconditionalJumpStatementInLoop.kt	95.00% <83.33%> (+5.00%)	⬆️
.../arturbosch/detekt/rules/style/UseIsNullOrEmpty.kt	54.66% <0.00%> (-2.67%)	⬇️
...bosch/detekt/rules/complexity/LongParameterList.kt	83.33% <0.00%> (-1.12%)	⬇️
...otlin/io/gitlab/arturbosch/detekt/core/Analyzer.kt	80.95% <0.00%> (-0.30%)	⬇️
...ab/arturbosch/detekt/core/config/ValidateConfig.kt	100.00% <0.00%> (ø)
...tlab/arturbosch/detekt/rules/style/UseDataClass.kt	77.94% <0.00%> (ø)
...b/arturbosch/detekt/rules/naming/FunctionNaming.kt	100.00% <0.00%> (ø)
...tlab/arturbosch/detekt/rules/bugs/LateinitUsage.kt	90.00% <0.00%> (ø)
...sch/detekt/rules/style/UnnecessaryAbstractClass.kt	82.92% <0.00%> (ø)
...etekt/rules/style/FunctionOnlyReturningConstant.kt	94.87% <0.00%> (ø)
... and 4 more

---

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 4e265df...e5ec18e. Read the comment docs.

[chao2zhang]

Thank you for fixing two bugs (The originally reported one for isConditionalJumpStatement() and the additional one for isFollowedByElvisJump())!

[BraisGabin]

👏👏👏 great refactor in the tests. It's way easier now. Great first contribution 👍.