Skip to content
This repository has been archived by the owner on Jan 29, 2020. It is now read-only.

Added PHPUnit 8 support #76

Merged
merged 7 commits into from
Jun 11, 2019
Merged

Added PHPUnit 8 support #76

merged 7 commits into from
Jun 11, 2019

Conversation

michalbundyra
Copy link
Member

No description provided.

Added traits with methods which can be used in tests and in PHPUnit 8 have
RTH defined. We load proper trait depends on used PHPUnit version.
In our classes to keep compatibility we use method with "Compat" suffix.
Fot the end user nothing is changed as long as the user decide to support
one PHPUnit version (any <7 or 8+). Depends on the version user can define
setUp/tearDown method compatible with the used version of PHPUnit and call
the parent method.
@weierophinney
Copy link
Member

@webimpress Please look at the travis build history and the jobs run for this PR. For the PHP 5/PHPUnit 4 combination, the memory consumption is a full 5x, and for the PHP 7.3/PHPUnit 7 combination, a full 3x. I'm not sure if it's a change in the build system in the past month, or if this patch introduces the slow down, but it either needs to be addressed or explained before we can merge.

@michalbundyra michalbundyra mentioned this pull request May 22, 2019
@michalbundyra
Copy link
Member Author

@weierophinney There was a bug in my implementation. Instead of using static::class I've used __CLASS__ in abstract class, so because of that it was not working properly. Now it's fixed and tests looks promising :)

Still failing with latest dependencies on PHP 7.2 and 7.3. I'm looking on it.

@snapshotpl
Copy link

@webimpress looks like ready to deploy, right?

@michalbundyra
Copy link
Member Author

@snapshotpl Yes, just someone from maintainers need to find time to release it. I've already pinged @weierophinney

@snapshotpl
Copy link

@weierophinney 🙏

@Xerkus Xerkus self-assigned this Jun 11, 2019
@Xerkus Xerkus merged commit a69c346 into zendframework:develop Jun 11, 2019
Xerkus added a commit that referenced this pull request Jun 11, 2019
Xerkus added a commit that referenced this pull request Jun 11, 2019
@michalbundyra michalbundyra deleted the feature/phpunit-8 branch June 11, 2019 18:12
Xerkus added a commit to Xerkus/zend-test that referenced this pull request Jun 15, 2019
Xerkus added a commit to Xerkus/zend-test that referenced this pull request Jun 18, 2019
Xerkus added a commit to Xerkus/zend-test that referenced this pull request Jun 18, 2019
Xerkus added a commit to Xerkus/zend-test that referenced this pull request Jun 20, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants