Skip to content

Commit 441ca8b

Browse files
committed
fix(tests): Fix sharebymail tests
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
1 parent 5f903bb commit 441ca8b

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

apps/sharebymail/lib/ShareByMailProvider.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -678,8 +678,8 @@ public function update(IShare $share, ?string $plainTextPassword = null): IShare
678678
$qb = $this->dbConnection->getQueryBuilder();
679679
$qb->update('share')
680680
->where($qb->expr()->eq('id', $qb->createNamedParameter($share->getId())))
681-
->set('item_source', $qb->createNamedParameter($share->getNode()->getId()))
682-
->set('file_source', $qb->createNamedParameter($share->getNode()->getId()))
681+
->set('item_source', $qb->createNamedParameter($share->getNodeId()))
682+
->set('file_source', $qb->createNamedParameter($share->getNodeId()))
683683
->set('share_with', $qb->createNamedParameter($share->getSharedWith()))
684684
->set('permissions', $qb->createNamedParameter($share->getPermissions()))
685685
->set('uid_owner', $qb->createNamedParameter($share->getShareOwner()))

apps/sharebymail/tests/ShareByMailProviderTest.php

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -643,6 +643,8 @@ public function testUpdate() {
643643
$this->share->expects($this->once())->method('getSharedBy')->willReturn($sharedBy);
644644
$this->share->expects($this->any())->method('getNote')->willReturn($note);
645645
$this->share->expects($this->atLeastOnce())->method('getId')->willReturn($id);
646+
$this->share->expects($this->atLeastOnce())->method('getNodeId')->willReturn($itemSource);
647+
$this->share->expects($this->once())->method('getSharedWith')->willReturn($shareWith);
646648

647649
$this->assertSame($this->share,
648650
$instance->update($this->share)

0 commit comments

Comments
 (0)