Inline MethodOverloading case file

[schalkms]

No description provided.

[codecov-io]

Codecov Report

Merging #2241 into master will increase coverage by 0.02%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #2241      +/-   ##
============================================
+ Coverage     81.35%   81.37%   +0.02%     
- Complexity     2067     2073       +6     
============================================
  Files           341      343       +2     
  Lines          5968     5981      +13     
  Branches       1086     1086              
============================================
+ Hits           4855     4867      +12     
  Misses          532      532              
- Partials        581      582       +1

Impacted Files	Coverage Δ	Complexity Δ
...bosch/detekt/rules/providers/CoroutinesProvider.kt	100% <0%> (ø)	3% <0%> (?)
...ch/detekt/rules/coroutines/GlobalCoroutineUsage.kt	88.88% <0%> (ø)	3% <0%> (?)

---

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 43a0982...1792311. Read the comment docs.

[BraisGabin]

We are losing the test of overloading in an inner class. I didn't check if it's really needed or not. I just point it out if it was removed by mistake. If it was intentional, all good.

[schalkms]

We are losing the test of overloading in an inner class.

The reason for that is described in #2247. I'll tackle this bug fix in another PR.
The test case was not catching that because it saw the function x() as a single overload (6 times). @BraisGabin

class OverloadedMethods {
  fun x() {}
  fun x(i: Int) {}
  fun x(i: Int, j: Int) {}

  private class InnerClass {
    // if these functions were named `y()` instead the bug would have been caught.
    fun x() {}
    fun x(i: Int) {}
    fun x(i: Int, j: Int) {}
  }
}