zambezi / ez-build

Zambezi build tool
MIT License
2 stars 1 forks source link

Refactoring for better testability #31

Closed mstade closed 8 years ago

mstade commented 8 years ago

Description

With #30 we gained some initial scaffolding for tests, and with that we'll start increasing the test coverage of this project. To that end, this PR aims to refactor ez-build to make it more testable, and ideally easier to implement new functionality as well. Additionally, integration with code coverage tooling is another explicit goal of this work.

Motivation and Context

While #30 gave us a good start, it didn't include code coverage, and also highlighted that some parts (the main routine, for instance) was muddied by mixing too many concerns.

How Was This Tested?

Additional tests are included, to increase coverage, and instrumented building is enabled for the test script as well, to ensure we can get accurate code coverage reports on every build.

Types of changes

codecov-io commented 8 years ago

Current coverage is 78.63% (diff: 91.17%)

No coverage report found for master at e73ea49.

Powered by Codecov. Last update e73ea49...c29240b

Poetro commented 8 years ago

:+1: