[BrowserKit] [HttpKernel] StreamedResponse is not working with BrowserKit after Symfony is upgraded to 6.1

[HtunHtunHtet]

Symfony version(s) affected

6.1

Description

Symfony\Component\HttpFoundation\StreamedResponse is not working in tests (BrowserKit) after Symfony is upgraded to 6.1 - the output is not rendered.

How to reproduce

When asserting that following StreamedResponse using BrowserKit produces 'test' output:

return new StreamedResponse(static function (): void {
    echo 'test';
});

the test is passing on Symfony 6.0 (see reproducer app: https://github.com/HtunHtunHtet/bug_app/tree/main) but starts to fail after upgrading to 6.1-RC1 (see: https://github.com/HtunHtunHtet/bug_app/tree/symfony-6.1) as the output is empty:

Below information is the output of the PHPUnit test.

Time: 00:00.150, Memory: 12.00 MB

There was 1 failure:

1) App\Tests\DefaultControllerTest::testSomething
Failed asserting that two strings are equal.
--- Expected
+++ Actual
@@ @@
-'test'
+''

Only the tests are affected as navigating to the controller page returns a correct response.

Possible Solution

No response

Additional Context

It might be related to recent StreamedResponseListener deprecation: #45476

[minorbug]

Thank you @HtunHtunHtet for creating this bug report! This indeed looks like a bug. I reproduced the bug in the "symfony-6.1" branch of https://github.com/HtunHtunHtet/bug_app.

Status: Reviewed

[m-idler]

I think i got a similar problem regarding a test for a StreamedResponse:

working test with symfony 6.0

ob_start();
$client->request(Request::METHOD_GET, 'myTestUrl');
$content = \ob_get_clean();
self::assertEquals('foo', $content);

What (know) works for me with symfony 6.1:

$client->request(Request::METHOD_GET, 'myTestUrl');
$content = $client->getInternalResponse()->getContent();
self::assertEquals('foo', $content);

[DaDeather]

If it's currently still unclear why this is happening:
#45476 (comment)

[carsonbot]

Hey, thanks for your report!
There has not been a lot of activity here for a while. Is this bug still relevant? Have you managed to find a workaround?

[carsonbot]

Could I get an answer? If I do not hear anything I will assume this issue is resolved or abandoned. Please get back to me <3

[IonBazan]

@carsonbot, the issue is still relevant. See also linked issue

[miladrahimi]

I still see the issue in Symfony 6.3.
And hopefully the $client->getInternalResponse()->getContent() solution which @m-idler mentioned still works!

[nicolas-grekas]

Anyone up to give this a try?

[encreinformatique]

The $client->getInternalResponse()->getContent() worked for me.

        self::$client = self::loggedIn();
        self::$client->request('GET', '/export');
        $content = self::$client->getInternalResponse()->getContent();

        self::assertResponseIsSuccessful();
        self::assertResponseHasHeader('Content-Disposition', 'attachment; filename=users.csv');
        self::assertStringStartsWith('xxx;yyy', $content);

[carsonbot]

Hey, thanks for your report!
There has not been a lot of activity here for a while. Is this bug still relevant? Have you managed to find a workaround?

[carsonbot]

Friendly ping? Should this still be open? I will close if I don't hear anything.

[miladrahimi]

Is there any pull request to fix that bug? If not, I think it should be fixed before closing, the bug is easily reproducible.

[DaDeather]

Is there any pull request to fix that bug? If not, I think it should be fixed before closing, the bug is easily reproducible.

Someone probably should check if it's not already fixed by the revert here:
#51396

And the ticket here:
#46743

[H4wKs]

@DaDeather #51396 did not fix this specific issue, the test still fails.