chore(tests): make "execute" method in Add command public

to simplify test execution

Signed-off-by: Misha M.-Kupriyanov <kupriyanov@strato.de>

Author: printminion-co

apps/settings/lib/Command/AdminDelegation/Add.php

@@ -36,7 +36,7 @@ protected function configure(): void {
 		;
 	}
 
-	protected function execute(InputInterface $input, OutputInterface $output): int {
+	public function execute(InputInterface $input, OutputInterface $output): int {
 		$io = new SymfonyStyle($input, $output);
 		$settingClass = $input->getArgument('settingClass');
 		if (!in_array(IDelegatedSettings::class, (array)class_implements($settingClass), true)) {

apps/settings/tests/Command/AdminDelegation/AddTest.php

@@ -44,13 +44,6 @@ protected function setUp(): void {
 		$this->output = $this->createMock(OutputInterface::class);
 	}
 
-	/**
-	 * Helper method to execute the command using reflection since execute() is protected
-	 */
-	private function executeCommand(): int {
-		return self::invokePrivate($this->command, 'execute', [$this->input, $this->output]);
-	}
-
 	public function testExecuteSuccessfulDelegation(): void {
 		$settingClass = \OCA\Settings\Settings\Admin\Server::class;
 		$groupId = 'testgroup';
@@ -79,7 +72,7 @@ public function testExecuteSuccessfulDelegation(): void {
 			->with($groupId, $settingClass)
 			->willReturn($authorizedGroup);
 
-		$result = $this->executeCommand();
+		$result = $this->command->execute($this->input, $this->output);
 
 		$this->assertEquals(0, $result);
 	}
@@ -93,7 +86,7 @@ public function testExecuteInvalidSettingClass(): void {
 			->with('settingClass')
 			->willReturn($settingClass);
 
-		$result = $this->executeCommand();
+		$result = $this->command->execute($this->input, $this->output);
 
 		// Should return exit code 2 for invalid setting class
 		$this->assertEquals(2, $result);
@@ -116,7 +109,7 @@ public function testExecuteNonExistentGroup(): void {
 			->with($groupId)
 			->willReturn(false);
 
-		$result = $this->executeCommand();
+		$result = $this->command->execute($this->input, $this->output);
 
 		// Should return exit code 3 for non-existent group
 		$this->assertEquals(3, $result);