Check for unexecuted covered or used code is too strict

[sebastianbergmann]

The implementation of #234 in 168eaf4 is too strict. When @covers Foo::bar is used then the current implementation expects all lines of code in Foo::bar() to be executed when instead it should only expect one line of code in Foo::bar() to be executed.

[sebastianbergmann]

I started to work on this in 1cedd8b. This should work unless the sourcecode file of the code unit to be covered or used has not been loaded at all. In that case https://github.com/sebastianbergmann/code-unit-reverse-lookup does not yield a useful result.

All things considered, this approach is rather stupid. PHPUnit turns code unit information (@covers class::method) into filename and line numbers information which PHP_CodeCoverage then tries to convert back into code unit information.

It might be best for PHPUnit to pass lists of code units (from @covers and @uses annotations) to PHP_CodeCoverage and consistently use that information inside PHP_CodeCoverage.

[sebastianbergmann]

@belanur We missed the opportunity of the PHPUnit Code Sprint to discuss this. @spriebsch just reminded me that this still does not work. Will try to look into it but any help appreciated.

[sebastianbergmann]

Updated the initial work done in March 2016 to the state of master in April 2017 in 88653cb.

I will probably do what I thought about in #420 (comment), though.