DocBlock line in empty class not covered...

[andreasnij]

I don't know if I'm doing something wrong or doesn't get something, but this doesn't seem right to me:

This doesn't happen with earlier versions (PHPUnit 3.7 and Code Coverage 1.2). I've also discovered that if the first line starts with /** instead of /*, the class will not be in the code coverage report at all, which is better since then it doesn't affect my total code coverage % like it does now.

(Xdebug v2.2.4)

[avelicka]

Can confirm the same issue with xdebug v2.2.3, PHPUnit 4.0.14 and CodeCoverage 2.0.5.

[sebastianbergmann]

A valid docblock starts with /** not with /*.

[andreasnij]

Then I guess it's a comment and not a DocBlock. Should a comment really affect the code coverage? All the Symfony framework's classes looks like this.

[mvtp]

@sebastianbergmann this is not about DocBlock. This is about comment. Just the title is incorrect.

Please reopen this issue.

[sebastianbergmann]

Please supply a minimal, self-contained, reproducing test case.

[andreasnij]

Not sure what and how I can provide more information than in the original example. Put an empty class like the one in my example in an existing project with 100% test coverage, run tests and look at the cover report. If anyone else subscribing to this issue know how to provide a self-contained test case in this matter, feel free to contribute.

[mvtp]

E.g., this file is covered incorrectly:
https://github.com/sebastianbergmann/php-code-coverage/blob/master/tests/_files/source_with_oneline_annotations.php

See ignored lines:
https://github.com/sebastianbergmann/php-code-coverage/blob/master/tests/PHP/CodeCoverage/ParserTest.php#L92

Shouldn't lines 21 and 25 be ignored too?

[francoispluchino]

@sebastianbergmann, @jandreasn Same problem with PHPUnit 4.1.0 with PHP Code Coverage 2.0.6, PHP 5.5.12 and Xdebug 2.2.5.

@sebastianbergmann The Dockblock /* is used for include the license directly in file (see the example for Symfony2 Symfony\Component\Form\Exception\InvalidArgumentException).

Example for the empty class and empty interface that are not covered:

• Symfony\Component\Form\Exception\ExceptionInterface
• Symfony\Component\Form\Exception\InvalidArgumentException

[egeloen]

👍

[egeloen]

This issue should be fixed by #247.