csarrazi / CsaGuzzleBundle

A bundle integrating Guzzle >=4.0 in Symfony
250 stars 77 forks source link

Symfony 3.4 / 4.0 compatibility #211

Closed skobkin closed 7 years ago

skobkin commented 7 years ago

When I'm running PHPUnit tests in my CI it fails with current stable csa/guzzle-bundle 2.2.1 and Symfony 3.4.x-dev (symfony/symfony@e973c247c8) I'm getting an error:

PHPUnit 5.7.23 by Sebastian Bergmann and contributors.

......................................                            38 / 38 (100%)

Time: 16.65 seconds, Memory: 40.00MB

OK (38 tests, 86 assertions)

Generating code coverage report in Clover XML format ... done

Remaining deprecation notices (16)

Implementing "Symfony\Component\HttpKernel\DataCollector\DataCollectorInterface" without the "reset()" method is deprecated since version 3.4 and will be unsupported in 4.0 for class "Csa\Bundle\GuzzleBundle\DataCollector\GuzzleCollector": 16x
4x in ApplicationAvailabilityFunctionalTest::testPageIsSuccessful from Tests\Skobkin\PointToolsBundle
2x in MainControllerTest::testUserSearch from Tests\Skobkin\PointToolsBundle\Controller
2x in MainControllerTest::testNonExistingUserSearch from Tests\Skobkin\PointToolsBundle\Controller
1x in EventsControllerTest::testHasEvents from Tests\Skobkin\PointToolsBundle\Controller
1x in MainControllerTest::testUserStats from Tests\Skobkin\PointToolsBundle\Controller
1x in MainControllerTest::testAjaxUserAutoComplete from Tests\Skobkin\PointToolsBundle\Controller
1x in MainControllerTest::testAjaxUserAutoCompleteForNonExistingUser from Tests\Skobkin\PointToolsBundle\Controller
1x in PostControllerTest::testNonExistingPostPage from Tests\Skobkin\PointToolsBundle\Controller
1x in PostControllerTest::testShortPostPageIsOk from Tests\Skobkin\PointToolsBundle\Controller
1x in UserControllerTest::testTestuserPageHasHeading from Tests\Skobkin\PointToolsBundle\Controller
1x in UserControllerTest::testTestuserHasSubscribers from Tests\Skobkin\PointToolsBundle\Controller

I think this is not a problem of my tests (ApplicationAvailabilityFunctionalTest::testPageIsSuccessful for example).

Can it be fixed in stable release of this major branch?

csarrazi commented 7 years ago

Did you test using dev-master or 3.0.0@dev?

This won't be fixed in branch 2.x. This will only be fixed for version 3.0 of the bundle, which will be released around December.

csarrazi commented 7 years ago

FYI, see a874b4a

skobkin commented 7 years ago

As I said earlier I was using 3.4.x-dev.

FYI, see a874b4a

Nice! Will it be in the current stable branch of csa/guzzle-bundle (^2)?

UPD:

This won't be fixed in branch 2.x. This will only be fixed for version 3.0 of the bundle, which will be released around December.

Uh sorry. I thought that you're talking about symfony/symfony. I'm using ^2 branch of csa/guzzle-bundle.

Thank you.

csarrazi commented 7 years ago

Don't worry, you'll be able to use version 3.0 without any major changes. Currently, work is being done to move the middleware out of the bundle, and split them in multiple third-party libraries, in order to facilitate reuse without the bundle.

luispabon commented 6 years ago

May I ask how this work is going? We're currently migrating projects to symfony 4 and this is so far the last dependency holding us back.

cordoval commented 6 years ago

guyz be patient, it is coming today probably!

csarrazi commented 6 years ago

Your wishes have been answered, @luispabon @cordoval @skobkin!

cordoval commented 6 years ago

told ya

luispabon commented 6 years ago

yay thank you :+1: first proj migrated with zarro boogs