Skip to content

Add missing status type 6 (warning)#458

Merged
sebastianbergmann merged 1 commit into
sebastianbergmann:masterfrom
theseer:add-missing-warning-status
Jul 26, 2016
Merged

Add missing status type 6 (warning)#458
sebastianbergmann merged 1 commit into
sebastianbergmann:masterfrom
theseer:add-missing-warning-status

Conversation

@theseer

@theseer theseer commented Jul 26, 2016

Copy link
Copy Markdown
Contributor

Implementing phpunit/#1802 added a new status type which somehow never made it into the XML Coverage Report.
That lead to the following error condition:

     [exec] PHP Notice:  Undefined offset: 6 in phar:///usr/bin/phpunit/php-code-coverage/Report/Xml/Tests.php on line 43
     [exec] PHP Stack trace:
     [exec] PHP   1. {main}() /usr/bin/phpunit:0
     [exec] PHP   2. PHPUnit_TextUI_Command::main() /usr/bin/phpunit:569
     [exec] PHP   3. PHPUnit_TextUI_Command->run() phar:///usr/bin/phpunit/phpunit/TextUI/Command.php:113
     [exec] PHP   4. PHPUnit_TextUI_TestRunner->doRun() phar:///usr/bin/phpunit/phpunit/TextUI/Command.php:162
     [exec] PHP   5. SebastianBergmann\CodeCoverage\Report\Xml\Facade->process() phar:///usr/bin/phpunit/phpunit/TextUI/TestRunner.php:583
     [exec] PHP   6. SebastianBergmann\CodeCoverage\Report\Xml\Facade->processTests() phar:///usr/bin/phpunit/php-code-coverage/Report/Xml/Facade.php:52
     [exec] PHP   7. SebastianBergmann\CodeCoverage\Report\Xml\Tests->addTest() phar:///usr/bin/phpunit/php-code-coverage/Report/Xml/Facade.php:210

This PR should fix that by adding the missing status type

@sebastianbergmann sebastianbergmann merged commit 970529b into sebastianbergmann:master Jul 26, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants