magento-engcom / php-7.2-support

Repository for implementing support of php-7.2 support in Magento
Open Software License 3.0
23 stars 25 forks source link

Upgrade phpunit to 7.X #125

Open mikeymike opened 6 years ago

mikeymike commented 6 years ago

Description

Upgrade PHPUnit to 7

Currently only resolved failing unit tests and I'm fully expecting functional / integration tests to have some issues too but thought I'd start with this and check the tests in Travis

Fixed Issues (if relevant)

https://github.com/magento-engcom/zf1-php-7.2-support/issues/6

Manual testing scenarios

phpunit -c dev/tests/unit/phpunit.xml.dist

Contribution checklist

buskamuza commented 6 years ago

Thanks for submitting the PR! I see that some tests failed on Travis and there is one small question in the PR. Also, I changed the target branch to 2.3-develop as all the work from libs-upgrade has just been merged to 2.3-develop recently and we'll stop the work in libs-upgrade.

mikeymike commented 6 years ago

@buskamuza thanks for the review, I'll take a look at the failing tests

mikeymike commented 6 years ago

@buskamuza I'm not really sure why the last run is failing, potentially some edge case of some kind... would probably pass if rerun 🤔

Anyway I just read on slack that PHPUnit upgrade has already been completed by someone else so I guess this might have been a waste of time... I'll leave it for you to handle as needed 👍