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

Compatibility with recent PHPUnit (8) #10

Closed
wants to merge 1 commit into from

Conversation

DavidPrevot
Copy link

No description provided.

Copy link
Member

@derickr derickr left a comment

Choose a reason for hiding this comment

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

I missed this PR, sorry. But running this with PHPUnit 8(.5.8) gives the following warnings too:

$ ./vendor/bin/phpunit 
PHPUnit 8.5.8 by Sebastian Bergmann and contributors.

..........W....................................................  63 / 754 (  8%)
............................................................... 126 / 754 ( 16%)
.........................................................WWW..W 189 / 754 ( 25%)
WWW..W......................................W.................. 252 / 754 ( 33%)
............................................................... 315 / 754 ( 41%)
............................................................... 378 / 754 ( 50%)
..........W.................................................... 441 / 754 ( 58%)
............................................................... 504 / 754 ( 66%)
............................................................... 567 / 754 ( 75%)
............................................................... 630 / 754 ( 83%)
............................................................... 693 / 754 ( 91%)
.............................................................   754 / 754 (100%)

Time: 1.03 seconds, Memory: 12.00 MB

There were 11 warnings:

1) ezcConsoleArgumentsTest::testOffsetUnsetSuccess
assertAttributeEquals() is deprecated and will be removed in PHPUnit 9.
readAttribute() is deprecated and will be removed in PHPUnit 9.
getObjectAttribute() is deprecated and will be removed in PHPUnit 9.

2) ezcConsoleOptionTest::testAddDependency
assertAttributeEquals() is deprecated and will be removed in PHPUnit 9.
readAttribute() is deprecated and will be removed in PHPUnit 9.
getObjectAttribute() is deprecated and will be removed in PHPUnit 9.

3) ezcConsoleOptionTest::testRemoveDependency
assertAttributeEquals() is deprecated and will be removed in PHPUnit 9.
readAttribute() is deprecated and will be removed in PHPUnit 9.
getObjectAttribute() is deprecated and will be removed in PHPUnit 9.

4) ezcConsoleOptionTest::testRemoveAllDependencies
assertAttributeEquals() is deprecated and will be removed in PHPUnit 9.
readAttribute() is deprecated and will be removed in PHPUnit 9.
getObjectAttribute() is deprecated and will be removed in PHPUnit 9.

5) ezcConsoleOptionTest::testResetDependencies
assertAttributeEquals() is deprecated and will be removed in PHPUnit 9.
readAttribute() is deprecated and will be removed in PHPUnit 9.
getObjectAttribute() is deprecated and will be removed in PHPUnit 9.

6) ezcConsoleOptionTest::testAddExclusion
assertAttributeEquals() is deprecated and will be removed in PHPUnit 9.
readAttribute() is deprecated and will be removed in PHPUnit 9.
getObjectAttribute() is deprecated and will be removed in PHPUnit 9.

7) ezcConsoleOptionTest::testRemoveExclusion
assertAttributeEquals() is deprecated and will be removed in PHPUnit 9.
readAttribute() is deprecated and will be removed in PHPUnit 9.
getObjectAttribute() is deprecated and will be removed in PHPUnit 9.

8) ezcConsoleOptionTest::testRemoveAllExclusions
assertAttributeEquals() is deprecated and will be removed in PHPUnit 9.
readAttribute() is deprecated and will be removed in PHPUnit 9.
getObjectAttribute() is deprecated and will be removed in PHPUnit 9.

9) ezcConsoleOptionTest::testResetExclusions
assertAttributeEquals() is deprecated and will be removed in PHPUnit 9.
readAttribute() is deprecated and will be removed in PHPUnit 9.
getObjectAttribute() is deprecated and will be removed in PHPUnit 9.

10) ezcConsoleOutputFormatsTest::testIterator
readAttribute() is deprecated and will be removed in PHPUnit 9.
getObjectAttribute() is deprecated and will be removed in PHPUnit 9.

11) ezcConsoleStatusbarTest::testReset
readAttribute() is deprecated and will be removed in PHPUnit 9.
getObjectAttribute() is deprecated and will be removed in PHPUnit 9.

Would you have time to look at that? I also had to update composer.json to change the phpunit line to "~8.0"

@williamdes
Copy link

williamdes commented Oct 30, 2020

I also had to update composer.json to change the phpunit line to "~8.0"

I would say ^7 || ^8

Why: because your tests are run on Debian or other packaging systems that can still use phpunit 7

@derickr
Copy link
Member

derickr commented Jul 25, 2021

I've bumped the PHP Unit requirement to ~8.0 in a separate commit.

@derickr derickr closed this Jul 25, 2021
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.

None yet

3 participants