fix(tests): Fix share tests to test new reshare promotion system

Signed-off-by: Côme Chilliet <[email protected]>
nextcloud · Oct 3, 2024 · a26b8f0 · a26b8f0
1 parent 8974542
commit a26b8f0
Showing 1 changed file with 17 additions and 17 deletions.
diff --git a/tests/lib/Share20/ManagerTest.php b/tests/lib/Share20/ManagerTest.php
@@ -228,7 +228,7 @@ public function dataTestDelete() {
  */
  public function testDelete($shareType, $sharedWith): void {
  $manager = $this->createManagerMock()
- ->setMethods(['getShareById', 'deleteChildren', 'deleteReshares'])
+ ->setMethods(['getShareById', 'deleteChildren', 'promoteReshares'])
  ->getMock();
 
  $manager->method('deleteChildren')->willReturn([]);
@@ -246,7 +246,7 @@ public function testDelete($shareType, $sharedWith): void {
  ->setTarget('myTarget');
 
  $manager->expects($this->once())->method('deleteChildren')->with($share);
- $manager->expects($this->once())->method('deleteReshares')->with($share);
+ $manager->expects($this->once())->method('promoteReshares')->with($share);
 
  $this->defaultProvider
  ->expects($this->once())
@@ -271,7 +271,7 @@ public function testDelete($shareType, $sharedWith): void {
 
  public function testDeleteLazyShare(): void {
  $manager = $this->createManagerMock()
- ->setMethods(['getShareById', 'deleteChildren', 'deleteReshares'])
+ ->setMethods(['getShareById', 'deleteChildren', 'promoteReshares'])
  ->getMock();
 
  $manager->method('deleteChildren')->willReturn([]);
@@ -290,7 +290,7 @@ public function testDeleteLazyShare(): void {
  $this->rootFolder->expects($this->never())->method($this->anything());
 
  $manager->expects($this->once())->method('deleteChildren')->with($share);
- $manager->expects($this->once())->method('deleteReshares')->with($share);
+ $manager->expects($this->once())->method('promoteReshares')->with($share);
 
  $this->defaultProvider
  ->expects($this->once())
@@ -315,7 +315,7 @@ public function testDeleteLazyShare(): void {
 
  public function testDeleteNested(): void {
  $manager = $this->createManagerMock()
- ->setMethods(['getShareById', 'deleteReshares'])
+ ->setMethods(['getShareById', 'promoteReshares'])
  ->getMock();
 
  $path = $this->createMock(File::class);
@@ -472,9 +472,9 @@ public function testDeleteChildren(): void {
  $this->assertSame($shares, $result);
  }
 
- public function testDeleteReshareWhenUserHasOneShare(): void {
+ public function testPromoteReshareWhenUserHasOneShare(): void {
  $manager = $this->createManagerMock()
- ->setMethods(['deleteShare', 'getSharesInFolder', 'generalCreateChecks'])
+ ->setMethods(['updateShare', 'getSharesInFolder', 'generalCreateChecks'])
  ->getMock();
 
  $folder = $this->createMock(Folder::class);
@@ -499,14 +499,14 @@ public function testDeleteReshareWhenUserHasOneShare(): void {
  $this->defaultProvider->method('getSharesBy')
  ->willReturn([$reShare]);
 
- $manager->expects($this->atLeast(2))->method('deleteShare')->withConsecutive([$reShare], [$reShareInSubFolder]);
+ $manager->expects($this->atLeast(2))->method('updateShare')->withConsecutive([$reShare], [$reShareInSubFolder]);
 
- self::invokePrivate($manager, 'deleteReshares', [$share]);
+ self::invokePrivate($manager, 'promoteReshares', [$share]);
  }
 
- public function testDeleteReshareWhenUserHasAnotherShare(): void {
+ public function testPromoteReshareWhenUserHasAnotherShare(): void {
  $manager = $this->createManagerMock()
- ->setMethods(['deleteShare', 'getSharesInFolder', 'getSharedWith', 'generalCreateChecks'])
+ ->setMethods(['updateShare', 'getSharesInFolder', 'getSharedWith', 'generalCreateChecks'])
  ->getMock();
 
  $folder = $this->createMock(Folder::class);
@@ -527,14 +527,14 @@ public function testDeleteReshareWhenUserHasAnotherShare(): void {
  $manager->method('getSharesInFolder')->willReturn([]);
  $manager->method('generalCreateChecks')->willReturn(true);
 
- $manager->expects($this->never())->method('deleteShare');
+ $manager->expects($this->never())->method('updateShare');
 
- self::invokePrivate($manager, 'deleteReshares', [$share]);
+ self::invokePrivate($manager, 'promoteReshares', [$share]);
  }
 
- public function testDeleteReshareOfUsersInGroupShare(): void {
+ public function testPromoteReshareOfUsersInGroupShare(): void {
  $manager = $this->createManagerMock()
- ->setMethods(['deleteShare', 'getSharesInFolder', 'getSharedWith', 'generalCreateChecks'])
+ ->setMethods(['updateShare', 'getSharesInFolder', 'getSharedWith', 'generalCreateChecks'])
  ->getMock();
 
  $folder = $this->createMock(Folder::class);
@@ -576,9 +576,9 @@ public function testDeleteReshareOfUsersInGroupShare(): void {
  $manager->method('getSharedWith')->willReturn([]);
  $manager->expects($this->exactly(2))->method('getSharesInFolder')->willReturnOnConsecutiveCalls([[$reShare1]], [[$reShare2]]);
 
- $manager->expects($this->exactly(2))->method('deleteShare')->withConsecutive([$reShare1], [$reShare2]);
+ $manager->expects($this->exactly(2))->method('updateShare')->withConsecutive([$reShare1], [$reShare2]);
 
- self::invokePrivate($manager, 'deleteReshares', [$share]);
+ self::invokePrivate($manager, 'promoteReshares', [$share]);
  }
 
  public function testGetShareById(): void {