Added include and exclude params for coverage reports (dup for issue #6)#58
Added include and exclude params for coverage reports (dup for issue #6)#58
Conversation
|
I'm just wondering. Any opinions? |
|
I'm not using code coverage yet so I don't have experience on the matter, but I see this change as low-hanging fruit I'd like to include as similar to the db framework as possible. |
|
@justedro: I'd like to add a system test for the code-coverage use case before merging. We don't have one yet and for include/exclude functionality only affects these use-cases I'd want to add it right now. |
Also separated unit- and system-tests for RunCommand and refactored RunCommand a bit to use the fluent api provided
No description provided.