rom-rb/devtools — pull request #46 by exbinary (2013-12-07 05:25:14+00:00)
Title: Make coverage task execute unit specs, even if already run.

`metrics:coverage` currently uses `Rake::Task['spec:unit'].invoke`, which is a no-op if the specs have already been run. This results in coverage being foregone when, e.g. `rake ci` is invoked.

The issue is reproducible by comparing the output of `rake metrics:coverage` to `rake spec:unit metrics:coverage`. I believe we'd expect the latter to run the specs once and if they pass, run them again with coverage. I think using #execute is the simplest way to make that happen, but maybe i'm missing something?

[This commit](https://github.com/rom-rb/devtools/commit/dc3710c55718f16848857acc8951ed06134f517b) from March switched from #execute to #invoke; it wasn't clear why since that didn't seem to be the primary purpose of the commit.

Please let me know if there's a more preferable way to bring such minor issues to your attention. Thanks!