ggshield icon indicating copy to clipboard operation
ggshield copied to clipboard

Improve test coverage

Open agateau-gg opened this issue 2 years ago • 0 comments

Description

Our test coverage is currently at 93%. This sounds good, but some files actually cheat and make too much use of # pragma: no cover. Here is a table of the top 3 (ab)users of this directive:

Module statements missing excluded coverage
ggshield/ci.py 10 0 154 100%
ggshield/dev_scan.py 39 0 75 100%
ggshield/hook_cmd.py 30 0 42 100%

To do

  • [ ] Inspect these 3 files
  • [ ] If possible, write tests to cover some of the currently excluded code and remove the "no cover" directive

agateau-gg avatar Apr 01 '22 08:04 agateau-gg