phpspec / prophecy

Highly opinionated mocking framework for PHP 5.3+
MIT License
8.53k stars 241 forks source link

Add disable spy behavior #509

Open Jean85 opened 3 years ago

Jean85 commented 3 years ago

Fix #508

This PR is incomplete, I don't know how to fix the new specs.

ciaranmcnulty commented 3 years ago

How compatible is this with #518?

Jean85 commented 3 years ago

@ciaranmcnulty I've already commented there: https://github.com/phpspec/prophecy/pull/518#issuecomment-801091103

We're battling the same issue in opposite directions: Prophecy's behavior is unclear when the subject of the prophecy has return types declared on its methods. We should discuss how to proceed, probably in #472