Symfony 3.3: Upgrade, Autowiring & Autoconfigure

Hey Nikolic,

Your import statement looks valid except the important fact that this statement should be first in the file, try to move it to the top of file, so first you have import and then you have services declaration. But yes, it's totally doable to split services in different files like you said.

P.S. Oh, and do not forget that "_defaults" option applies to the current file only, so you'll need to have its own _defaults in "listener_service.yml". Well, if you needed of course :)

Cheers!

Hey Fox C.

Haha, definitely you must throw your laptop into a swamp... Ok, being serious, I think you only have to exclude the "WebTestCase" class from being registered as a service.

services:
    Twencha\Bundle\EventRegistrationBundle\:
        ...
        exclude: '../../src/Twencha/Bundle/EventRegistrationBundle/{entity,repository, WebTestCase.php}' # Probably you will have to update this to point to where "WebTestCase.php" actually lives

Cheers!

I tried to debug it for a while, and here is the difference with and without the autowiring:
docroot/vendor/symfony/symfony/src/Symfony/Component/HttpKernel/Kernel.php:541

if (!$cache->isFresh()) {

with autowiring, the test result is true, therefore cache is rebuilt. without autowiring it doesn't go in the if statement.
I've debug a little more to see what changed the result, and it seems that it comes from here:
vendor/symfony/symfony/src/Symfony/Component/Config/ResourceCheckerConfigCache.php:113

if ($checker->isFresh($resource, $time)) {

and when I print the $resource, here is what I have:

object(Symfony\Component\Config\Resource\ClassExistenceResource)#14539 (2) {
  ["resource":"Symfony\Component\Config\Resource\ClassExistenceResource":private]=>
  string(101) "EntityManager59564fd03a693_546a8d27f194334ee012bfe64f629947b07e4919\__CG__\Doctrine\ORM\EntityManager"
  ["existsStatus":"Symfony\Component\Config\Resource\ClassExistenceResource":private]=>
  int(0)
}

I was not able to understand more from here.. or to change it..

Hey Igor Weigel

We are investigating your situation, it's really interesting :)
Can you show us how look's like your test code ? the whole file if possible

Cheers!

Hi Igor!

Apologies for my slow reply! Ok, this is very interesting, and you've done some great debugging! Your ClassExistenceResource dump is really interesting: something is actually looking for the EntityManager as a proxy class. I don't see that in my config (I tried on this project). So, I have a few questions:

A) If you run ./bin/console debug:container doctrine.orm.default_entity_manager, what is the class? Is it Doctrine\ORM\EntityManager or is it this long EntityManager...__CG__... proxy class?

B) Do you have any special bundles installed that might be adding any other magic. Like JMSDIExtraBundle? I'm trying to figure out what causes the EntityManager to become proxied like this. I expect the proxy is at the root of the problem, but I can't repeat it.

Cheers!

Hi Ryan,

A) I do have the long proxy class: EntityManager597857be7d816_546a8d27f194334ee012bfe64f629947b07e4919\__CG__\Doctrine\ORM\EntityManager

B) I do have JMSDIExtraBundle

Thanks

Hi Igor!

Ah, then I think we have the cause! It looks like JMSDIExtraBundle creates a proxy for the entity manager. And - I believe because that proxy is just saved to a file in the cache directory - this class_exists() check fails and the entire container rebuilds... constantly. To know if this is the issue, add this config to config.yml

jms_di_extra:
    doctrine_integration: false

This will remove the proxy. However, this also removes the ability to inject dependencies into your repository (http://jmsyst.com/bundles/JMSDiExtraBundle/master/doctrine). Try the config to see if it helps. I believe this is a bug in JMSDiExtraBundle - a bug with its compatibility with something new in Symfony 3.3 (but I'm not 100% sure - could be a subtle Symfony bug). If you're interested, I would open an issue on the JMSDiExtraBundle repo.

Let me know if it works!

I debugged a bit more and here is what happens in phpunit.
I have some @dataProvider for some of my tests, which are called in early stage. If I debug the Kernel $cache->isFresh() at this point (while in a @dataProvider), the cache is fresh (all good).
But once I'm in a test (so not in the @dataProvider anymore), $cache->isFresh() returns false, and the problems begin...^^"
But I have no idea why..

Hey Bruno Z.

That's a good question! I'm not sure if there is an official way to do it yet, but here at Knp we like to tackle this problem by specifying some "test" aliases that we configure on "config_test.yml" or "services_test.yml" (is up to you). It should look something like this:

// services_test.yml
services:
    test_alias.AppBundle\Path\To\Service: '@AppBundle\Path\To\Service'

Cheers!

ok, but with these declaration the service is not public automatically. right?
I've to set the service public globally in services.yml File.

Hey Bruno,

You can make this alias public:

services:
    test_alias.AppBundle\Path\To\Service:
        class: '@AppBundle\Path\To\Service'
        public: true

Or better make all the services inside services_test.yml file public with:

// services_test.yml
services:
    # default configuration for services in *this* file
    _defaults:
        autowire: true
        autoconfigure: true
        public: true

    test_alias.AppBundle\Path\To\Service: '@AppBundle\Path\To\Service'

Cheers!

Hey victor
Oh yes, clear! I go with you second solution. Thanks!

Hey Igor Weigel!

GREAT question - and it's something we need to document. The proper solution would be to create a public alias for the service you want to test in the test environment only:

# app/config/config_test.yml
services:
    test_alias.AppBundle\Service\MyService: ' @AppBundle\Service\MyService'

Then, you would actually fetch test_alias.AppBundle\Service\MyService out of the container in your unit test. It's a nice idea too: you only need to expose your service as public in the test environment.

Cheers!

Hey Mike,

You're welcome! That's really cool when you can ask questions to someone who can help you during the learning, I know that ;)
And congrats with finishing the course, well done!

Cheers!

Yea, congrats! You rock!