PHPunit coverage reports fail if the is out to the stdout or header

[pbearne]

trying to get the coverage report to work :-)

​https://phpunit.de/manual/current/en/textui.html

and their error due to the ech statements in the phpunit install.php and bootstrap.php

i.e.

<?php
echo "Installing..." . PHP_EOL;

and due to the header value been set in wp_redirect() in pluggable.php why this is running for unittest ???

it looks like we need to test for if a coverage report is been created and not echo to the console

[pbearne]

this might help here for a note ​http://stackoverflow.com/questions/6710418/how-to-display-underlying-tested-method-in-phpunit-tests

[swissspidy]

I noticed the same a while ago. It used to work, but perhaps something changed in PHPUnit.

@jorbin @netweb Maybe you guys have some ideas?

[pbearne]

It could that its the coverage reports.

Happy to provide patch for this but we will lose the current stdout if I do

[pbiron]

I think the underlying problem here is also preventing me from writing unit tests for export.

I'm trying to have something like the following in a unit test:

ob_start();

export_wp();

$wxr = ob_get_flush();
// $wxr should now contain the exported XML as a string

When export_wp() calls header(...) an exception is raised because "headers already sent".

Does anyone have a work around that I could use to get the output of an export?

[pbearne]

I think we need to add a flag so we don't output to stdout

or just remove them are they needed?

Paul

[schlessera]

Related: https://core.trac.wordpress.org/ticket/43067

[johnbillion]

@pbearne @swissspidy @pbiron Which version of PHPUnit are you using?

[pbearne]

@johnbillion I was doing this in VVV but it has been a while since I checked it was still broken

The current PHPUnit in vvv is 6.5.5

Hope this helps

Paul

[swissspidy]

@johnbillion I'm currently running PHPUnit 5.7.26.