mbk-lab
/
nextcloud
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Fix sabre directory test cases 

- rely on a mock subfolder instead of the root
- remove obsolete "Shared" limitation tests which did pass but not for the right
  reasons
- added test for the prevention of root deletion
This commit is contained in:
Vincent Petry 2015-06-05 19:04:59 +02:00
parent 6ae5ae2e31
commit b97be0ea02
1 changed files with 15 additions and 24 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

39
tests/lib/connector/sabre/directory.php
View File

@ -20,14 +20,14 @@ class Test_OC_Connector_Sabre_Directory extends \Test\TestCase {
$this->info = $this->getMock('OC\Files\FileInfo', array(), array(), '', false);
}
private function getRootDir() {
private function getDir($path = '/') {
$this->view->expects($this->once())
->method('getRelativePath')
->will($this->returnValue(''));
->will($this->returnValue($path));
$this->info->expects($this->once())
->method('getPath')
->will($this->returnValue(''));
->will($this->returnValue($path));
return new \OC\Connector\Sabre\Directory($this->view, $this->info);
}
@ -35,24 +35,13 @@ class Test_OC_Connector_Sabre_Directory extends \Test\TestCase {
/**
* @expectedException \Sabre\DAV\Exception\Forbidden
*/
public function testCreateSharedFileFails() {
$dir = $this->getRootDir();
$dir->createFile('Shared');
}
/**
* @expectedException \Sabre\DAV\Exception\Forbidden
*/
public function testCreateSharedFolderFails() {
$dir = $this->getRootDir();
$dir->createDirectory('Shared');
}
/**
* @expectedException \Sabre\DAV\Exception\Forbidden
*/
public function testDeleteSharedFolderFails() {
$dir = $this->getRootDir();
public function testDeleteRootFolderFails() {
$this->info->expects($this->any())
->method('isDeletable')
->will($this->returnValue(true));
$this->view->expects($this->never())
->method('rmdir');
$dir = $this->getDir();
$dir->delete();
}
@ -68,9 +57,10 @@ class Test_OC_Connector_Sabre_Directory extends \Test\TestCase {
// but fails
$this->view->expects($this->once())
->method('rmdir')
->with('sub')
->will($this->returnValue(true));
$dir = $this->getRootDir();
$dir = $this->getDir('sub');
$dir->delete();
}
@ -82,7 +72,7 @@ class Test_OC_Connector_Sabre_Directory extends \Test\TestCase {
->method('isDeletable')
->will($this->returnValue(false));
$dir = $this->getRootDir();
$dir = $this->getDir('sub');
$dir->delete();
}
@ -98,9 +88,10 @@ class Test_OC_Connector_Sabre_Directory extends \Test\TestCase {
// but fails
$this->view->expects($this->once())
->method('rmdir')
->with('sub')
->will($this->returnValue(false));
$dir = $this->getRootDir();
$dir = $this->getDir('sub');
$dir->delete();
}