From 6f46e939a6000bcad01b8266eb1452697f67fb47 Mon Sep 17 00:00:00 2001 From: Marco Pivetta Date: Wed, 5 Jun 2019 18:56:20 +0200 Subject: [PATCH] Reverted BC brak: `MockObject#method()` now correctly detected by Psalm Ref: https://github.com/sebastianbergmann/phpunit/pull/3708#discussion_r287614080 Ref: https://github.com/vimeo/psalm/issues/1692 --- src/Framework/MockObject/MockObject.php | 4 ---- 1 file changed, 4 deletions(-) diff --git a/src/Framework/MockObject/MockObject.php b/src/Framework/MockObject/MockObject.php index 0a8f30f7162..2df6810753e 100644 --- a/src/Framework/MockObject/MockObject.php +++ b/src/Framework/MockObject/MockObject.php @@ -28,8 +28,4 @@ public function __phpunit_hasMatchers(): bool; public function __phpunit_setReturnValueGeneration(bool $returnValueGeneration): void; public function expects(Invocation $matcher): BuilderInvocationMocker; - - // @TODO get rid of this - required by psalm-only right now - bug in parsing @method? - // @TODO this addition is a BC break and needs to be reverted before merge. - public function method($constraint) : BuilderInvocationMocker; }