More fixes

Signed-off-by: Roeland Jago Douma <[email protected]>

apps/dav/tests/unit/CalDAV/Search/SearchPluginTest.php

@@ -26,6 +26,7 @@
 use OCA\DAV\CalDAV\CalendarHome;
 use OCA\DAV\CalDAV\Search\SearchPlugin;
 use OCA\DAV\CalDAV\Search\Xml\Request\CalendarSearchReport;
+use Sabre\Xml\Service;
 use Test\TestCase;
 
 class SearchPluginTest extends TestCase {
@@ -41,6 +42,7 @@ public function setUp(): void {
 		$this->server = $this->createMock(\Sabre\DAV\Server::class);
 		$this->server->tree = $this->createMock(\Sabre\DAV\Tree::class);
 		$this->server->httpResponse = $this->createMock(\Sabre\HTTP\Response::class);
+		$this->server->xml = new Service();
 
 		$this->plugin = new SearchPlugin();
 		$this->plugin->initialize($this->server);
@@ -62,6 +64,7 @@ public function testInitialize() {
 		$server->expects($this->at(0))
 			->method('on')
 			->with('report', [$plugin, 'report']);
+		$server->xml = new Service();
 
 		$plugin->initialize($server);
 
@@ -93,6 +96,11 @@ public function testReport() {
 			->method('getHTTPDepth')
 			->with(2)
 			->will($this->returnValue(2));
+		$this->server
+			->method('getHTTPPrefer')
+			->willReturn([
+				'return' => null
+			]);
 		$calendarHome->expects($this->at(0))
 			->method('calendarSearch')
 			->will($this->returnValue([]));

tests/Core/Controller/AvatarControllerTest.php

@@ -332,7 +332,7 @@ public function testPostAvatarFileGif() {
 		$this->cache->method('get')->willReturn(file_get_contents(\OC::$SERVERROOT.'/tests/data/testimage.gif'));
 
 		//Create request return
-		$reqRet = ['error' => [0], 'tmp_name' => [$fileName], 'size' => filesize(\OC::$SERVERROOT.'/tests/data/testimage.gif')];
+		$reqRet = ['error' => [0], 'tmp_name' => [$fileName], 'size' => [filesize(\OC::$SERVERROOT.'/tests/data/testimage.gif')]];
 		$this->request->method('getUploadedFile')->willReturn($reqRet);
 
 		$response = $this->avatarController->postAvatar(null);

tests/lib/Files/Storage/Wrapper/EncryptionTest.php

@@ -284,13 +284,17 @@ function($path) use ($encrypted) {
 			->method('getCache')
 			->with($path)
 			->willReturn($fileEntry);
-		$fileEntry->expects($this->any())
-			->method('get')
-			->with($metaData['fileid']);
+		if ($metaData !== null) {
+			$fileEntry->expects($this->any())
+				->method('get')
+				->with($metaData['fileid']);
+		}
 
 		$this->instance->expects($this->any())->method('getCache')->willReturn($cache);
-		$this->instance->expects($this->any())->method('verifyUnencryptedSize')
-			->with($path, 0)->willReturn($expected['size']);
+		if ($expected !== null) {
+			$this->instance->expects($this->any())->method('verifyUnencryptedSize')
+				->with($path, 0)->willReturn($expected['size']);
+		}
 
 		$result = $this->instance->getMetaData($path);
 		if(isset($expected['encrypted'])) {
@@ -300,7 +304,12 @@ function($path) use ($encrypted) {
 				$this->assertSame($expected['encryptedVersion'], $result['encryptedVersion']);
 			}
 		}
-		$this->assertSame($expected['size'], $result['size']);
+
+		if ($expected !== null) {
+			$this->assertSame($expected['size'], $result['size']);
+		} else {
+			$this->assertSame(null, $result);
+		}
 	}
 
 	public function dataTestGetMetaData() {

tests/lib/Session/CryptoWrappingTest.php

@@ -52,6 +52,9 @@ protected function setUp(): void {
 		$this->crypto->expects($this->any())
 			->method('decrypt')
 			->willReturnCallback(function ($input) {
+				if ($input === '') {
+					return '';
+				}
 				return substr($input, 1, -1);
 			});