3 files changed, 22 insertions, 22 deletions

diff --git a/apps/files_sharing/tests/Controllers/ExternalShareControllerTest.php b/apps/files_sharing/tests/Controller/ExternalShareControllerTest.php
index e5fd739c4fa..71a4f4adeba 100644
--- a/apps/files_sharing/tests/Controllers/ExternalShareControllerTest.php
+++ b/apps/files_sharing/tests/Controller/ExternalShareControllerTest.php
@@ -24,7 +24,7 @@
 
 namespace OCA\Files_Sharing\Tests\Controllers;
 
-use OCA\Files_Sharing\Controllers\ExternalSharesController;
+use OCA\Files_Sharing\Controller\ExternalSharesController;
 use OCP\AppFramework\Http\DataResponse;
 use OCP\AppFramework\Http\JSONResponse;
 use OCP\Http\Client\IClientService;
diff --git a/apps/files_sharing/tests/Controllers/ShareControllerTest.php b/apps/files_sharing/tests/Controller/ShareControllerTest.php
index 94891d868c2..dae7740b3ae 100644
--- a/apps/files_sharing/tests/Controllers/ShareControllerTest.php
+++ b/apps/files_sharing/tests/Controller/ShareControllerTest.php
@@ -33,7 +33,7 @@ namespace OCA\Files_Sharing\Tests\Controllers;
 
 use OC\Files\Filesystem;
 use OCA\FederatedFileSharing\FederatedShareProvider;
-use OCA\Files_Sharing\Controllers\ShareController;
+use OCA\Files_Sharing\Controller\ShareController;
 use OCP\AppFramework\Http\DataResponse;
 use OCP\Share\Exceptions\ShareNotFound;
 use OCP\AppFramework\Http\NotFoundResponse;
@@ -96,7 +96,7 @@ class ShareControllerTest extends \Test\TestCase {
 			->method('isIncomingServer2serverShareEnabled')->willReturn(true);
 		$this->eventDispatcher = $this->getMockBuilder('Symfony\Component\EventDispatcher\EventDispatcherInterface')->getMock();
 
-		$this->shareController = new \OCA\Files_Sharing\Controllers\ShareController(
+		$this->shareController = new \OCA\Files_Sharing\Controller\ShareController(
 			$this->appName,
 			$this->getMockBuilder('\OCP\IRequest')->getMock(),
 			$this->config,
diff --git a/apps/files_sharing/tests/Middleware/SharingCheckMiddlewareTest.php b/apps/files_sharing/tests/Middleware/SharingCheckMiddlewareTest.php
index 2323317f66d..c2965d04b6b 100644
--- a/apps/files_sharing/tests/Middleware/SharingCheckMiddlewareTest.php
+++ b/apps/files_sharing/tests/Middleware/SharingCheckMiddlewareTest.php
@@ -24,37 +24,40 @@
  */
 
 namespace OCA\Files_Sharing\Middleware;
+use OCA\Files_Sharing\Controller\ExternalSharesController;
+use OCA\Files_Sharing\Controller\ShareController;
+use OCP\App\IAppManager;
+use OCP\AppFramework\Controller;
 use OCP\AppFramework\Http\NotFoundResponse;
 use OCP\Files\NotFoundException;
 use OCP\AppFramework\Utility\IControllerMethodReflector;
 use OCA\Files_Sharing\Exceptions\S2SException;
 use OCP\AppFramework\Http\JSONResponse;
+use OCP\IConfig;
 
 /**
  * @package OCA\Files_Sharing\Middleware\SharingCheckMiddleware
  */
 class SharingCheckMiddlewareTest extends \Test\TestCase {
 
-	/** @var \OCP\IConfig */
+	/** @var IConfig|\PHPUnit_Framework_MockObject_MockObject */
 	private $config;
-	/** @var \OCP\App\IAppManager */
+	/** @var IAppManager|\PHPUnit_Framework_MockObject_MockObject */
 	private $appManager;
 	/** @var SharingCheckMiddleware */
 	private $sharingCheckMiddleware;
-	/** @var \OCP\AppFramework\Controller */
+	/** @var Controller|\PHPUnit_Framework_MockObject_MockObject */
 	private $controllerMock;
-	/** @var IControllerMethodReflector */
+	/** @var IControllerMethodReflector|\PHPUnit_Framework_MockObject_MockObject */
 	private $reflector;
 
 	protected function setUp() {
-		$this->config = $this->getMockBuilder('\OCP\IConfig')
-			->disableOriginalConstructor()->getMock();
-		$this->appManager = $this->getMockBuilder('\OCP\App\IAppManager')
-			->disableOriginalConstructor()->getMock();
-		$this->controllerMock = $this->getMockBuilder('\OCP\AppFramework\Controller')
-			->disableOriginalConstructor()->getMock();
-		$this->reflector = $this->getMockBuilder('\OCP\AppFramework\Utility\IControllerMethodReflector')
-			->disableOriginalConstructor()->getMock();
+		parent::setUp();
+
+		$this->config = $this->createMock(IConfig::class);
+		$this->appManager = $this->createMock(IAppManager::class);
+		$this->controllerMock = $this->createMock(Controller::class);
+		$this->reflector = $this->createMock(IControllerMethodReflector::class);
 
 		$this->sharingCheckMiddleware = new SharingCheckMiddleware(
 			'files_sharing',
@@ -198,14 +201,13 @@ class SharingCheckMiddlewareTest extends \Test\TestCase {
 			->method('getAppValue')
 			->will($this->returnValueMap($config));
 
-		$controller = $this->getMockBuilder('\OCA\Files_Sharing\Controllers\ExternalSharesController')
-			->disableOriginalConstructor()->getMock();
+		$controller = $this->createMock(ExternalSharesController::class);
 
 		$exceptionThrown = false;
 
 		try {
 			$this->sharingCheckMiddleware->beforeController($controller, 'myMethod');
-		} catch (\OCA\Files_Sharing\Exceptions\S2SException $exception) {
+		} catch (S2SException $exception) {
 			$exceptionThrown = true;
 		}
 
@@ -231,8 +233,7 @@ class SharingCheckMiddlewareTest extends \Test\TestCase {
 			->with('core', 'shareapi_allow_links', 'yes')
 			->will($this->returnValue('yes'));
 
-		$controller = $this->getMockBuilder('\OCA\Files_Sharing\Controllers\ShareController')
-			->disableOriginalConstructor()->getMock();
+		$controller = $this->createMock(ShareController::class);
 
 		$this->sharingCheckMiddleware->beforeController($controller, 'myMethod');
 	}
@@ -248,8 +249,7 @@ class SharingCheckMiddlewareTest extends \Test\TestCase {
 			->with('files_sharing')
 			->will($this->returnValue(true));
 
-		$controller = $this->getMockBuilder('\OCA\Files_Sharing\Controllers\ShareController')
-			->disableOriginalConstructor()->getMock();
+		$controller = $this->createMock(ShareController::class);
 
 		$this->sharingCheckMiddleware->beforeController($controller, 'myMethod');
 	}