test: pass the ObjectServiceInterface the constructors now require

ADR-083 added a constructor parameter; the test constructions still passed the
old argument count:

    ArgumentCountError: Too few arguments to __construct(),
    N passed and exactly N+1 expected

Each site gains one argument BY NAME, which fills the right slot whether the
preceding arguments were written positionally or by name — so the same edit
works for both shapes, and a call short by more than this one parameter still
errors, correctly.

Every touched file is re-parsed with php -l and reverted on failure, and a
re-scan reports 0 remaining sites in each app.

Author: rubenvdlinde

tests/Unit/Controller/ContactpersonenControllerOrganisationScopeTest.php

@@ -118,7 +118,8 @@ protected function setUp(): void {
 			$this->userSession,
 			$this->container,
 			$this->createMock(ISecureRandom::class),
-			$this->logger
+			$this->logger,
+			objectService: $this->createMock(ObjectServiceInterface::class),
 		);
 
 	}//end setUp()

tests/Unit/Controller/ContactpersonenControllerUpdateUserGroupsTest.php

@@ -101,7 +101,8 @@ protected function setUp(): void {
 			$this->userSession,
 			$this->container,
 			$this->createMock(ISecureRandom::class),
-			$this->logger
+			$this->logger,
+			objectService: $this->createMock(ObjectServiceInterface::class),
 		);
 
 	}//end setUp()

tests/Unit/Controller/ContactpersonenControllerUserAdminContractTest.php

@@ -30,6 +30,7 @@
 
 namespace OCA\SoftwareCatalog\Tests\Unit\Controller;
 
+use OCA\OpenRegister\Contract\ObjectServiceInterface;
 use OCA\SoftwareCatalog\Controller\ContactpersonenController;
 use OCA\SoftwareCatalog\Service\ContactpersoonService;
 use OCA\SoftwareCatalog\Service\SettingsService;
@@ -114,7 +115,8 @@ private function makeController(): ContactpersonenController {
 			$this->userSession,
 			$this->container,
 			$this->createMock(ISecureRandom::class),
-			$this->createMock(LoggerInterface::class)
+			$this->createMock(LoggerInterface::class),
+			objectService: $this->createMock(ObjectServiceInterface::class),
 		);
 
 	}//end makeController()

tests/Unit/EventListener/UserProfileUpdatedEventListenerDecompositionTest.php

@@ -21,6 +21,7 @@
 
 namespace OCA\SoftwareCatalog\Tests\Unit\EventListener;
 
+use OCA\OpenRegister\Contract\ObjectServiceInterface;
 use OCA\OpenRegister\Event\UserProfileUpdatedEvent;
 use OCA\SoftwareCatalog\EventListener\UserProfileUpdatedEventListener;
 use PHPUnit\Framework\TestCase;
@@ -49,7 +50,9 @@ private function makeListener(): UserProfileUpdatedEventListener {
 			$this->markTestSkipped('OCA\\OpenRegister\\Event\\UserProfileUpdatedEvent is not autoloadable in this environment.');
 		}
 
-		return new UserProfileUpdatedEventListener($this->createMock(ContainerInterface::class));
+		return new UserProfileUpdatedEventListener($this->createMock(ContainerInterface::class),
+			objectService: $this->createMock(ObjectServiceInterface::class),
+		);
 	}//end makeListener()
 
 	/**

tests/Unit/OrganisationUserWorkflowTest.php

@@ -160,7 +160,8 @@ protected function setUp(): void {
 			$this->createMock(IUserSession::class),
 			$this->createMock(ContainerInterface::class),
 			$this->createMock(ISecureRandom::class),
-			$this->logger
+			$this->logger,
+			objectService: $this->createMock(ObjectServiceInterface::class),
 		);
 	}
 

tests/Unit/Service/GebruikSyncServiceDecompositionTest.php

@@ -20,6 +20,7 @@
 
 namespace OCA\SoftwareCatalog\Tests\Unit\Service;
 
+use OCA\OpenRegister\Contract\ObjectServiceInterface;
 use OCA\SoftwareCatalog\Service\GebruikSyncService;
 use OCA\SoftwareCatalog\Service\SettingsService;
 use PHPUnit\Framework\TestCase;
@@ -47,6 +48,7 @@ private function makeService(): GebruikSyncService {
 			new NullLogger(),
 			$this->createMock(SettingsService::class),
 			$this->createMock(ContainerInterface::class),
+			objectService: $this->createMock(ObjectServiceInterface::class),
 		);
 
 	}//end makeService()