fix(tests): Update encryption command tests for IAppConfig changes

Update unit tests for Enable and Disable commands to mock the new
IAppConfig dependency that was added in the previous commit.

Changes:
- tests/Core/Command/Encryption/EnableTest: Add IAppConfig mock
- tests/Core/Command/Encryption/DisableTest: Add IAppConfig mock
- Update test expectations to use getValueString/setValueString

Fixes 7 test failures in the NODB test suite.

Co-authored-by: Claude Sonnet 4.5 (1M context) <[email protected]>
Signed-off-by: Stephen Cuppett <[email protected]>

Author: cuppett

tests/Core/Command/Encryption/DisableTest.php

@@ -9,6 +9,7 @@
 namespace Tests\Core\Command\Encryption;
 
 use OC\Core\Command\Encryption\Disable;
+use OCP\IAppConfig;
 use OCP\IConfig;
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Output\OutputInterface;
@@ -18,6 +19,8 @@ class DisableTest extends TestCase {
 	/** @var \PHPUnit\Framework\MockObject\MockObject */
 	protected $config;
 	/** @var \PHPUnit\Framework\MockObject\MockObject */
+	protected $appConfig;
+	/** @var \PHPUnit\Framework\MockObject\MockObject */
 	protected $consoleInput;
 	/** @var \PHPUnit\Framework\MockObject\MockObject */
 	protected $consoleOutput;
@@ -31,11 +34,15 @@ protected function setUp(): void {
 		$config = $this->config = $this->getMockBuilder(IConfig::class)
 			->disableOriginalConstructor()
 			->getMock();
+		$appConfig = $this->appConfig = $this->getMockBuilder(IAppConfig::class)
+			->disableOriginalConstructor()
+			->getMock();
 		$this->consoleInput = $this->getMockBuilder(InputInterface::class)->getMock();
 		$this->consoleOutput = $this->getMockBuilder(OutputInterface::class)->getMock();
 
 		/** @var IConfig $config */
-		$this->command = new Disable($config);
+		/** @var IAppConfig $appConfig */
+		$this->command = new Disable($config, $appConfig);
 	}
 
 
@@ -54,18 +61,18 @@ public static function dataDisable(): array {
 	 */
 	#[\PHPUnit\Framework\Attributes\DataProvider('dataDisable')]
 	public function testDisable($oldStatus, $isUpdating, $expectedString): void {
-		$this->config->expects($this->once())
-			->method('getAppValue')
-			->with('core', 'encryption_enabled', $this->anything())
+		$this->appConfig->expects($this->once())
+			->method('getValueString')
+			->with('core', 'encryption_enabled', 'no')
 			->willReturn($oldStatus);
 
 		$this->consoleOutput->expects($this->once())
 			->method('writeln')
 			->with($this->stringContains($expectedString));
 
 		if ($isUpdating) {
-			$this->config->expects($this->once())
-				->method('setAppValue')
+			$this->appConfig->expects($this->once())
+				->method('setValueString')
 				->with('core', 'encryption_enabled', 'no');
 		}
 

tests/Core/Command/Encryption/EnableTest.php

@@ -10,6 +10,7 @@
 
 use OC\Core\Command\Encryption\Enable;
 use OCP\Encryption\IManager;
+use OCP\IAppConfig;
 use OCP\IConfig;
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Output\OutputInterface;
@@ -19,6 +20,8 @@ class EnableTest extends TestCase {
 	/** @var \PHPUnit\Framework\MockObject\MockObject */
 	protected $config;
 	/** @var \PHPUnit\Framework\MockObject\MockObject */
+	protected $appConfig;
+	/** @var \PHPUnit\Framework\MockObject\MockObject */
 	protected $manager;
 	/** @var \PHPUnit\Framework\MockObject\MockObject */
 	protected $consoleInput;
@@ -34,15 +37,19 @@ protected function setUp(): void {
 		$config = $this->config = $this->getMockBuilder(IConfig::class)
 			->disableOriginalConstructor()
 			->getMock();
+		$appConfig = $this->appConfig = $this->getMockBuilder(IAppConfig::class)
+			->disableOriginalConstructor()
+			->getMock();
 		$manager = $this->manager = $this->getMockBuilder(IManager::class)
 			->disableOriginalConstructor()
 			->getMock();
 		$this->consoleInput = $this->getMockBuilder(InputInterface::class)->getMock();
 		$this->consoleOutput = $this->getMockBuilder(OutputInterface::class)->getMock();
 
 		/** @var \OCP\IConfig $config */
+		/** @var \OCP\IAppConfig $appConfig */
 		/** @var \OCP\Encryption\IManager $manager */
-		$this->command = new Enable($config, $manager);
+		$this->command = new Enable($config, $appConfig, $manager);
 	}
 
 
@@ -59,8 +66,8 @@ public static function dataEnable(): array {
 	#[\PHPUnit\Framework\Attributes\DataProvider('dataEnable')]
 	public function testEnable(string $oldStatus, ?string $defaultModule, array $availableModules, bool $isUpdating, string $expectedString, string $expectedDefaultModuleString): void {
 		if ($isUpdating) {
-			$this->config->expects($this->once())
-				->method('setAppValue')
+			$this->appConfig->expects($this->once())
+				->method('setValueString')
 				->with('core', 'encryption_enabled', 'yes');
 		}
 
@@ -69,14 +76,14 @@ public function testEnable(string $oldStatus, ?string $defaultModule, array $ava
 			->willReturn($availableModules);
 
 		if (empty($availableModules)) {
-			$this->config->expects($this->once())
-				->method('getAppValue')
+			$this->appConfig->expects($this->once())
+				->method('getValueString')
 				->willReturnMap([
 					['core', 'encryption_enabled', 'no', $oldStatus],
 				]);
 		} else {
-			$this->config->expects($this->exactly(2))
-				->method('getAppValue')
+			$this->appConfig->expects($this->exactly(2))
+				->method('getValueString')
 				->willReturnMap([
 					['core', 'encryption_enabled', 'no', $oldStatus],
 					['core', 'default_encryption_module', '', $defaultModule],