fix: integration tests

Signed-off-by: Luka Trovic <luka@nextcloud.com>

lib/Db/AssignmentMapper.php

@@ -154,19 +154,17 @@ class AssignmentMapper extends QBMapper implements IPermissionMapper {
 	 * @return void
 	 */
 	public function transferOwnership(string $ownerId, string $newOwnerId) {
-		$params = [
-			'newOwner' => $newOwnerId,
-			'type' => Assignment::TYPE_USER
-		];
-		$qb = $this->db->getQueryBuilder();
-		$sql = "DELETE FROM `*PREFIX*{$this->tableName}`  WHERE `participant` = :newOwner AND `type`= :type";
-		$stmt = $this->db->executeQuery($sql, $params);
-		$stmt->closeCursor();
 		$params = [
 			'owner' => $ownerId,
 			'newOwner' => $newOwnerId,
 			'type' => Assignment::TYPE_USER
 		];
+		$qb = $this->db->getQueryBuilder();
+		$sql = "DELETE FROM `*PREFIX*{$this->tableName}`  WHERE `participant` = :newOwner AND `type`= :type AND id IN 
+			(SELECT id FROM `*PREFIX*{$this->tableName}`  WHERE `participant` = :owner)";
+		$stmt = $this->db->executeQuery($sql, $params);
+		$stmt->closeCursor();
+		
 		$sql = "UPDATE `*PREFIX*{$this->tableName}`  SET `participant` = :newOwner WHERE `participant` = :owner AND `type`= :type";
 		$stmt = $this->db->executeQuery($sql, $params);
 		$stmt->closeCursor();

lib/Db/BoardMapper.php

@@ -182,7 +182,7 @@ class BoardMapper extends QBMapper implements IPermissionMapper {
 
 		// shared with user
 		$qb->resetQueryParts();
-		$qb->select('b.id', 'title', 'owner', 'color', 'archived', 'deleted_at', 'last_modified')
+		$qb->selectDistinct('b.id', 'title', 'owner', 'color', 'archived', 'deleted_at', 'last_modified')
 			//->selectAlias('1', 'shared')
 			->from('deck_boards', 'b')
 			->innerJoin('b', 'deck_board_acl', 'acl', $qb->expr()->eq('b.id', 'acl.board_id'))
@@ -490,4 +490,25 @@ class BoardMapper extends QBMapper implements IPermissionMapper {
 		$stmt = $this->db->executeQuery($sql, $params);
 		$stmt->closeCursor();
 	}
+
+	/**
+	 * Reset Cache for a 
+	 * given board or a given user
+	 * 
+	 * @param int|null $boardId
+	 * @param int|null $userId
+	 */
+	public function flushCache(?int $boardId = null, ?string $userId = null)
+	{
+		if ($boardId) {
+			unset($this->boardCache[$boardId]);
+		} else {
+			$this->boardCache = null;
+		}
+		if ($userId) {
+			unset($this->userBoardCache[$userId]);
+		} else {
+			$this->userBoardCache = null;
+		}
+	}
 }

lib/Service/BoardService.php

@@ -528,6 +528,9 @@ class BoardService {
 		$this->boardMapper->mapAcl($newAcl);
 		$this->changeHelper->boardChanged($boardId);
 
+		$board = $this->find($boardId);
+		$this->clearBoardFromCache($board);
+
 		// TODO: use the dispatched event for this
 		try {
 			$resourceProvider = \OC::$server->query(\OCA\Deck\Collaboration\Resources\ResourceProvider::class);
@@ -681,6 +684,7 @@ class BoardService {
 	public function transferOwnership(string $owner, string $newOwner): void {
 		$boards = $this->boardMapper->findAllByUser($owner);
 		foreach ($boards as $board) {
+			$this->clearBoardFromCache($board);
 			$this->aclMapper->transferOwnership($board->getId(), $owner, $newOwner);
 		}
 		$this->boardMapper->transferOwnership($owner, $newOwner);
@@ -723,4 +727,18 @@ class BoardService {
 	private function clearBoardsCache() {
 		$this->boardsCache = null;
 	}
+
+	/**
+	 * Clean a given board data
+	 * from the Cache
+	 * 
+	 * @param OCA\Deck\Db\Board $board
+	 */
+	private function clearBoardFromCache(Board $board) {
+		$boardId = $board->getId();
+		$boardOwnerId = $board->getOwner();
+
+		$this->boardMapper->flushCache($boardId, $boardOwnerId);
+		unset($this->boardsCache[$boardId]);
+	}
 }

tests/integration/database/TransferOwnershipTest.php

@@ -82,7 +82,7 @@ class TransferOwnershipTest extends \Test\TestCase {
 	 */
 	public function testTransferBoardOwnership() {
 		$this->boardService->transferOwnership(self::TEST_USER_1, self::TEST_USER_2);
-		$this->invokePrivate($this->boardService, 'clearBoardsCache');
+		/* $this->invokePrivate($this->boardService, 'clearBoardsCache'); */
 		$board = $this->boardService->find($this->board->getId());
 		$boardOwner = $board->getOwner();
 		$this->assertEquals(self::TEST_USER_2, $boardOwner);