ESMCI / manage_externals / 150

86%

push

travis-ci

Merge pull request #93 from billsacks/work_on_coverage

Run a single coverage command rather than two separate commands

I was finding that, even with the '--append' flag, some of the coverage
results were getting overwritten in the second run, resulting in a
too-low coverage report.

This turns out to have no effect on the reported coverage on master, but
it fixes the coverage report for #90: without this change, some lines
that are covered only by unit tests are listed as uncovered in the final
coverage report (at least, I think that's what was going on).

User interface changes?: No

Fixes: None

Testing:
  test removed: changed 'make coverage' operation
  unit tests:
  system tests:
  manual testing:

340 of 394 branches covered (86.29%)

Branch coverage included in aggregate %.

1056 of 1149 relevant lines covered (91.91%)

3.68 hits per line