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
Make PHP 8 build green #1220
Make PHP 8 build green #1220
Conversation
assertContains() is no longer supposed to be used with strings.
9090cdf
to
392eda0
Compare
392eda0
to
16e0643
Compare
It is a code smell, but I am not sure it makes sense to introduce a public interface for this.
That version is compatible with PHP 8
7958030
to
0b4bd67
Compare
0b4bd67
to
4114425
Compare
@@ -140,7 +138,7 @@ public function testReset() : void | |||
$noProxyManager->expects($this->once()) | |||
->method('clear'); | |||
|
|||
$proxyManager = $this->getMockBuilder([LazyLoadingInterface::class, EntityManagerInterface::class])->getMock(); | |||
$proxyManager = $this->createMock(LazyLoadingEntityManagerInterface::class); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why did you create LazyLoadingEntityManagerInterface
? Isn't the previous multi-interface mock creation not working anymore in PHP 8?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It does not have to do with PHP 8, but with PHPUnit 9, which no longer supports it because it's a code smell IIRC.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Guess I have to remove another one of my old code snippets 😄
No description provided.