-
-
Notifications
You must be signed in to change notification settings - Fork 9.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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
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 | MIT 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 Commits ------- 32e5f67 [DependencyInjection] Fix travis unit tests c7befd5 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