Skip to content

Use simple-phpunit to run tests on travis-ci #178

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

Closed
wants to merge 1 commit into from
Closed

Use simple-phpunit to run tests on travis-ci #178

wants to merge 1 commit into from

Conversation

fbourigault
Copy link
Contributor

@fbourigault fbourigault commented Jun 27, 2017

Q A
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Related tickets partially #175
Documentation N/A
License MIT

What's in this PR?

This remove the phpunit/phpunit dev dependency and use symfony/phpunit-bridge simple-phpunit wrapper to run tests. PHPUnit installation is also cached.

I also moved global environment variables under the env.global .travis.yml section. Those variables are now hidden from the jobs list. See latest master build and current build for comparaison.

I also bumped matthiasnoback/symfony-dependency-injection-test minimum version to ^1.0.1 to avoid having phpunit/phpunit installed as dependency which caused some autoloading mess.

It looks like this was forgotten when doing #149.

@fbourigault fbourigault requested a review from Nyholm June 28, 2017 07:22
@Nyholm
Copy link
Member

Nyholm commented Jun 28, 2017

Thank you.
Tell me why simple-phpunit is better? What is the win of this PR?

@fbourigault
Copy link
Contributor Author

My bad. I get confused myself by depending on symfony/phpunit-bridge for deprecation helper and using simple-phpunit.

There is no added value for php-http/HttplugBundle at the moment.

@Nyholm
Copy link
Member

Nyholm commented Jun 28, 2017

Don't close it just because I do not understand. :/

@fbourigault
Copy link
Contributor Author

I didn't closed because you didn't understand. I thought there was an issue by using vendor/bin/phpunit directly but there is none. So using vendor/bin/simple-phpunit over vendor/bin/phpunit has 0 added value here.

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.

2 participants