Merge pull request #40424 from nextcloud/fix/remove-at-matcher-in-dav-2

Remove last at matcher call in dav application tests

1 files changed, 6 insertions, 13 deletions

diff --git a/apps/dav/tests/unit/CalDAV/CalendarManagerTest.php b/apps/dav/tests/unit/CalDAV/CalendarManagerTest.php
index 8a559255a8c..c803afe6acf 100644
--- a/apps/dav/tests/unit/CalDAV/CalendarManagerTest.php
+++ b/apps/dav/tests/unit/CalDAV/CalendarManagerTest.php
@@ -36,7 +36,6 @@ use PHPUnit\Framework\MockObject\MockObject;
 use Psr\Log\LoggerInterface;
 
 class CalendarManagerTest extends \Test\TestCase {
-
 	/** @var CalDavBackend | MockObject */
 	private $backend;
 
@@ -77,22 +76,16 @@ class CalendarManagerTest extends \Test\TestCase {
 
 		/** @var IManager | MockObject $calendarManager */
 		$calendarManager = $this->createMock(Manager::class);
-		$calendarManager->expects($this->at(0))
-			->method('registerCalendar')
-			->willReturnCallback(function (): void {
-				$parameter = func_get_arg(0);
-				$this->assertInstanceOf(CalendarImpl::class, $parameter);
-				$this->assertEquals(123, $parameter->getKey());
-			});
-
-		$calendarManager->expects($this->at(1))
+		$registeredIds = [];
+		$calendarManager->expects($this->exactly(2))
 			->method('registerCalendar')
-			->willReturnCallback(function (): void {
-				$parameter = func_get_arg(0);
+			->willReturnCallback(function ($parameter) use (&$registeredIds): void {
 				$this->assertInstanceOf(CalendarImpl::class, $parameter);
-				$this->assertEquals(456, $parameter->getKey());
+				$registeredIds[] = $parameter->getKey();
 			});
 
 		$this->manager->setupCalendarProvider($calendarManager, 'user123');
+
+		$this->assertEquals(['123','456'], $registeredIds);
 	}
 }