Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork9.6k
[2.3] Fix phpunit test suite on travis #2#10826
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.
Conversation
Forget what I said, This implementation does not seem to work, anyway |
Reopening as waiting for the PHPUnit release was necessary for |
arf, PHP 5.3.3 does not have openssl on Travis 😞 |
@romainneutron an alternative could be to download |
yep that's the only solution, I'm gonna give a try to this proposition |
This PR was merged into the 2.3 branch.Discussion----------[2.3] Fix phpunit test suite on travis#2| Q | A| ------------- | ---| Bug fix? | yes| New feature? | no| BC breaks? | no| Deprecations? | no| Tests pass? | yes| Fixed tickets |#10821| License | MITThis is a second approach to solve#10821.Something is weird: `Symfony\Component\Form\Tests\Extension\Core\EventListener\TrimListenerTest` passes when using PHPUnit as phar and fails when using a composer global installCommits-------32e5f67 [DependencyInjection] Fix travis unit testsc7befd5 Update PHPUnit before run
@romainneutron this is actually not needed anymore (for the Travis update part), as Travis updated PHPUnit to 4.0.19 |
This is a second approach to solve#10821.
Something is weird:
Symfony\Component\Form\Tests\Extension\Core\EventListener\TrimListenerTest
passes when using PHPUnit as phar and fails when using a composer global install