theappbusiness / android-proxy-toggle

Small application to help android developers to quickly enable and disable proxy settings
MIT License
425 stars 59 forks source link

[Test] Improve test coverage #9

Closed fmontesino closed 4 years ago

fmontesino commented 4 years ago

Do not merge before #8

Why?

We want to improve our testing coverage

What?

Added a bunch of missing unit tests I didn't do because I'm lazy 😜

Bonus

I learnt how to use Hilt during tests to replace dependencies. Pretty cool.

codecov-commenter commented 4 years ago

Codecov Report

Merging #9 into main will increase coverage by 9.31%. The diff coverage is n/a.

@@             Coverage Diff              @@
##               main       #9      +/-   ##
============================================
+ Coverage     32.86%   42.17%   +9.31%     
+ Complexity       37       32       -5     
============================================
  Files            18       15       -3     
  Lines           213      147      -66     
  Branches         17       12       -5     
============================================
- Hits             70       62       -8     
+ Misses          139       84      -55     
+ Partials          4        1       -3     
Impacted Files Coverage Δ Complexity Δ
...feature/manager/viewmodel/ProxyManagerViewModel.kt 100.00% <ø> (ø) 7.00 <0.00> (ø)
...ain/java/com/kinandcarta/create/proxytoggle/App.kt 0.00% <0.00%> (-100.00%) 0.00% <0.00%> (-1.00%)
...reate/proxytoggle/feature/tile/ProxyTileService.kt 0.00% <0.00%> (-89.48%) 0.00% <0.00%> (-8.00%)
...proxytoggle/feature/widget/ToggleWidgetProvider.kt 0.00% <0.00%> (-2.95%) 0.00% <0.00%> (-1.00%)
...inandcarta/create/proxytoggle/model/ProxyMapper.kt 85.71% <0.00%> (+14.28%) 5.00% <0.00%> (+1.00%)
...oggle/broadcast/ProxyUpdateListenerProviderImpl.kt 100.00% <0.00%> (+100.00%) 2.00% <0.00%> (+2.00%)
...ture/widget/broadcast/WidgetProxyUpdateListener.kt 100.00% <0.00%> (+100.00%) 2.00% <0.00%> (+2.00%)