Clover coverage report counts ignored methods

[dharkness]

PHP_CodeCoverage_Report_Clover::process() should make sure at least one line of a method is not ignored before incrementing the number of methods in a class. It should probably do the same thing for classes.

[dharkness]

An untested simple fix for ignoring ignored methods is here:

http://github.com/dharkness/php-code-coverage/commit/eb8aef3d3687594935064fae2937973c68f54ad5

[dcopeland]

I applied this diff to 3.5.14 and it works as intended - ignored methods are properly ignored.

I do think it might be better to treat ignored methods as covered just for consistency with the HTML report.

[dcopeland]

To be perfectly clear, I think both the Clover and the HTML coverage report should ignore ignored methods, not show them as covered. But that's beyond the scope of this fix and I think the two reports should match.

[sebastianbergmann]

I won't work on PHP_CodeCoverage 1.0.X anymore as the source has diverged too much from master and PHPUnit 3.6 and PHP_CodeCoverage 1.1 will be released soon.

Please test whether this issue still persists in PHP_CodeCoverage 1.1 (master) and open a new ticket if it does. Thanks!