Skip to content

Commit f801fe7

Browse files
test: fix setMiddlewareExpectations method to match implementation
Wasn't a problem before that it was null here because getStatusHeader() lacked an explicit return type. Signed-off-by: Josh <[email protected]>
1 parent 590620d commit f801fe7

File tree

1 file changed

+7
-3
lines changed

1 file changed

+7
-3
lines changed

tests/lib/AppFramework/Http/DispatcherTest.php

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -148,9 +148,13 @@ protected function setUp(): void {
148148
* @param string $out
149149
* @param string $httpHeaders
150150
*/
151-
private function setMiddlewareExpectations($out = null,
152-
$httpHeaders = null, $responseHeaders = [],
153-
$ex = false, $catchEx = true) {
151+
private function setMiddlewareExpectations(
152+
$out = null,
153+
$httpHeaders = '',
154+
$responseHeaders = [],
155+
$ex = false,
156+
$catchEx = true
157+
) {
154158
if ($ex) {
155159
$exception = new \Exception();
156160
$this->middlewareDispatcher->expects($this->once())

0 commit comments

Comments
 (0)