Skip to content
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

[Travis] Fix tests, move to newer version of phpunit #30

Merged
merged 8 commits into from Mar 1, 2019
Merged

Conversation

andrerom
Copy link
Contributor

@andrerom andrerom commented Mar 1, 2019

Some cleanup here too in order to align with other repos.

@andrerom andrerom requested review from glye and alongosz March 1, 2019 13:40
@@ -10,7 +10,7 @@

use EzSystems\EzSupportToolsBundle\SystemInfo\Collector\ConfigurationSymfonyKernelSystemInfoCollector;
use EzSystems\EzSupportToolsBundle\SystemInfo\Value\SymfonyKernelSystemInfo;
use PHPUnit_Framework_TestCase;
use PHPUnit\Framework\TestCase as PHPUnit_Framework_TestCase;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wouldn't it be simpler to change extends PHPUnit_Framework_TestCase into extends TestCase? Just one line more ;)

Copy link
Contributor Author

@andrerom andrerom Mar 1, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sure nitty ;)

Copy link
Contributor Author

@andrerom andrerom Mar 1, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed in e63eadc
Also bumped php-cs-fixer in 8438f57 so runs on PHP 7.3 works

so hopefully should be green now

.travis.yml Outdated Show resolved Hide resolved
@andrerom andrerom merged commit 7294f07 into master Mar 1, 2019
@andrerom andrerom deleted the fix_travis branch March 1, 2019 14:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants