Add informative message to UselessCallOnNotNull report

[veyndan]

No description provided.

[codecov]

Codecov Report

Merging #2920 into master will decrease coverage by 0.00%.
The diff coverage is 70.00%.

@@             Coverage Diff              @@
##             master    #2920      +/-   ##
============================================
- Coverage     80.23%   80.23%   -0.01%     
- Complexity     2448     2449       +1     
============================================
  Files           421      421              
  Lines          7402     7407       +5     
  Branches       1352     1354       +2     
============================================
+ Hits           5939     5943       +4     
  Misses          764      764              
- Partials        699      700       +1     

Impacted Files	Coverage Δ	Complexity Δ
...urbosch/detekt/rules/style/UselessCallOnNotNull.kt	80.76% <70.00%> (-0.19%)	8.00 <0.00> (+1.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 43ac74a...9262d9e. Read the comment docs.

[schalkms]

I think adding some tests would be beneficial.

[cortinico]

I think adding some tests would be beneficial.

Yup, @veyndan you should be able to assert on the findings.message inside your tests

[veyndan]

@schalkms @cortinico I expanded the previous tests to include verification of the error message. I'm not too sure why the code coverage tool is failing though…

[cortinico]

@schalkms @cortinico I expanded the previous tests to include verification of the error message. I'm not too sure why the code coverage tool is failing though…

The failure is related to a drop in codecov coverage for your diff. You can disregard this has it has been really flaky recently (also the overall coverage is increasing).