123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460461462463464465466467468469 |
- <?php
- /**
- * @copyright Copyright (c) 2016, ownCloud, Inc.
- *
- * @author Björn Schießle <bjoern@schiessle.org>
- * @author Joas Schilling <coding@schilljs.com>
- * @author Lukas Reschke <lukas@statuscode.ch>
- * @author Morris Jobke <hey@morrisjobke.de>
- * @author Robin Appelman <robin@icewind.nl>
- * @author Roeland Jago Douma <roeland@famdouma.nl>
- * @author Thomas Müller <thomas.mueller@tmit.eu>
- * @author Vincent Petry <pvince81@owncloud.com>
- *
- * @license AGPL-3.0
- *
- * This code is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Affero General Public License, version 3,
- * as published by the Free Software Foundation.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU Affero General Public License for more details.
- *
- * You should have received a copy of the GNU Affero General Public License, version 3,
- * along with this program. If not, see <http://www.gnu.org/licenses/>
- *
- */
- namespace OCA\Files_Sharing\Tests;
- use OCP\IGroupManager;
- use OCP\IUserManager;
- /**
- * Class SharedMountTest
- *
- * @group SLOWDB
- */
- class SharedMountTest extends TestCase {
- /** @var IGroupManager */
- private $groupManager;
- /** @var IUserManager */
- private $userManager;
- protected function setUp(): void {
- parent::setUp();
- $this->folder = '/folder_share_storage_test';
- $this->filename = '/share-api-storage.txt';
- $this->view->mkdir($this->folder);
- // save file with content
- $this->view->file_put_contents($this->filename, 'root file');
- $this->view->file_put_contents($this->folder . $this->filename, 'file in subfolder');
- $this->groupManager = \OC::$server->getGroupManager();
- $this->userManager = \OC::$server->getUserManager();
- }
- protected function tearDown(): void {
- if ($this->view) {
- if ($this->view->file_exists($this->folder)) {
- $this->view->unlink($this->folder);
- }
- if ($this->view->file_exists($this->filename)) {
- $this->view->unlink($this->filename);
- }
- }
- parent::tearDown();
- }
- /**
- * test if the mount point moves up if the parent folder no longer exists
- */
- public function testShareMountLoseParentFolder() {
- // share to user
- $share = $this->share(
- \OCP\Share::SHARE_TYPE_USER,
- $this->folder,
- self::TEST_FILES_SHARING_API_USER1,
- self::TEST_FILES_SHARING_API_USER2,
- \OCP\Constants::PERMISSION_ALL);
- $this->shareManager->acceptShare($share, self::TEST_FILES_SHARING_API_USER2);
- $share->setTarget('/foo/bar' . $this->folder);
- $this->shareManager->moveShare($share, self::TEST_FILES_SHARING_API_USER2);
- $share = $this->shareManager->getShareById($share->getFullId());
- $this->assertSame('/foo/bar' . $this->folder, $share->getTarget());
- self::loginHelper(self::TEST_FILES_SHARING_API_USER2);
- // share should have moved up
- $share = $this->shareManager->getShareById($share->getFullId());
- $this->assertSame($this->folder, $share->getTarget());
- //cleanup
- self::loginHelper(self::TEST_FILES_SHARING_API_USER1);
- $this->shareManager->deleteShare($share);
- $this->view->unlink($this->folder);
- }
- /**
- * @medium
- */
- public function testDeleteParentOfMountPoint() {
- // share to user
- $share = $this->share(
- \OCP\Share::SHARE_TYPE_USER,
- $this->folder,
- self::TEST_FILES_SHARING_API_USER1,
- self::TEST_FILES_SHARING_API_USER2,
- \OCP\Constants::PERMISSION_ALL
- );
- self::loginHelper(self::TEST_FILES_SHARING_API_USER2);
- $user2View = new \OC\Files\View('/' . self::TEST_FILES_SHARING_API_USER2 . '/files');
- $this->assertTrue($user2View->file_exists($this->folder));
- // create a local folder
- $result = $user2View->mkdir('localfolder');
- $this->assertTrue($result);
- // move mount point to local folder
- $result = $user2View->rename($this->folder, '/localfolder/' . $this->folder);
- $this->assertTrue($result);
- // mount point in the root folder should no longer exist
- $this->assertFalse($user2View->is_dir($this->folder));
- // delete the local folder
- $result = $user2View->unlink('/localfolder');
- $this->assertTrue($result);
- //enforce reload of the mount points
- self::loginHelper(self::TEST_FILES_SHARING_API_USER2);
- //mount point should be back at the root
- $this->assertTrue($user2View->is_dir($this->folder));
- //cleanup
- self::loginHelper(self::TEST_FILES_SHARING_API_USER1);
- $this->view->unlink($this->folder);
- }
- public function testMoveSharedFile() {
- $share = $this->share(
- \OCP\Share::SHARE_TYPE_USER,
- $this->filename,
- self::TEST_FILES_SHARING_API_USER1,
- self::TEST_FILES_SHARING_API_USER2,
- \OCP\Constants::PERMISSION_READ | \OCP\Constants::PERMISSION_UPDATE | \OCP\Constants::PERMISSION_SHARE
- );
- self::loginHelper(self::TEST_FILES_SHARING_API_USER2);
- \OC\Files\Filesystem::rename($this->filename, $this->filename . '_renamed');
- $this->assertTrue(\OC\Files\Filesystem::file_exists($this->filename . '_renamed'));
- $this->assertFalse(\OC\Files\Filesystem::file_exists($this->filename));
- self::loginHelper(self::TEST_FILES_SHARING_API_USER1);
- $this->assertTrue(\OC\Files\Filesystem::file_exists($this->filename));
- $this->assertFalse(\OC\Files\Filesystem::file_exists($this->filename . '_renamed'));
- // rename back to original name
- self::loginHelper(self::TEST_FILES_SHARING_API_USER2);
- \OC\Files\Filesystem::rename($this->filename . '_renamed', $this->filename);
- $this->assertFalse(\OC\Files\Filesystem::file_exists($this->filename . '_renamed'));
- $this->assertTrue(\OC\Files\Filesystem::file_exists($this->filename));
- //cleanup
- $this->shareManager->deleteShare($share);
- }
- /**
- * share file with a group if a user renames the file the filename should not change
- * for the other users
- */
- public function testMoveGroupShare () {
- $testGroup = $this->groupManager->createGroup('testGroup');
- $user1 = $this->userManager->get(self::TEST_FILES_SHARING_API_USER1);
- $user2 = $this->userManager->get(self::TEST_FILES_SHARING_API_USER2);
- $user3 = $this->userManager->get(self::TEST_FILES_SHARING_API_USER3);
- $testGroup->addUser($user1);
- $testGroup->addUser($user2);
- $testGroup->addUser($user3);
- $fileinfo = $this->view->getFileInfo($this->filename);
- $share = $this->share(
- \OCP\Share::SHARE_TYPE_GROUP,
- $this->filename,
- self::TEST_FILES_SHARING_API_USER1,
- 'testGroup',
- \OCP\Constants::PERMISSION_READ | \OCP\Constants::PERMISSION_UPDATE | \OCP\Constants::PERMISSION_SHARE
- );
- $this->shareManager->acceptShare($share, $user1->getUID());
- $this->shareManager->acceptShare($share, $user2->getUID());
- $this->shareManager->acceptShare($share, $user3->getUID());
- self::loginHelper(self::TEST_FILES_SHARING_API_USER2);
- $this->assertTrue(\OC\Files\Filesystem::file_exists($this->filename));
- \OC\Files\Filesystem::rename($this->filename, 'newFileName');
- $this->assertTrue(\OC\Files\Filesystem::file_exists('newFileName'));
- $this->assertFalse(\OC\Files\Filesystem::file_exists($this->filename));
- self::loginHelper(self::TEST_FILES_SHARING_API_USER3);
- $this->assertTrue(\OC\Files\Filesystem::file_exists($this->filename));
- $this->assertFalse(\OC\Files\Filesystem::file_exists('newFileName'));
- self::loginHelper(self::TEST_FILES_SHARING_API_USER3);
- $this->assertTrue(\OC\Files\Filesystem::file_exists($this->filename));
- $this->assertFalse(\OC\Files\Filesystem::file_exists('newFileName'));
- //cleanup
- self::loginHelper(self::TEST_FILES_SHARING_API_USER1);
- $this->shareManager->deleteShare($share);
- $testGroup->removeUser($user1);
- $testGroup->removeUser($user2);
- $testGroup->removeUser($user3);
- }
- /**
- * @dataProvider dataProviderTestStripUserFilesPath
- * @param string $path
- * @param string $expectedResult
- * @param bool $exception if a exception is expected
- */
- public function testStripUserFilesPath($path, $expectedResult, $exception) {
- $testClass = new DummyTestClassSharedMount(null, null);
- try {
- $result = $testClass->stripUserFilesPathDummy($path);
- $this->assertSame($expectedResult, $result);
- } catch (\Exception $e) {
- if ($exception) {
- $this->assertSame(10, $e->getCode());
- } else {
- $this->assertTrue(false, 'Exception catched, but expected: ' . $expectedResult);
- }
- }
- }
- public function dataProviderTestStripUserFilesPath() {
- return array(
- array('/user/files/foo.txt', '/foo.txt', false),
- array('/user/files/folder/foo.txt', '/folder/foo.txt', false),
- array('/data/user/files/foo.txt', null, true),
- array('/data/user/files/', null, true),
- array('/files/foo.txt', null, true),
- array('/foo.txt', null, true),
- );
- }
- public function dataPermissionMovedGroupShare() {
- $data = [];
- $powerset = function($permissions) {
- $results = [\OCP\Constants::PERMISSION_READ];
- foreach ($permissions as $permission) {
- foreach ($results as $combination) {
- $results[] = $permission | $combination;
- }
- }
- return $results;
- };
- //Generate file permissions
- $permissions = [
- \OCP\Constants::PERMISSION_UPDATE,
- \OCP\Constants::PERMISSION_SHARE,
- ];
- $allPermissions = $powerset($permissions);
- foreach ($allPermissions as $before) {
- foreach ($allPermissions as $after) {
- if ($before === $after) { continue; }
- $data[] = [
- 'file',
- $before,
- $after,
- ];
- }
- }
- //Generate folder permissions
- $permissions = [
- \OCP\Constants::PERMISSION_UPDATE,
- \OCP\Constants::PERMISSION_CREATE,
- \OCP\Constants::PERMISSION_SHARE,
- \OCP\Constants::PERMISSION_DELETE,
- ];
- $allPermissions = $powerset($permissions);
- foreach ($allPermissions as $before) {
- foreach ($allPermissions as $after) {
- if ($before === $after) { continue; }
- $data[] = [
- 'folder',
- $before,
- $after,
- ];
- }
- }
- return $data;
- }
- /**
- * moved mountpoints of a group share should keep the same permission as their parent group share.
- * See #15253
- *
- * @dataProvider dataPermissionMovedGroupShare
- */
- public function testPermissionMovedGroupShare($type, $beforePerm, $afterPerm) {
- $this->markTestSkipped('Unreliable test');
- if ($type === 'file') {
- $path = $this->filename;
- } else if ($type === 'folder') {
- $path = $this->folder;
- }
- $testGroup = $this->groupManager->createGroup('testGroup');
- $user1 = $this->userManager->get(self::TEST_FILES_SHARING_API_USER1);
- $user2 = $this->userManager->get(self::TEST_FILES_SHARING_API_USER2);
- $user3 = $this->userManager->get(self::TEST_FILES_SHARING_API_USER3);
- $testGroup->addUser($user1);
- $testGroup->addUser($user2);
- $testGroup->addUser($user3);
- // Share item with group
- $share = $this->share(
- \OCP\Share::SHARE_TYPE_GROUP,
- $path,
- self::TEST_FILES_SHARING_API_USER1,
- 'testGroup',
- $beforePerm
- );
- $this->shareManager->acceptShare($share, $user1->getUID());
- $this->shareManager->acceptShare($share, $user2->getUID());
- $this->shareManager->acceptShare($share, $user3->getUID());
- // Login as user 2 and verify the item exists
- self::loginHelper(self::TEST_FILES_SHARING_API_USER2);
- $this->assertTrue(\OC\Files\Filesystem::file_exists($path)); // TODO: unreliable - this is sometimes false
- $result = $this->shareManager->getShareById($share->getFullId(), self::TEST_FILES_SHARING_API_USER2);
- $this->assertEquals($beforePerm, $result->getPermissions());
- // Now move the item forcing a new entry in the share table
- \OC\Files\Filesystem::rename($path, 'newPath');
- $this->assertTrue(\OC\Files\Filesystem::file_exists('newPath'));
- $this->assertFalse(\OC\Files\Filesystem::file_exists($path));
- // change permissions
- $share->setPermissions($afterPerm);
- $this->shareManager->updateShare($share);
- // Login as user 3 and verify that the permissions are changed
- self::loginHelper(self::TEST_FILES_SHARING_API_USER3);
- $result = $this->shareManager->getShareById($share->getFullId(), self::TEST_FILES_SHARING_API_USER3);
- $this->assertNotEmpty($result);
- $this->assertEquals($afterPerm, $result->getPermissions());
- // Login as user 2 and verify that the permissions are changed
- self::loginHelper(self::TEST_FILES_SHARING_API_USER2);
- $result = $this->shareManager->getShareById($share->getFullId(), self::TEST_FILES_SHARING_API_USER2);
- $this->assertNotEmpty($result);
- $this->assertEquals($afterPerm, $result->getPermissions());
- $this->assertEquals('/newPath', $result->getTarget());
- //cleanup
- self::loginHelper(self::TEST_FILES_SHARING_API_USER1);
- $this->shareManager->deleteShare($share);
- $testGroup->removeUser($user1);
- $testGroup->removeUser($user2);
- $testGroup->removeUser($user3);
- }
- /**
- * If the permissions on a group share are upgraded be sure to still respect
- * removed shares by a member of that group
- */
- public function testPermissionUpgradeOnUserDeletedGroupShare() {
- $testGroup = $this->groupManager->createGroup('testGroup');
- $user1 = $this->userManager->get(self::TEST_FILES_SHARING_API_USER1);
- $user2 = $this->userManager->get(self::TEST_FILES_SHARING_API_USER2);
- $user3 = $this->userManager->get(self::TEST_FILES_SHARING_API_USER3);
- $testGroup->addUser($user1);
- $testGroup->addUser($user2);
- $testGroup->addUser($user3);
- $connection = \OC::$server->getDatabaseConnection();
- // Share item with group
- $fileinfo = $this->view->getFileInfo($this->folder);
- $share = $this->share(
- \OCP\Share::SHARE_TYPE_GROUP,
- $this->folder,
- self::TEST_FILES_SHARING_API_USER1,
- 'testGroup',
- \OCP\Constants::PERMISSION_READ
- );
- $this->shareManager->acceptShare($share, $user1->getUID());
- $this->shareManager->acceptShare($share, $user2->getUID());
- $this->shareManager->acceptShare($share, $user3->getUID());
- // Login as user 2 and verify the item exists
- self::loginHelper(self::TEST_FILES_SHARING_API_USER2);
- $this->assertTrue(\OC\Files\Filesystem::file_exists($this->folder));
- $result = $this->shareManager->getShareById($share->getFullId(), self::TEST_FILES_SHARING_API_USER2);
- $this->assertNotEmpty($result);
- $this->assertEquals(\OCP\Constants::PERMISSION_READ, $result->getPermissions());
- // Delete the share
- $this->assertTrue(\OC\Files\Filesystem::rmdir($this->folder));
- $this->assertFalse(\OC\Files\Filesystem::file_exists($this->folder));
- // Verify we do not get a share
- $result = $this->shareManager->getShareById($share->getFullId(), self::TEST_FILES_SHARING_API_USER2);
- $this->assertEquals(0, $result->getPermissions());
- // Login as user 1 again and change permissions
- self::loginHelper(self::TEST_FILES_SHARING_API_USER1);
- $share->setPermissions(\OCP\Constants::PERMISSION_ALL);
- $share = $this->shareManager->updateShare($share);
- // Login as user 2 and verify
- self::loginHelper(self::TEST_FILES_SHARING_API_USER2);
- $this->assertFalse(\OC\Files\Filesystem::file_exists($this->folder));
- $result = $this->shareManager->getShareById($share->getFullId(), self::TEST_FILES_SHARING_API_USER2);
- $this->assertEquals(0, $result->getPermissions());
- $this->shareManager->deleteShare($share);
- //cleanup
- self::loginHelper(self::TEST_FILES_SHARING_API_USER1);
- $testGroup->removeUser($user1);
- $testGroup->removeUser($user2);
- $testGroup->removeUser($user3);
- }
- }
- class DummyTestClassSharedMount extends \OCA\Files_Sharing\SharedMount {
- public function __construct($storage, $mountpoint, $arguments = null, $loader = null){
- // noop
- }
- public function stripUserFilesPathDummy($path) {
- return $this->stripUserFilesPath($path);
- }
- }
|