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

Allow users to customise Symfony's ContainerBuilder #360

Merged
merged 2 commits into from
Oct 18, 2021

Conversation

lcobucci
Copy link
Owner

Under certain conditions, users might be interested in extending the container builder to modify its behaviour in the way that compiler passes aren't able to do.

Under certain conditions, users might be interested in extending the
container builder to modify its behaviour in the way that compiler
passes aren't able to do.
Depending on the order of the test suite execution we were having issues
as the compiled container for `CompilerTest` was using the same FQCN as
the `GeneratorTest`.

This makes sure we use different namespaces for the generated
containers, avoiding those conflicts.

It also plugs vfsStream to remove the file system dependency on
`CompilerTest`.

Running PHPUnit using the flag `--random-order-seed=1634593025` was
enough to reproduce the issue.
@lcobucci lcobucci added this to the 7.1.0 milestone Oct 18, 2021
@lcobucci lcobucci self-assigned this Oct 18, 2021
@lcobucci lcobucci merged commit 0fde70e into 7.1.x Oct 18, 2021
@lcobucci lcobucci deleted the allow-customisation-of-container-builder branch October 18, 2021 22:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant