Skip to content

Fix exception class in InvocationMocker und MockBuilder strub#109

Merged
weirdan merged 1 commit intopsalm:masterfrom
weph:master
May 29, 2021
Merged

Fix exception class in InvocationMocker und MockBuilder strub#109
weirdan merged 1 commit intopsalm:masterfrom
weph:master

Conversation

@weph
Copy link
Copy Markdown
Contributor

@weph weph commented May 29, 2021

After enabling checkForThrowsDocblock psalm crashed with

Uncaught Exception: Could not get class storage for phpunit\framework\mockobject\builder\runtimeexception

I had a look at PHPUnit and there really isn't a RuntimeException in that namespaces and after browsing through some versions I think there never has been. So I guess what's really meant is the global one.

@weirdan
Copy link
Copy Markdown
Member

weirdan commented May 29, 2021

I fixed test failures on master, mind rebasing this? I can't push to your repo, either because you did not enable 'Updates from maintainers' checkbox or because the PR was created from weph:master and not from a dedicated branch.

@weph
Copy link
Copy Markdown
Contributor Author

weph commented May 29, 2021

I checked "Allow edits by maintainers" and according to the help text you should be allowed to push to my master 🤷🏻‍♂️
But anyway... I just rebased it

@weirdan weirdan merged commit 31d15bb into psalm:master May 29, 2021
@weirdan
Copy link
Copy Markdown
Member

weirdan commented May 29, 2021

Thanks!

@weirdan
Copy link
Copy Markdown
Member

weirdan commented May 29, 2021

Released in 0.15.2.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants