Remove calls to TestCase::at()

It has been deprecated as of PHPUnit 9
See sebastianbergmann/phpunit#4297

Note that we still rely on the call order here, I am not the original
author of this test but they order looks like it is pretty important for
this test, or maybe at least for understand it.
Also note that we no longer check that save() is called after all calls
to fetch().

tests/Doctrine/Tests/Common/Annotations/CachedReaderTest.php

@@ -157,53 +157,39 @@ public function testAvoidCallingFilemtimeTooMuch() : void
         /* @var $cache Cache|MockObject */
         $cache = $this->createMock('Doctrine\Common\Cache\Cache');
 
-        // first pass => cache ok for method 1
-        // we load annotations AND filemtimes for this file
         $cache
-            ->expects($this->at(0))
-            ->method('fetch')
-            ->with($this->equalTo($cacheKeyMethod1))
-            ->will($this->returnValue([$route1])); // Result was cached, but there was an annotation;
-        $cache
-            ->expects($this->at(1))
-            ->method('fetch')
-            ->with($this->equalTo('[C]' . $cacheKeyMethod1))
-            ->will($this->returnValue($cacheTime));
-
-        // second pass => cache ok for method 2
-        // filemtime is seen as fresh even if it's not
-        $cache
-            ->expects($this->at(2))
-            ->method('fetch')
-            ->with($this->equalTo($cacheKeyMethod2))
-            ->will($this->returnValue([$route2])); // Result was cached, but there was an annotation;
-        $cache
-            ->expects($this->at(3))
+            ->expects($this->exactly(6))
             ->method('fetch')
-            ->with($this->equalTo('[C]' . $cacheKeyMethod2))
-            ->will($this->returnValue($cacheTime));
+            ->withConsecutive(
+                // first pass => cache ok for method 1
+                // we load annotations AND filemtimes for this file
+                [$this->equalTo($cacheKeyMethod1)],
+                // second pass => cache ok for method 2
+                // filemtime is seen as fresh even if it's not
+                [$this->equalTo('[C]' . $cacheKeyMethod1)],
+                // third pass => cache stale for method 2
+                // filemtime is seen as not fresh => we save
+                [$this->equalTo($cacheKeyMethod2)],
+                [$this->equalTo('[C]' . $cacheKeyMethod2)],
+                [$this->equalTo($cacheKeyMethod2)],
+                [$this->equalTo('[C]' . $cacheKeyMethod2)]
+            )
+            ->will($this->onConsecutiveCalls(
+                [$route1], // Result was cached, but there was an annotation;
+                $cacheTime,
+                [$route2], // Result was cached, but there was an annotation;
+                $cacheTime,
+                [$route2], // Result was cached, but there was an annotation;
+                $cacheTime
+            ));
 
-        // third pass => cache stale for method 2
-        // filemtime is seen as not fresh => we save
-        $cache
-            ->expects($this->at(4))
-            ->method('fetch')
-            ->with($this->equalTo($cacheKeyMethod2))
-            ->will($this->returnValue([$route2])); // Result was cached, but there was an annotation;
-        $cache
-            ->expects($this->at(5))
-            ->method('fetch')
-            ->with($this->equalTo('[C]' . $cacheKeyMethod2))
-            ->will($this->returnValue($cacheTime));
-        $cache
-            ->expects($this->at(6))
-            ->method('save')
-            ->with($this->equalTo($cacheKeyMethod2))
-        ;
         $cache
-            ->expects($this->at(7))
+            ->expects($this->exactly(2))
             ->method('save')
-            ->with($this->equalTo('[C]'.$cacheKeyMethod2))
+            ->withConsecutive(
+                [$this->equalTo($cacheKeyMethod2)],
+                [$this->equalTo('[C]'.$cacheKeyMethod2)]
+            )
         ;
 
         $reader = new CachedReader(new AnnotationReader(), $cache, true);