mszostok / codeowners-validator

The GitHub CODEOWNERS file validator
Apache License 2.0
219 stars 47 forks source link

Add test coverage to internal pkg #14

Closed mszostok closed 3 years ago

mszostok commented 4 years ago

Description

Currently, only the pkg package has full test coverage. The internal pkg also should be full-tested. The coverage reports should be generated and reported on some external services that we do not have to manage.

Choose one of the following services:

Acceptance Criteria

mszostok commented 4 years ago

The integration tests were added in this PR: #30

mszostok commented 3 years ago

The unit-test coverage was increased here: https://github.com/mszostok/codeowners-validator/pull/43

mszostok commented 3 years ago

The unit-tests were added for all places where it makes sense. Rest is covered by integration tests.

There is a follow-up issue to show the test-coverage from integration tests (https://github.com/mszostok/codeowners-validator/issues/49)