123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460461462463464465466467468469470471472473474475476477478479480481482483484 |
- <?php
- /**
- * SPDX-FileCopyrightText: 2016-2024 Nextcloud GmbH and Nextcloud contributors
- * SPDX-FileCopyrightText: 2016 ownCloud, Inc.
- * SPDX-License-Identifier: AGPL-3.0-only
- */
- namespace OCA\DAV\Tests\Unit\Connector\Sabre;
- use OC\Files\FileInfo;
- use OC\Files\Filesystem;
- use OC\Files\Node\Node;
- use OC\Files\Storage\Wrapper\Quota;
- use OC\Files\View;
- use OCA\DAV\Connector\Sabre\Directory;
- use OCA\DAV\Connector\Sabre\Exception\Forbidden;
- use OCA\DAV\Connector\Sabre\Exception\InvalidPath;
- use OCP\Constants;
- use OCP\Files\ForbiddenException;
- use OCP\Files\InvalidPathException;
- use OCP\Files\Mount\IMountPoint;
- use OCP\Files\StorageNotAvailableException;
- use Test\Traits\UserTrait;
- class TestViewDirectory extends View {
- public function __construct(
- private $updatables,
- private $deletables,
- private $canRename = true,
- ) {
- }
- public function isUpdatable($path) {
- return $this->updatables[$path];
- }
- public function isCreatable($path) {
- return $this->updatables[$path];
- }
- public function isDeletable($path) {
- return $this->deletables[$path];
- }
- public function rename($path1, $path2) {
- return $this->canRename;
- }
- public function getRelativePath($path): ?string {
- return $path;
- }
- }
- /**
- * @group DB
- */
- class DirectoryTest extends \Test\TestCase {
- use UserTrait;
- /** @var View|\PHPUnit\Framework\MockObject\MockObject */
- private $view;
- /** @var FileInfo|\PHPUnit\Framework\MockObject\MockObject */
- private $info;
- protected function setUp(): void {
- parent::setUp();
- $this->view = $this->createMock('OC\Files\View');
- $this->info = $this->createMock('OC\Files\FileInfo');
- $this->info->method('isReadable')
- ->willReturn(true);
- $this->info->method('getType')
- ->willReturn(Node::TYPE_FOLDER);
- $this->info->method('getName')
- ->willReturn('folder');
- $this->info->method('getPath')
- ->willReturn('/admin/files/folder');
- $this->info->method('getPermissions')
- ->willReturn(Constants::PERMISSION_READ);
- }
- private function getDir($path = '/') {
- $this->view->expects($this->once())
- ->method('getRelativePath')
- ->willReturn($path);
- $this->info->expects($this->once())
- ->method('getPath')
- ->willReturn($path);
- return new Directory($this->view, $this->info);
- }
- public function testDeleteRootFolderFails(): void {
- $this->expectException(\Sabre\DAV\Exception\Forbidden::class);
- $this->info->expects($this->any())
- ->method('isDeletable')
- ->willReturn(true);
- $this->view->expects($this->never())
- ->method('rmdir');
- $dir = $this->getDir();
- $dir->delete();
- }
- public function testDeleteForbidden(): void {
- $this->expectException(Forbidden::class);
- // deletion allowed
- $this->info->expects($this->once())
- ->method('isDeletable')
- ->willReturn(true);
- // but fails
- $this->view->expects($this->once())
- ->method('rmdir')
- ->with('sub')
- ->willThrowException(new ForbiddenException('', true));
- $dir = $this->getDir('sub');
- $dir->delete();
- }
- public function testDeleteFolderWhenAllowed(): void {
- // deletion allowed
- $this->info->expects($this->once())
- ->method('isDeletable')
- ->willReturn(true);
- // but fails
- $this->view->expects($this->once())
- ->method('rmdir')
- ->with('sub')
- ->willReturn(true);
- $dir = $this->getDir('sub');
- $dir->delete();
- }
- public function testDeleteFolderFailsWhenNotAllowed(): void {
- $this->expectException(\Sabre\DAV\Exception\Forbidden::class);
- $this->info->expects($this->once())
- ->method('isDeletable')
- ->willReturn(false);
- $dir = $this->getDir('sub');
- $dir->delete();
- }
- public function testDeleteFolderThrowsWhenDeletionFailed(): void {
- $this->expectException(\Sabre\DAV\Exception\Forbidden::class);
- // deletion allowed
- $this->info->expects($this->once())
- ->method('isDeletable')
- ->willReturn(true);
- // but fails
- $this->view->expects($this->once())
- ->method('rmdir')
- ->with('sub')
- ->willReturn(false);
- $dir = $this->getDir('sub');
- $dir->delete();
- }
- public function testGetChildren(): void {
- $info1 = $this->getMockBuilder(FileInfo::class)
- ->disableOriginalConstructor()
- ->getMock();
- $info2 = $this->getMockBuilder(FileInfo::class)
- ->disableOriginalConstructor()
- ->getMock();
- $info1->method('getName')
- ->willReturn('first');
- $info1->method('getPath')
- ->willReturn('folder/first');
- $info1->method('getEtag')
- ->willReturn('abc');
- $info2->method('getName')
- ->willReturn('second');
- $info2->method('getPath')
- ->willReturn('folder/second');
- $info2->method('getEtag')
- ->willReturn('def');
- $this->view->expects($this->once())
- ->method('getDirectoryContent')
- ->willReturn([$info1, $info2]);
- $this->view->expects($this->any())
- ->method('getRelativePath')
- ->willReturnCallback(function ($path) {
- return str_replace('/admin/files/', '', $path);
- });
- $this->view->expects($this->any())
- ->method('getAbsolutePath')
- ->willReturnCallback(function ($path) {
- return Filesystem::normalizePath('/admin/files' . $path);
- });
- $this->overwriteService(View::class, $this->view);
- $dir = new Directory($this->view, $this->info);
- $nodes = $dir->getChildren();
- $this->assertEquals(2, count($nodes));
- // calling a second time just returns the cached values,
- // does not call getDirectoryContents again
- $dir->getChildren();
- }
- public function testGetChildrenNoPermission(): void {
- $this->expectException(\Sabre\DAV\Exception\Forbidden::class);
- $info = $this->createMock(FileInfo::class);
- $info->expects($this->any())
- ->method('isReadable')
- ->willReturn(false);
- $dir = new Directory($this->view, $info);
- $dir->getChildren();
- }
- public function testGetChildNoPermission(): void {
- $this->expectException(\Sabre\DAV\Exception\NotFound::class);
- $this->info->expects($this->any())
- ->method('isReadable')
- ->willReturn(false);
- $dir = new Directory($this->view, $this->info);
- $dir->getChild('test');
- }
- public function testGetChildThrowStorageNotAvailableException(): void {
- $this->expectException(\Sabre\DAV\Exception\ServiceUnavailable::class);
- $this->view->expects($this->once())
- ->method('getFileInfo')
- ->willThrowException(new StorageNotAvailableException());
- $dir = new Directory($this->view, $this->info);
- $dir->getChild('.');
- }
- public function testGetChildThrowInvalidPath(): void {
- $this->expectException(InvalidPath::class);
- $this->view->expects($this->once())
- ->method('verifyPath')
- ->willThrowException(new InvalidPathException());
- $this->view->expects($this->never())
- ->method('getFileInfo');
- $dir = new Directory($this->view, $this->info);
- $dir->getChild('.');
- }
- public function testGetQuotaInfoUnlimited(): void {
- self::createUser('user', 'password');
- self::loginAsUser('user');
- $mountPoint = $this->createMock(IMountPoint::class);
- $storage = $this->getMockBuilder(Quota::class)
- ->disableOriginalConstructor()
- ->getMock();
- $mountPoint->method('getStorage')
- ->willReturn($storage);
- $storage->expects($this->any())
- ->method('instanceOfStorage')
- ->willReturnMap([
- ['\OCA\Files_Sharing\SharedStorage', false],
- ['\OC\Files\Storage\Wrapper\Quota', false],
- ]);
- $storage->expects($this->once())
- ->method('getOwner')
- ->willReturn('user');
- $storage->expects($this->never())
- ->method('getQuota');
- $storage->expects($this->once())
- ->method('free_space')
- ->willReturn(800);
- $this->info->expects($this->any())
- ->method('getPath')
- ->willReturn('/admin/files/foo');
- $this->info->expects($this->once())
- ->method('getSize')
- ->willReturn(200);
- $this->info->expects($this->once())
- ->method('getMountPoint')
- ->willReturn($mountPoint);
- $this->view->expects($this->any())
- ->method('getRelativePath')
- ->willReturn('/foo');
- $this->info->expects($this->once())
- ->method('getInternalPath')
- ->willReturn('/foo');
- $mountPoint->method('getMountPoint')
- ->willReturn('/user/files/mymountpoint');
- $dir = new Directory($this->view, $this->info);
- $this->assertEquals([200, -3], $dir->getQuotaInfo()); //200 used, unlimited
- }
- public function testGetQuotaInfoSpecific(): void {
- self::createUser('user', 'password');
- self::loginAsUser('user');
- $mountPoint = $this->createMock(IMountPoint::class);
- $storage = $this->getMockBuilder(Quota::class)
- ->disableOriginalConstructor()
- ->getMock();
- $mountPoint->method('getStorage')
- ->willReturn($storage);
- $storage->expects($this->any())
- ->method('instanceOfStorage')
- ->willReturnMap([
- ['\OCA\Files_Sharing\SharedStorage', false],
- ['\OC\Files\Storage\Wrapper\Quota', true],
- ]);
- $storage->expects($this->once())
- ->method('getOwner')
- ->willReturn('user');
- $storage->expects($this->once())
- ->method('getQuota')
- ->willReturn(1000);
- $storage->expects($this->once())
- ->method('free_space')
- ->willReturn(800);
- $this->info->expects($this->once())
- ->method('getSize')
- ->willReturn(200);
- $this->info->expects($this->once())
- ->method('getMountPoint')
- ->willReturn($mountPoint);
- $this->info->expects($this->once())
- ->method('getInternalPath')
- ->willReturn('/foo');
- $mountPoint->method('getMountPoint')
- ->willReturn('/user/files/mymountpoint');
- $this->view->expects($this->any())
- ->method('getRelativePath')
- ->willReturn('/foo');
- $dir = new Directory($this->view, $this->info);
- $this->assertEquals([200, 800], $dir->getQuotaInfo()); //200 used, 800 free
- }
- /**
- * @dataProvider moveFailedProvider
- */
- public function testMoveFailed($source, $destination, $updatables, $deletables): void {
- $this->expectException(\Sabre\DAV\Exception\Forbidden::class);
- $this->moveTest($source, $destination, $updatables, $deletables);
- }
- /**
- * @dataProvider moveSuccessProvider
- */
- public function testMoveSuccess($source, $destination, $updatables, $deletables): void {
- $this->moveTest($source, $destination, $updatables, $deletables);
- $this->addToAssertionCount(1);
- }
- /**
- * @dataProvider moveFailedInvalidCharsProvider
- */
- public function testMoveFailedInvalidChars($source, $destination, $updatables, $deletables): void {
- $this->expectException(InvalidPath::class);
- $this->moveTest($source, $destination, $updatables, $deletables);
- }
- public function moveFailedInvalidCharsProvider() {
- return [
- ['a/valid', "a/i\nvalid", ['a' => true, 'a/valid' => true, 'a/c*' => false], []],
- ];
- }
- public function moveFailedProvider() {
- return [
- ['a/b', 'a/c', ['a' => false, 'a/b' => false, 'a/c' => false], []],
- ['a/b', 'b/b', ['a' => false, 'a/b' => false, 'b' => false, 'b/b' => false], []],
- ['a/b', 'b/b', ['a' => false, 'a/b' => true, 'b' => false, 'b/b' => false], []],
- ['a/b', 'b/b', ['a' => true, 'a/b' => true, 'b' => false, 'b/b' => false], []],
- ['a/b', 'b/b', ['a' => true, 'a/b' => true, 'b' => true, 'b/b' => false], ['a/b' => false]],
- ['a/b', 'a/c', ['a' => false, 'a/b' => true, 'a/c' => false], []],
- ];
- }
- public function moveSuccessProvider() {
- return [
- ['a/b', 'b/b', ['a' => true, 'a/b' => true, 'b' => true, 'b/b' => false], ['a/b' => true]],
- // older files with special chars can still be renamed to valid names
- ['a/b*', 'b/b', ['a' => true, 'a/b*' => true, 'b' => true, 'b/b' => false], ['a/b*' => true]],
- ];
- }
- /**
- * @param $source
- * @param $destination
- * @param $updatables
- */
- private function moveTest($source, $destination, $updatables, $deletables): void {
- $view = new TestViewDirectory($updatables, $deletables);
- $sourceInfo = new FileInfo($source, null, null, [
- 'type' => FileInfo::TYPE_FOLDER,
- ], null);
- $targetInfo = new FileInfo(dirname($destination), null, null, [
- 'type' => FileInfo::TYPE_FOLDER,
- ], null);
- $sourceNode = new Directory($view, $sourceInfo);
- $targetNode = $this->getMockBuilder(Directory::class)
- ->setMethods(['childExists'])
- ->setConstructorArgs([$view, $targetInfo])
- ->getMock();
- $targetNode->expects($this->any())->method('childExists')
- ->with(basename($destination))
- ->willReturn(false);
- $this->assertTrue($targetNode->moveInto(basename($destination), $source, $sourceNode));
- }
- public function testFailingMove(): void {
- $this->expectException(\Sabre\DAV\Exception\Forbidden::class);
- $this->expectExceptionMessage('Could not copy directory b, target exists');
- $source = 'a/b';
- $destination = 'c/b';
- $updatables = ['a' => true, 'a/b' => true, 'b' => true, 'c/b' => false];
- $deletables = ['a/b' => true];
- $view = new TestViewDirectory($updatables, $deletables);
- $sourceInfo = new FileInfo($source, null, null, ['type' => FileInfo::TYPE_FOLDER], null);
- $targetInfo = new FileInfo(dirname($destination), null, null, ['type' => FileInfo::TYPE_FOLDER], null);
- $sourceNode = new Directory($view, $sourceInfo);
- $targetNode = $this->getMockBuilder(Directory::class)
- ->onlyMethods(['childExists'])
- ->setConstructorArgs([$view, $targetInfo])
- ->getMock();
- $targetNode->expects($this->once())->method('childExists')
- ->with(basename($destination))
- ->willReturn(true);
- $targetNode->moveInto(basename($destination), $source, $sourceNode);
- }
- }
|