Improve AutowireTest to ignore TrustedCallbackInterface

here's test and fixes

The last submitted patch, 2: 3392396-2-fail.patch, failed testing. View results →

Should we ignore TrustedCallbackInterface when building the implementation array? If something implements that and one other unique interface it should probably also be considered for autowiring?

Good point! so test needs further improvement

https://symfony.com/blog/new-in-symfony-3-3-psr-4-based-service-discovery

When using a service definition prototype, if only one service is discovered that implements an interface, configuring the alias is not mandatory and Symfony will automatically create one.

So I extended test as no interface is registered automatically

📌 Enable more service autowiring by adding interface aliases to core modules Fixed should have got a CR, and the AutowireTest which it added needs much better failure messages, and docs, to explain how all this is meant to be done.

At the moment, adding a new service to core causes AutowireTest to fail and there's no clue at all of how to create services in a way that will satisfy it.

Should a follow up be opened for #8?

I've asked that 📌 Enable more service autowiring by adding interface aliases to core modules Fixed be reopened.

🐛 Improve the failure message from Drupal\KernelTests\Core\DependencyInjection\AutowireTest Active solves #8.

Committed and pushed 4cfa1b5ffb to 11.x and 6bdcb415f7 to 10.2.x. Thanks!

Automatically closed - issue fixed for 2 weeks with no activity.

andypost → closed merge request !4962