summaryrefslogtreecommitdiffstats
path: root/apps/files_sharing/tests
diff options
context:
space:
mode:
authorCarl Schwan <carl@carlschwan.eu>2022-09-15 16:43:01 +0200
committerGitHub <noreply@github.com>2022-09-15 16:43:01 +0200
commit02d55eb90fdc61137745c5e5b3d8c7330e0717c2 (patch)
treee560bf8b22e696f3dd0aaa73c6dd4ae083cc8a78 /apps/files_sharing/tests
parent1013c3bf66803fd262fa2c82f5633af61305a151 (diff)
parent7ded5a769b7d7d681c3a3260b77869b18c9822ae (diff)
downloadnextcloud-server-02d55eb90fdc61137745c5e5b3d8c7330e0717c2.tar.gz
nextcloud-server-02d55eb90fdc61137745c5e5b3d8c7330e0717c2.zip
Merge pull request #33941 from nextcloud/fix/public-sharing
Only normalize data in public shares and not crud
Diffstat (limited to 'apps/files_sharing/tests')
-rw-r--r--apps/files_sharing/tests/Controller/ShareAPIControllerTest.php6
1 files changed, 3 insertions, 3 deletions
diff --git a/apps/files_sharing/tests/Controller/ShareAPIControllerTest.php b/apps/files_sharing/tests/Controller/ShareAPIControllerTest.php
index f8f4fb18bc8..4ac4cd6956e 100644
--- a/apps/files_sharing/tests/Controller/ShareAPIControllerTest.php
+++ b/apps/files_sharing/tests/Controller/ShareAPIControllerTest.php
@@ -3037,7 +3037,7 @@ class ShareAPIControllerTest extends TestCase {
// legacy had no delete
[
\OCP\Constants::PERMISSION_READ | \OCP\Constants::PERMISSION_CREATE | \OCP\Constants::PERMISSION_UPDATE,
- null, null, 'password'
+ 'true', null, 'password'
],
// correct
[
@@ -3538,7 +3538,7 @@ class ShareAPIControllerTest extends TestCase {
$this->shareManager->method('shareApiLinkAllowPublicUpload')->willReturn(true);
$this->shareManager->expects($this->once())->method('updateShare')->with(
- $this->callback(function (\OCP\Share\IShare $share) use ($date) {
+ $this->callback(function (\OCP\Share\IShare $share) use ($date): bool {
return $share->getPermissions() === (\OCP\Constants::PERMISSION_READ | \OCP\Constants::PERMISSION_CREATE | \OCP\Constants::PERMISSION_UPDATE | \OCP\Constants::PERMISSION_DELETE) &&
$share->getPassword() === 'password' &&
$share->getSendPasswordByTalk() === true &&
@@ -3567,7 +3567,7 @@ class ShareAPIControllerTest extends TestCase {
->willReturn(42);
$expected = new DataResponse([]);
- $result = $ocs->updateShare(42, 7, null, null, null, null, null, null, null);
+ $result = $ocs->updateShare(42, 7, null, null, 'true', null, null, null, null);
$this->assertInstanceOf(get_class($expected), $result);
$this->assertEquals($expected->getData(), $result->getData());