Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[testing] tell about self::$container #9974

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

Merged
merged 1 commit into from
Jun 29, 2018

Conversation

nicolas-grekas
Copy link
Member

@nicolas-grekas nicolas-grekas commented Jun 28, 2018

testing.rst Outdated
.. tip::

The special container that gives access to private services exists only in
the ``test` environment and is itself a service that you can accessed from
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing a backpack here:

- ``test`
+ ``test``

testing.rst Outdated
.. tip::

If the information you need to check is available from the profiler, use
it instead.

.. versionadded:: 4.1
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you please move this directive before line 511?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

Copy link
Contributor

@HeahDude HeahDude left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@javiereguiluz javiereguiluz merged commit e64d89d into symfony:4.1 Jun 29, 2018
javiereguiluz added a commit that referenced this pull request Jun 29, 2018
This PR was merged into the 4.1 branch.

Discussion
----------

[testing] tell about self::$container

Fixes #9455.

Same as https://symfony.com/blog/new-in-symfony-4-1-simpler-service-testing

Commits
-------

e64d89d [testing] tell about self::$container
@javiereguiluz
Copy link
Member

Thanks Nicolas!

@nicolas-grekas nicolas-grekas deleted the test_contaienr branch July 1, 2018 20:44
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.

4 participants