kodi-plugin-routing icon indicating copy to clipboard operation
kodi-plugin-routing copied to clipboard

Add coverage support

Open dagwieers opened this issue 6 years ago • 5 comments

This adds coverage support for unit tests.

dagwieers avatar Aug 31 '19 20:08 dagwieers

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@744281a). Click here to learn what that means. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master      #19   +/-   ##
=========================================
  Coverage          ?   92.52%           
=========================================
  Files             ?        2           
  Lines             ?      174           
  Branches          ?        0           
=========================================
  Hits              ?      161           
  Misses            ?       13           
  Partials          ?        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 744281a...9e723d5. Read the comment docs.

codecov-io avatar Aug 31 '19 20:08 codecov-io

I would rather get rid of @codecov-io messages though.

dagwieers avatar Aug 31 '19 21:08 dagwieers

The latest changes disable the @codecov-io bot.

dagwieers avatar Sep 01 '19 14:09 dagwieers

So a coverage of 97.94% is the best we can do without Mocking xbmc and xbmcaddon. If we want to get to 100% coverage we have to be able to import xbmc and xbmcaddon. I don't mind doing that if you agree.

https://codecov.io/gh/tamland/kodi-plugin-routing/commits

dagwieers avatar Sep 05 '19 00:09 dagwieers

Can we see this merged? It would be nice to know exactly what has not been covered by tests.

dagwieers avatar Sep 10 '19 21:09 dagwieers