Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork5.3k
Doc about new way of running tests#5654
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Changes fromall commits
File filter
Filter by extension
Conversations
Uh oh!
There was an error while loading.Please reload this page.
Jump to
Uh oh!
There was an error while loading.Please reload this page.
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -16,14 +16,13 @@ In any case, it's a good practice to run tests locally before submitting a | ||
| Before Running the Tests | ||
| ------------------------ | ||
| To run the Symfony test suite, install the external dependencies used during the | ||
| tests, such as Doctrine, Twig and Monolog. To do so, | ||
| :doc:`install Composer </cookbook/composer>` and execute the following: | ||
| .. code-block:: bash | ||
| $ composerupdate | ||
Member There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others.Learn more. Why do we need Member There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others.Learn more. but have the same result, as the project doesn't have a lock file MemberAuthor There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others.Learn more. When one works on a patch and a composer file is changed, only update works, that's why it's more generic | ||
| .. _running: | ||
| @@ -35,7 +34,7 @@ command: | ||
| .. code-block:: bash | ||
| $php ./phpunit symfony | ||
Contributor There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others.Learn more. is this correct? my binary is located under MemberAuthor There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others.Learn more. yes it is: we don't want your phpunit, we want symfony's one (see linked PR) Contributor There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others.Learn more. 👍 | ||
| The output should display ``OK``. If not, read the reported errors to figure out | ||
| what's going on and if the tests are broken because of the new code. | ||
| @@ -48,6 +47,12 @@ what's going on and if the tests are broken because of the new code. | ||
| .. code-block:: bash | ||
| $php ./phpunit src/Symfony/Component/Finder/ | ||
| .. tip:: | ||
| On Windows, install the `ConEmu`_ or `ANSICON`_ free applications to see | ||
| colored test results. | ||
| .. _ConEmu: https://code.google.com/p/conemu-maximus5/ | ||
| .. _ANSICON: https://github.com/adoxa/ansicon/releases | ||