-
Notifications
You must be signed in to change notification settings - Fork 30
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Internal error triggered by ProphesizeDynamicReturnTypeExtension::getTypeFromMethodCall with phpspec/prophecy 1.18 #315
Comments
Definitely a regression in 1.18. Here's a full stacktrace: phpstan/phpstan#10272 (comment) |
Since this appears to be a breaking change in phpspec/prophecy, is it worth opening an issue there? |
Hello @danepowell is there an issue opened in |
I've not had time to open an issue with phpspec/prophecy |
I think I've also hit this: phpstan/phpstan#10582 |
To be clear, I proposed opening an issue in phpspec/prophecy only to document (and to some degree, complain about 😄 ) the undocumented breaking change. But ultimately, I think this needs to be fixed via #316 |
Steps required to reproduce the problem
(new Prophet())->prophesize(Some::class)
Expected Result
Actual Result
Xdebug analysis shows that the difference before/after is in
ProphesizeDynamicReturnTypeExtension::getTypeFromMethodCall
line 70 where the$returnType
instanceof check used to be true but is not anymore:The text was updated successfully, but these errors were encountered: