From 3cf9544e857981013bdc27e4c26ff754fb704554 Mon Sep 17 00:00:00 2001 From: Klaus Heigren Date: Tue, 19 Oct 2021 13:12:52 +0200 Subject: [PATCH 1/2] Use stdout stream for reports when available Use stdout stream for reports when available. This avoids "Headers already sent" types of issues, when manipulating a session during testing. --- src/Runner.php | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/Runner.php b/src/Runner.php index db78a1a..7fe8047 100644 --- a/src/Runner.php +++ b/src/Runner.php @@ -139,7 +139,10 @@ protected function applyReporters(\PHPUnit\Framework\TestResult $result, array $ } if ($arguments['report']) { - self::$persistentListeners[] = $this->instantiateReporter('report'); + self::$persistentListeners[] = $this->instantiateReporter( + 'report', + [@fopen(STDOUT, 'w') ?: fopen('php://output', 'w')] + ); } if ($arguments['html']) { From c675cb693ec49211d8cf681b503432f053940f2e Mon Sep 17 00:00:00 2001 From: Klaus Heigren Date: Wed, 20 Oct 2021 14:12:06 +0200 Subject: [PATCH 2/2] Fixed wrong use of the STDOUT constant The constant is defined as fopen('php://stdout', 'w'), not php://stdout. --- src/Runner.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/Runner.php b/src/Runner.php index 7fe8047..f4cece6 100644 --- a/src/Runner.php +++ b/src/Runner.php @@ -141,7 +141,7 @@ protected function applyReporters(\PHPUnit\Framework\TestResult $result, array $ if ($arguments['report']) { self::$persistentListeners[] = $this->instantiateReporter( 'report', - [@fopen(STDOUT, 'w') ?: fopen('php://output', 'w')] + [@STDOUT ?: fopen('php://output', 'w')] ); }