Skip to content

[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

Merged
merged 2 commits into from
Apr 30, 2014

Conversation

romainneutron
Copy link
Contributor

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

@romainneutron romainneutron changed the title [2.3] Fix phpunit test suite on travis [2.3] Fix phpunit test suite on travis #2 Apr 30, 2014
@romainneutron
Copy link
Contributor Author

Forget what I said, Symfony\Component\Form\Tests\Extension\Core\EventListener\TrimListenerTest is still failing

This implementation does not seem to work, anyway

@stof
Copy link
Member

stof commented Apr 30, 2014

Reopening as waiting for the PHPUnit release was necessary for phpunit --self-update to give us a fixed version

@stof
Copy link
Member

stof commented Apr 30, 2014

arf, PHP 5.3.3 does not have openssl on Travis 😞

@stof
Copy link
Member

stof commented Apr 30, 2014

@romainneutron an alternative could be to download https://phar.phpunit.de/phpunit.phar with wget and replacing the existing phar with it, instead of asking PHPUnit to perform the download. This would fix the openssl issue on 5.3.3

@romainneutron
Copy link
Contributor Author

yep that's the only solution, I'm gonna give a try to this proposition

@fabpot fabpot merged commit 32e5f67 into symfony:2.3 Apr 30, 2014
fabpot added a commit that referenced this pull request Apr 30, 2014
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
@stof
Copy link
Member

stof commented May 1, 2014

@romainneutron this is actually not needed anymore (for the Travis update part), as Travis updated PHPUnit to 4.0.19

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants