RootTest does not require DB

* Removed from DB group
* Also mock the manager
* Fixed deprecated getMock warnings
This commit is contained in:
Roeland Jago Douma 2016-07-13 14:35:34 +02:00
parent 77642535cd
commit 927be847b9
No known key found for this signature in database
GPG Key ID: 1E152838F164D13B
1 changed files with 39 additions and 20 deletions

View File

@ -9,18 +9,29 @@
namespace Test\Files\Node; namespace Test\Files\Node;
use OC\Files\FileInfo; use OC\Files\FileInfo;
use OCP\Files\NotPermittedException;
use OC\Files\Mount\Manager;
/**
* @group DB
*/
class RootTest extends \Test\TestCase { class RootTest extends \Test\TestCase {
/** @var \OC\User\User */
private $user; private $user;
/** @var \OC\Files\Mount\Manager */
private $manager;
protected function setUp() { protected function setUp() {
parent::setUp(); parent::setUp();
$this->user = new \OC\User\User('', new \Test\Util\User\Dummy);
$config = $this->getMockBuilder('\OCP\IConfig')
->disableOriginalConstructor()
->getMock();
$urlgenerator = $this->getMockBuilder('\OCP\IURLGenerator')
->disableOriginalConstructor()
->getMock();
$this->user = new \OC\User\User('', new \Test\Util\User\Dummy, null, $config, $urlgenerator);
$this->manager = $this->getMockBuilder('\OC\Files\Mount\Manager')
->disableOriginalConstructor()
->getMock();
} }
protected function getFileInfo($data) { protected function getFileInfo($data) {
@ -28,16 +39,19 @@ class RootTest extends \Test\TestCase {
} }
public function testGet() { public function testGet() {
$manager = new Manager();
/** /**
* @var \OC\Files\Storage\Storage $storage * @var \OC\Files\Storage\Storage $storage
*/ */
$storage = $this->getMock('\OC\Files\Storage\Storage'); $storage = $this->getMockBuilder('\OC\Files\Storage\Storage')
->disableOriginalConstructor()
->getMock();
/** /**
* @var \OC\Files\View | \PHPUnit_Framework_MockObject_MockObject $view * @var \OC\Files\View | \PHPUnit_Framework_MockObject_MockObject $view
*/ */
$view = $this->getMock('\OC\Files\View'); $view = $this->getMockBuilder('\OC\Files\View')
$root = new \OC\Files\Node\Root($manager, $view, $this->user); ->disableOriginalConstructor()
->getMock();
$root = new \OC\Files\Node\Root($this->manager, $view, $this->user);
$view->expects($this->once()) $view->expects($this->once())
->method('getFileInfo') ->method('getFileInfo')
@ -54,16 +68,19 @@ class RootTest extends \Test\TestCase {
* @expectedException \OCP\Files\NotFoundException * @expectedException \OCP\Files\NotFoundException
*/ */
public function testGetNotFound() { public function testGetNotFound() {
$manager = new Manager();
/** /**
* @var \OC\Files\Storage\Storage $storage * @var \OC\Files\Storage\Storage $storage
*/ */
$storage = $this->getMock('\OC\Files\Storage\Storage'); $storage = $this->getMockBuilder('\OC\Files\Storage\Storage')
->disableOriginalConstructor()
->getMock();
/** /**
* @var \OC\Files\View | \PHPUnit_Framework_MockObject_MockObject $view * @var \OC\Files\View | \PHPUnit_Framework_MockObject_MockObject $view
*/ */
$view = $this->getMock('\OC\Files\View'); $view = $this->getMockBuilder('\OC\Files\View')
$root = new \OC\Files\Node\Root($manager, $view, $this->user); ->disableOriginalConstructor()
->getMock();
$root = new \OC\Files\Node\Root($this->manager, $view, $this->user);
$view->expects($this->once()) $view->expects($this->once())
->method('getFileInfo') ->method('getFileInfo')
@ -78,12 +95,13 @@ class RootTest extends \Test\TestCase {
* @expectedException \OCP\Files\NotPermittedException * @expectedException \OCP\Files\NotPermittedException
*/ */
public function testGetInvalidPath() { public function testGetInvalidPath() {
$manager = new Manager();
/** /**
* @var \OC\Files\View | \PHPUnit_Framework_MockObject_MockObject $view * @var \OC\Files\View | \PHPUnit_Framework_MockObject_MockObject $view
*/ */
$view = $this->getMock('\OC\Files\View'); $view = $this->getMockBuilder('\OC\Files\View')
$root = new \OC\Files\Node\Root($manager, $view, $this->user); ->disableOriginalConstructor()
->getMock();
$root = new \OC\Files\Node\Root($this->manager, $view, $this->user);
$root->get('/../foo'); $root->get('/../foo');
} }
@ -92,12 +110,13 @@ class RootTest extends \Test\TestCase {
* @expectedException \OCP\Files\NotFoundException * @expectedException \OCP\Files\NotFoundException
*/ */
public function testGetNoStorages() { public function testGetNoStorages() {
$manager = new Manager();
/** /**
* @var \OC\Files\View | \PHPUnit_Framework_MockObject_MockObject $view * @var \OC\Files\View | \PHPUnit_Framework_MockObject_MockObject $view
*/ */
$view = $this->getMock('\OC\Files\View'); $view = $this->getMockBuilder('\OC\Files\View')
$root = new \OC\Files\Node\Root($manager, $view, $this->user); ->disableOriginalConstructor()
->getMock();
$root = new \OC\Files\Node\Root($this->manager, $view, $this->user);
$root->get('/bar/foo'); $root->get('/bar/foo');
} }