Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Expressive compatibility #2

Closed
michalbundyra opened this issue Mar 22, 2018 · 1 comment
Closed

Expressive compatibility #2

michalbundyra opened this issue Mar 22, 2018 · 1 comment

Comments

@michalbundyra
Copy link
Contributor

Hello!

Just letting you know that we recently published container configuration documentation:
https://docs.zendframework.com/zend-expressive/v3/features/container/config/
We are working on library which covers all these cases and deliver common tests to check if the container is compatible and behave as expected, please see http://github.com/weierophinney/zend-container-config-test and the latest PR with tests updates:
https://github.com/weierophinney/zend-container-config-test/pull/4
We have already prepared fixed for Aura.Di and Pimple containers:

Would be nice if also this library is updated to use common tests. Please bare in mind it's still dev version, and the structure can be changes, but I think at this point we have quite good coverage of the desired behaviour.

@jsoumelidis
Copy link
Owner

Hello,
I have updated the code to use the tests provided by zend-container-config-test.
The only thing to notice is the lack of support for closures and objects as factories/delegators.
Symfony DI Component has not been designed to handle those cases as it's ability to compile (optimize) the configuration and cache the compiled container is a core feature (closures and objects cannot be cached).
Nevertheless, I found a workaround to exclude unsupported data sets from the test suite.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants