Skip to content

Set mock methods for ResultSet#15531

Merged
markstory merged 1 commit intomasterfrom
othercorey-patch-3
May 21, 2021
Merged

Set mock methods for ResultSet#15531
markstory merged 1 commit intomasterfrom
othercorey-patch-3

Conversation

@othercorey
Copy link
Copy Markdown
Contributor

@othercorey othercorey commented May 21, 2021 •
edited
Loading

This throws a TypeError in 8.1 since the methods array is null. I'm pretty sure this is the cause.

186) Cake\Test\TestCase\ORM\QueryTest::testCollectionProxy with data set #2 ('every', Closure Object (...), false)
TypeError: PHPUnit\Framework\MockObject\InvocationHandler::__construct(): Argument #1 ($configurableMethods) must be of type array, null given, called in /home/runner/work/cakephp/cakephp/vendor/phpunit/phpunit/src/Framework/MockObject/Api/Api.php on line 70

@othercorey othercorey added the php label May 21, 2021
@othercorey othercorey added this to the 4.2.7 milestone May 21, 2021
@othercorey othercorey marked this pull request as ready for review May 21, 2021 00:55
@markstory markstory merged commit 520aedd into master May 21, 2021
@markstory markstory deleted the othercorey-patch-3 branch May 21, 2021 02:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

php

Projects

None yet

Milestone

4.2.7

Development

Successfully merging this pull request may close these issues.

2 participants

@othercorey @markstory