Add transfer ownership unit tests

tests/Integration/Api/ApiV2Test.php

@@ -1491,4 +1491,22 @@ public function testDeleteSingleSubmission(array $submissionsExpected) {
 
 		$this->testGetSubmissions($submissionsExpected);
 	}
+
+	/**
+	 * Test transfer owner endpoint for form
+	 *
+	 * Keep this test at the end as it might break other tests
+	 */
+	public function testTransferOwner() {
+		$resp = $this->http->request('POST', "api/v2.4/form/transfer", [
+			'json' => [
+				'formId' => $this->testForms[0]['id'],
+				'uid' => 'user1'
+			],
+		]);
+		$data = $this->OcsResponse2Data($resp);
+
+		$this->assertEquals(200, $resp->getStatusCode());
+		$this->assertEquals('user1', $data);
+	}
 };

tests/Unit/Controller/ApiControllerTest.php

@@ -904,4 +904,47 @@ public function dataTestDeletePermission() {
 			]
 		];
 	}
+
+	public function testTransferOwnerNotOwner() {
+		$form = new Form();
+		$form->setId(1);
+		$form->setHash('hash');
+		$form->setOwnerId('otherUser');
+
+		$this->formMapper
+			->method('findById')
+			->with(1)
+			->willReturn($form);
+
+		$newOwner = $this->createMock(IUser::class);
+		$this->userManager->expects($this->once())
+			->method('get')
+			->with('newOwner')
+			->willReturn($newOwner);
+
+		$this->expectException(OCSForbiddenException::class);
+		$this->expectExceptionMessage('This form is not owned by the current user');
+		$this->apiController->transferOwner(1, 'newOwner');
+	}
+
+	public function testTransferOwner() {
+		$form = new Form();
+		$form->setId(1);
+		$form->setHash('hash');
+		$form->setOwnerId('currentUser');
+
+		$this->formMapper
+			->method('findById')
+			->with(1)
+			->willReturn($form);
+
+		$newOwner = $this->createMock(IUser::class);
+		$this->userManager->expects($this->once())
+			->method('get')
+			->with('newOwner')
+			->willReturn($newOwner);
+
+		$this->assertEquals(new DataResponse('newOwner'), $this->apiController->transferOwner(1, 'newOwner'));
+		$this->assertEquals('newOwner', $form->getOwnerId());
+	}
 }