Issues #29 & #95

[StephenMP]

Issues #29 & #95 : Displays the average coverage % for all coverage types

[codecov]

Codecov Report

Merging #104 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #104   +/-   ##
=======================================
  Coverage   98.91%   98.91%           
=======================================
  Files          15       15           
  Lines        1378     1378           
=======================================
  Hits         1363     1363           
  Misses         15       15

---

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 883d1f7...6dccea3. Read the comment docs.

[StephenMP]

Interesting that run two different times, codecov reports two different results...

[StephenMP]

Made a benign whitespace change to get the build process to re-run and fix Codecov issues. @tonerdo let me know if there's anything I need to do.

[tonerdo]

@StephenMP no need. Will take a look once I get the chance

[tonerdo]

Hiya @StephenMP Can I please see a screenshot of the final output. Also, I have my reservations about this feature considering the 2.0 update simply compares each assembly's coverage against the threshold instead of the average coverage

[StephenMP]

@tonerdo

The final output looks like the following

Starting test execution, please wait...

  Total tests: 87. Passed: 87. Failed: 0. Skipped: 0.
  Test Run Successful.
  Test execution time: 9.2234 Seconds

  Calculating coverage result...
    Generating report 'path\to\coverage.xml'

  +---------------+--------+--------+--------+
  | Module        | Line   | Branch | Method |
  +---------------+--------+--------+--------+
  | coverlet.core | 90.3%  | 85.5%  | 94.1%  |
  +---------------+--------+--------+--------+

  +---------+--------+--------+--------+
  |         | Line   | Branch | Method |
  +---------+--------+--------+--------+
  | Average | 90.3%  | 85.5%  | 94.1%  |
  +---------+--------+--------+--------+

If you have your reservations, that's fine, I was just adding something suggested by two open tickets. If you don't want to include it you can close the pull request and the tickets 😄

[basilfx]

I would love to have this averaging feature.

In my solution I have unit tests and integration tests over multiple projects. Since these tests complement each other, it makes more sense to look at the average coverage, instead of the lowest coverage.

[tonerdo]

Well I'm on board then