Fix usage of at() in unittests

Signed-off-by: Sean Molenaar <sean@seanmolenaar.eu>

1 files changed, 5 insertions, 14 deletions

diff --git a/tests/Unit/Controller/FeedControllerTest.php b/tests/Unit/Controller/FeedControllerTest.php
index 7498d0ccb..f5c51b93e 100644
--- a/tests/Unit/Controller/FeedControllerTest.php
+++ b/tests/Unit/Controller/FeedControllerTest.php
@@ -142,22 +142,13 @@ class FeedControllerTest extends TestCase
 
     private function activeInitMocks($id, $type)
     {
-        $this->settings->expects($this->at(0))
+        $this->settings->expects($this->exactly(2))
             ->method('getUserValue')
-            ->with(
-                $this->equalTo($this->user),
-                $this->equalTo($this->appName),
-                $this->equalTo('lastViewedFeedId')
-            )
-            ->will($this->returnValue($id));
-        $this->settings->expects($this->at(1))
-            ->method('getUserValue')
-            ->with(
-                $this->equalTo($this->user),
-                $this->equalTo($this->appName),
-                $this->equalTo('lastViewedFeedType')
+            ->withConsecutive(
+                [$this->user, $this->appName, 'lastViewedFeedId'],
+                [$this->user, $this->appName, 'lastViewedFeedType']
             )
-            ->will($this->returnValue($type));
+            ->willReturnOnConsecutiveCalls($id, $type);
     }