adjust to phpstan being more precise which line of chained calls triggers an error

Author: dbu

.gitignore

@@ -1,6 +1,7 @@
 vendor/
 composer.lock
 phpunit.xml
+phpstan.neon
 doc/_build/
 .php-cs-fixer.cache
 puli.json

tests/Functional/Symfony/EventDispatchingHttpCacheTest.php

@@ -44,10 +44,10 @@ public function testEventListeners(): void
             ->shouldReceive('handle')
             ->andReturn($expectedResponse)
             ->getMock();
-        // https://github.com/phpstan/phpstan-mockery/issues/8
-        /** @phpstan-ignore-next-line */
         $store = \Mockery::mock(StoreInterface::class)
             ->shouldReceive('lookup')->andReturn(null)->times(1)
+            // https://github.com/phpstan/phpstan-mockery/issues/8
+            /* @phpstan-ignore-next-line */
             ->shouldReceive('write')->times(1)
             ->shouldReceive('unlock')->times(1)
             // need to declare the cleanup function explicitly to avoid issue between register_shutdown_function and mockery

tests/Functional/Varnish/UserContextFailureTest.php

@@ -26,9 +26,7 @@ class UserContextFailureTest extends VarnishTestCase
     public function setUp(): void
     {
         // needs to be decided before doing the setup
-        // phpunit 9 calls the method getName(), phpunit 10 name()
-        $name = method_exists($this, 'name') ? $this->name() : $this->getName(); /* @phpstan-ignore-line */
-        $this->mode = 'testHashRequestFailure' === $name ? 'failure' : 'cache';
+        $this->mode = 'testHashRequestFailure' === $this->name() ? 'failure' : 'cache';
 
         parent::setUp();
     }

tests/Unit/CacheInvalidatorTest.php

@@ -78,10 +78,10 @@ public function testSupportsInvalid(): void
     public function testInvalidatePath(): void
     {
         /** @var MockInterface&PurgeCapable $purge */
-        // https://github.com/phpstan/phpstan-mockery/issues/8
-        /** @phpstan-ignore-next-line */
         $purge = \Mockery::mock(PurgeCapable::class)
             ->shouldReceive('purge')->once()->with('/my/route', [])
+            // https://github.com/phpstan/phpstan-mockery/issues/8
+            /* @phpstan-ignore-next-line */
             ->shouldReceive('purge')->once()->with('/my/route', ['X-Test-Header' => 'xyz'])
             ->shouldReceive('flush')->once()
             ->getMock();
@@ -99,10 +99,10 @@ public function testRefreshPath(): void
     {
         $headers = ['X' => 'Y'];
         /** @var MockInterface&RefreshCapable $refresh */
-        // https://github.com/phpstan/phpstan-mockery/issues/8
-        /** @phpstan-ignore-next-line */
         $refresh = \Mockery::mock(RefreshCapable::class)
             ->shouldReceive('refresh')->once()->with('/my/route', $headers)
+            // https://github.com/phpstan/phpstan-mockery/issues/8
+            /* @phpstan-ignore-next-line */
             ->shouldReceive('flush')->never()
             ->getMock();
 
@@ -194,10 +194,10 @@ public function testProxyClientExceptionsAreLogged(): void
 
         $unreachableException = ProxyUnreachableException::proxyUnreachable($clientException);
 
-        // https://github.com/phpstan/phpstan-mockery/issues/8
-        /** @phpstan-ignore-next-line */
         $response = \Mockery::mock(ResponseInterface::class)
             ->shouldReceive('getStatusCode')->andReturn(403)
+            // https://github.com/phpstan/phpstan-mockery/issues/8
+            /* @phpstan-ignore-next-line */
             ->shouldReceive('getReasonPhrase')->andReturn('Forbidden')
             ->getMock();
         $responseException = ProxyResponseException::proxyResponse(new HttpException('test', $failedRequest, $response));
@@ -212,15 +212,15 @@ public function testProxyClientExceptionsAreLogged(): void
 
         $cacheInvalidator = new CacheInvalidator($proxyClient);
 
-        // https://github.com/phpstan/phpstan-mockery/issues/8
-        /** @phpstan-ignore-next-line */
         $logger = \Mockery::mock(LoggerInterface::class)
             ->shouldReceive('log')->once()
             ->with(
                 'critical',
                 'Request to caching proxy at 127.0.0.1 failed with message "Couldn\'t connect to host"',
                 ['exception' => $unreachableException]
             )
+            // https://github.com/phpstan/phpstan-mockery/issues/8
+            /* @phpstan-ignore-next-line */
             ->shouldReceive('log')->once()
             ->with(
                 'critical',

tests/Unit/ResponseTaggerTest.php

@@ -48,13 +48,13 @@ public function testGetTagsHeaderValue(): void
 
     public function testTagResponseReplace(): void
     {
-        // https://github.com/phpstan/phpstan-mockery/issues/8
-        /** @phpstan-ignore-next-line */
         $headerFormatter = \Mockery::mock(TagHeaderFormatter::class)
             ->shouldReceive('getTagsHeaderValue')
             ->with(['tag-1', 'tag-2'])
             ->once()
             ->andReturn('tag-1,tag-2')
+            // https://github.com/phpstan/phpstan-mockery/issues/8
+            /* @phpstan-ignore-next-line */
             ->shouldReceive('getTagsHeaderName')
             ->once()
             ->andReturn('FOS-Tags')
@@ -75,13 +75,13 @@ public function testTagResponseReplace(): void
 
     public function testTagResponseAdd(): void
     {
-        // https://github.com/phpstan/phpstan-mockery/issues/8
-        /** @phpstan-ignore-next-line */
         $headerFormatter = \Mockery::mock(TagHeaderFormatter::class)
             ->shouldReceive('getTagsHeaderValue')
             ->with(['tag-1', 'tag-2'])
             ->once()
             ->andReturn('tag-1,tag-2')
+            // https://github.com/phpstan/phpstan-mockery/issues/8
+            /* @phpstan-ignore-next-line */
             ->shouldReceive('getTagsHeaderName')
             ->once()
             ->andReturn('FOS-Tags')
@@ -109,10 +109,10 @@ public function testTagResponseNoTags(): void
 
         $tagger = new ResponseTagger(['header_formatter' => $headerFormatter]);
 
-        // https://github.com/phpstan/phpstan-mockery/issues/8
-        /** @phpstan-ignore-next-line */
         $response = \Mockery::mock(ResponseInterface::class)
             ->shouldReceive('withHeader')->never()
+            // https://github.com/phpstan/phpstan-mockery/issues/8
+            /* @phpstan-ignore-next-line */
             ->shouldReceive('withAddedHeader')->never()
             ->getMock();
 

tests/Unit/Test/PHPUnit/IsCacheHitConstraintTestCase.php

@@ -26,10 +26,10 @@ public function setUp(): void
 
     public function testMatches(): void
     {
-        // https://github.com/phpstan/phpstan-mockery/issues/8
-        /** @phpstan-ignore-next-line */
         $response = $this->getResponseMock()
             ->shouldReceive('hasHeader')->with('cache-header')->andReturn(true)
+            // https://github.com/phpstan/phpstan-mockery/issues/8
+            /* @phpstan-ignore-next-line */
             ->shouldReceive('getHeaderLine')->with('cache-header')->once()->andReturn('MISS')
             ->shouldReceive('getStatusCode')->andReturn(500)
             ->shouldReceive('getHeaders')->andReturn([])
@@ -46,10 +46,10 @@ public function testMatchesThrowsExceptionIfHeaderIsMissing(): void
         $this->expectException(\RuntimeException::class);
         $this->expectExceptionMessage('Response has no "cache-header" header');
 
-        // https://github.com/phpstan/phpstan-mockery/issues/8
-        /** @phpstan-ignore-next-line */
         $response = $this->getResponseMock()
             ->shouldReceive('hasHeader')->with('cache-header')->once()->andReturn(false)
+            // https://github.com/phpstan/phpstan-mockery/issues/8
+            /* @phpstan-ignore-next-line */
             ->shouldReceive('getStatusCode')->andReturn(200)
             ->shouldReceive('getHeaders')->andReturn([])
             ->shouldReceive('getBody')->andReturn(new Stream(fopen('php://temp', 'r+')))

tests/Unit/Test/PHPUnit/IsCacheMissConstraintTestCase.php

@@ -25,10 +25,10 @@ public function setUp(): void
 
     public function testMatches(): void
     {
-        // https://github.com/phpstan/phpstan-mockery/issues/8
-        /** @phpstan-ignore-next-line */
         $response = $this->getResponseMock()
             ->shouldReceive('hasHeader')->with('cache-header')->andReturn(true)
+            // https://github.com/phpstan/phpstan-mockery/issues/8
+            /* @phpstan-ignore-next-line */
             ->shouldReceive('getHeaderLine')->with('cache-header')->once()->andReturn('HIT')
             ->shouldReceive('getStatusCode')->andReturn(200)
             ->getMock();