fix(tests): Add tearDown to encryption tests to prevent state pollution

Fixes 375 PHPUnit errors in php8.2-s3-minio CI caused by encryption
tests not cleaning up their state, polluting subsequent tests.

Root Cause:
- 5 tests using EncryptionTrait enable encryption but have NO tearDown
- After tests complete, encryption_enabled remains 'yes'
- EncryptionWrapper sees encryption enabled for ALL subsequent tests
- Tests without encryption keys fail with "multikeyencryption failed"

Tests Fixed:
1. apps/encryption/tests/EncryptedStorageTest.php
2. apps/encryption/tests/Command/FixEncryptedVersionTest.php
3. apps/files_sharing/tests/EncryptedSizePropagationTest.php
4. apps/dav/tests/unit/Connector/Sabre/RequestTest/EncryptionUploadTest.php
5. apps/dav/tests/unit/Connector/Sabre/RequestTest/EncryptionMasterKeyUploadTest.php

Fix:
Add tearDown() method to each test that calls tearDownEncryptionTrait(),
which restores encryption_enabled to its previous value.

Also Fixed:
- tests/lib/Encryption/EncryptionWrapperTest.php: Mock isEnabled() for
  tests that expect wrapper to be applied

Testing:
- Local full suite run: 0 encryption errors (vs 375 in CI)
- Encryption state properly restored after tests
- ViewTest passes after encryption tests run

Impact:
- Prevents test state pollution in CI
- Each test properly cleans up encryption configuration
- No functional changes to production code

Signed-off-by: Stephen Cuppett <[email protected]>

Author: cuppett

apps/dav/tests/unit/Connector/Sabre/RequestTest/EncryptionMasterKeyUploadTest.php

@@ -34,4 +34,9 @@ protected function setupUser($name, $password): View {
 		$this->loginWithEncryption($name);
 		return new View('/' . $name . '/files');
 	}
+
+	protected function tearDown(): void {
+		$this->tearDownEncryptionTrait();
+		parent::tearDown();
+	}
 }

apps/dav/tests/unit/Connector/Sabre/RequestTest/EncryptionUploadTest.php

@@ -34,4 +34,9 @@ protected function setupUser($name, $password): View {
 		$this->loginWithEncryption($name);
 		return new View('/' . $name . '/files');
 	}
+
+	protected function tearDown(): void {
+		$this->tearDownEncryptionTrait();
+		parent::tearDown();
+	}
 }

apps/encryption/tests/Command/FixEncryptedVersionTest.php

@@ -390,4 +390,9 @@ public function testExecuteWithNoMasterKey(): void {
 
 		$this->assertStringContainsString('only works with master key', $output);
 	}
+
+	protected function tearDown(): void {
+		$this->tearDownEncryptionTrait();
+		parent::tearDown();
+	}
 }

apps/encryption/tests/EncryptedStorageTest.php

@@ -66,4 +66,9 @@ public function testMoveFromEncrypted(): void {
 		$this->assertEquals('bar', $unencryptedStorage->file_get_contents('foo.txt'));
 		$this->assertFalse($unencryptedCache->get('foo.txt')->isEncrypted());
 	}
+
+	protected function tearDown(): void {
+		$this->tearDownEncryptionTrait();
+		parent::tearDown();
+	}
 }

apps/files_sharing/tests/EncryptedSizePropagationTest.php

@@ -39,4 +39,9 @@ protected function loginHelper($user, $create = false, $password = false) {
 		$this->setupForUser($user, $password);
 		parent::loginHelper($user, $create, $password);
 	}
+
+	protected function tearDown(): void {
+		$this->tearDownEncryptionTrait();
+		parent::tearDown();
+	}
 }