Bug Report #4317
Coding standards do not accept @codeCoverageIgnore(Start|End) PHPUnit Tags
| Status: | Closed | Start date: | 10/28/2011 | |
|---|---|---|---|---|
| Priority: | Normal | Due date: | ||
| Assignee: | - | % Done: | 0% |
|
| Category: | PHPCS Coding Standards | |||
| Target version: | v3.2.1 | Estimated time: | 0.50 hour | |
| Resolution: | invalid | Points: | 1 |
Description
The PHPUnit in-source comment tags to exclude sections of code from code coverage analysis violate the current OneLineComment sniff as they do not include whitespace/begin with a capital letter.
History
Updated by Andrew Coulton 7 months ago
Pull (with unit test) at https://github.com/kohana/coding-standards/pull/7
Updated by Chris Bandy 6 months ago
- Status changed from New to Feedback
- Resolution set to invalid
afaik, these tags work and are allowed if you put a space between // and @.
Updated by Andrew Coulton 6 months ago
I did test that before working on this pull request, and it didn't - and it's not documented in PHPUnit that it should.
However a bit of digging just now turned up this https://github.com/sebastianbergmann/php-code-coverage/issues/32 so evidently // @ did work in 3.4 and does work now in 3.5 but didn't in some 3.5 versions in between.
Guess I need to update my pear packages.
Updated by Chris Bandy 6 months ago
- Status changed from Feedback to Closed