Fix usage of deprecated getMock

This is deprecated in phpunit >= 5.4
This generates a whole lot of output. Which causes drone to kill the
tests

apps/dav/tests/unit/Connector/Sabre/BlockLegacyClientPluginTest.php

@@ -41,7 +41,7 @@ class BlockLegacyClientPluginTest extends TestCase {
 	public function setUp() {
 		parent::setUp();
 
-		$this->config = $this->getMock('\OCP\IConfig');
+		$this->config = $this->createMock('\OCP\IConfig');
 		$this->blockLegacyClientVersionPlugin = new BlockLegacyClientPlugin($this->config);
 	}
 
@@ -66,7 +66,7 @@ public function oldDesktopClientProvider() {
 	 */
 	public function testBeforeHandlerException($userAgent) {
 		/** @var \Sabre\HTTP\RequestInterface $request */
-		$request = $this->getMock('\Sabre\HTTP\RequestInterface');
+		$request = $this->createMock('\Sabre\HTTP\RequestInterface');
 		$request
 			->expects($this->once())
 			->method('getHeader')
@@ -101,7 +101,7 @@ public function newAndAlternateDesktopClientProvider() {
 	 */
 	public function testBeforeHandlerSuccess($userAgent) {
 		/** @var \Sabre\HTTP\RequestInterface $request */
-		$request = $this->getMock('\Sabre\HTTP\RequestInterface');
+		$request = $this->createMock('\Sabre\HTTP\RequestInterface');
 		$request
 			->expects($this->once())
 			->method('getHeader')
@@ -119,7 +119,7 @@ public function testBeforeHandlerSuccess($userAgent) {
 
 	public function testBeforeHandlerNoUserAgent() {
 		/** @var \Sabre\HTTP\RequestInterface $request */
-		$request = $this->getMock('\Sabre\HTTP\RequestInterface');
+		$request = $this->createMock('\Sabre\HTTP\RequestInterface');
 		$request
 			->expects($this->once())
 			->method('getHeader')

apps/dav/tests/unit/Connector/Sabre/FilesPluginTest.php

@@ -89,11 +89,11 @@ public function setUp() {
 		$this->view = $this->getMockBuilder('\OC\Files\View')
 			->disableOriginalConstructor()
 			->getMock();
-		$this->config = $this->getMock('\OCP\IConfig');
+		$this->config = $this->createMock('\OCP\IConfig');
 		$this->config->expects($this->any())->method('getSystemValue')
 			->with($this->equalTo('data-fingerprint'), $this->equalTo(''))
 			->willReturn('my_fingerprint');
-		$this->request = $this->getMock('\OCP\IRequest');
+		$this->request = $this->createMock('\OCP\IRequest');
 
 		$this->plugin = new FilesPlugin(
 			$this->tree,
@@ -275,7 +275,7 @@ public function testGetPublicPermissions() {
 			$this->tree,
 			$this->view,
 			$this->config,
-			$this->getMock('\OCP\IRequest'),
+			$this->createMock('\OCP\IRequest'),
 			true);
 		$this->plugin->initialize($this->server);
 

apps/dav/tests/unit/Connector/Sabre/FilesReportPluginTest.php

@@ -82,11 +82,11 @@ public function setUp() {
 			->disableOriginalConstructor()
 			->getMock();
 
-		$this->tagManager = $this->getMock('\OCP\SystemTag\ISystemTagManager');
-		$this->tagMapper = $this->getMock('\OCP\SystemTag\ISystemTagObjectMapper');
-		$this->userSession = $this->getMock('\OCP\IUserSession');
+		$this->tagManager = $this->createMock('\OCP\SystemTag\ISystemTagManager');
+		$this->tagMapper = $this->createMock('\OCP\SystemTag\ISystemTagObjectMapper');
+		$this->userSession = $this->createMock('\OCP\IUserSession');
 
-		$user = $this->getMock('\OCP\IUser');
+		$user = $this->createMock('\OCP\IUser');
 		$user->expects($this->any())
 			->method('getUID')
 			->will($this->returnValue('testuser'));
@@ -114,7 +114,7 @@ public function testOnReportInvalidNode() {
 		$this->tree->expects($this->any())
 			->method('getNodeForPath')
 			->with('/' . $path)
-			->will($this->returnValue($this->getMock('\Sabre\DAV\INode')));
+			->will($this->returnValue($this->createMock('\Sabre\DAV\INode')));
 
 		$this->server->expects($this->any())
 			->method('getRequestUri')
@@ -133,7 +133,7 @@ public function testOnReportInvalidReportName() {
 		$this->tree->expects($this->any())
 			->method('getNodeForPath')
 			->with('/' . $path)
-			->will($this->returnValue($this->getMock('\Sabre\DAV\INode')));
+			->will($this->returnValue($this->createMock('\Sabre\DAV\INode')));
 
 		$this->server->expects($this->any())
 			->method('getRequestUri')
@@ -337,14 +337,14 @@ public function testPrepareResponses() {
 			->method('getSize')
 			->will($this->returnValue(1024));
 
-		$config = $this->getMock('\OCP\IConfig');
+		$config = $this->createMock('\OCP\IConfig');
 
 		$this->server->addPlugin(
 			new \OCA\DAV\Connector\Sabre\FilesPlugin(
 				$this->tree,
 				$this->view,
 				$config,
-				$this->getMock('\OCP\IRequest')
+				$this->createMock('\OCP\IRequest')
 			)
 		);
 		$this->plugin->initialize($this->server);
@@ -494,15 +494,15 @@ public function testProcessFilterRulesInvisibleTagAsAdmin() {
 			->method('isAdmin')
 			->will($this->returnValue(true));
 
-		$tag1 = $this->getMock('\OCP\SystemTag\ISystemTag');
+		$tag1 = $this->createMock('\OCP\SystemTag\ISystemTag');
 		$tag1->expects($this->any())
 			->method('getId')
 			->will($this->returnValue('123'));
 		$tag1->expects($this->any())
 			->method('isUserVisible')
 			->will($this->returnValue(true));
 
-		$tag2 = $this->getMock('\OCP\SystemTag\ISystemTag');
+		$tag2 = $this->createMock('\OCP\SystemTag\ISystemTag');
 		$tag2->expects($this->any())
 			->method('getId')
 			->will($this->returnValue('123'));
@@ -539,15 +539,15 @@ public function testProcessFilterRulesInvisibleTagAsUser() {
 			->method('isAdmin')
 			->will($this->returnValue(false));
 
-		$tag1 = $this->getMock('\OCP\SystemTag\ISystemTag');
+		$tag1 = $this->createMock('\OCP\SystemTag\ISystemTag');
 		$tag1->expects($this->any())
 			->method('getId')
 			->will($this->returnValue('123'));
 		$tag1->expects($this->any())
 			->method('isUserVisible')
 			->will($this->returnValue(true));
 
-		$tag2 = $this->getMock('\OCP\SystemTag\ISystemTag');
+		$tag2 = $this->createMock('\OCP\SystemTag\ISystemTag');
 		$tag2->expects($this->any())
 			->method('getId')
 			->will($this->returnValue('123'));
@@ -573,15 +573,15 @@ public function testProcessFilterRulesVisibleTagAsUser() {
 			->method('isAdmin')
 			->will($this->returnValue(false));
 
-		$tag1 = $this->getMock('\OCP\SystemTag\ISystemTag');
+		$tag1 = $this->createMock('\OCP\SystemTag\ISystemTag');
 		$tag1->expects($this->any())
 			->method('getId')
 			->will($this->returnValue('123'));
 		$tag1->expects($this->any())
 			->method('isUserVisible')
 			->will($this->returnValue(true));
 
-		$tag2 = $this->getMock('\OCP\SystemTag\ISystemTag');
+		$tag2 = $this->createMock('\OCP\SystemTag\ISystemTag');
 		$tag2->expects($this->any())
 			->method('getId')
 			->will($this->returnValue('123'));

apps/dav/tests/unit/Connector/Sabre/SharesPluginTest.php

@@ -56,17 +56,17 @@ public function setUp() {
 		$this->tree = $this->getMockBuilder('\Sabre\DAV\Tree')
 			->disableOriginalConstructor()
 			->getMock();
-		$this->shareManager = $this->getMock('\OCP\Share\IManager');
-		$user = $this->getMock('\OCP\IUser');
+		$this->shareManager = $this->createMock('\OCP\Share\IManager');
+		$user = $this->createMock('\OCP\IUser');
 		$user->expects($this->once())
 			->method('getUID')
 			->will($this->returnValue('user1'));
-		$userSession = $this->getMock('\OCP\IUserSession');
+		$userSession = $this->createMock('\OCP\IUserSession');
 		$userSession->expects($this->once())
 			->method('getUser')
 			->will($this->returnValue($user));
 
-		$this->userFolder = $this->getMock('\OCP\Files\Folder');
+		$this->userFolder = $this->createMock('\OCP\Files\Folder');
 
 		$this->plugin = new \OCA\DAV\Connector\Sabre\SharesPlugin(
 			$this->tree,
@@ -92,7 +92,7 @@ public function testGetProperties($shareTypes) {
 			->will($this->returnValue('/subdir'));
 
 		// node API nodes
-		$node = $this->getMock('\OCP\Files\Folder');
+		$node = $this->createMock('\OCP\Files\Folder');
 
 		$this->userFolder->expects($this->once())
 			->method('get')
@@ -168,15 +168,15 @@ public function testPreloadThenGetProperties($shareTypes) {
 			->will($this->returnValue('/subdir'));
 
 		// node API nodes
-		$node = $this->getMock('\OCP\Files\Folder');
+		$node = $this->createMock('\OCP\Files\Folder');
 		$node->expects($this->any())
 			->method('getId')
 			->will($this->returnValue(123));
-		$node1 = $this->getMock('\OCP\Files\File');
+		$node1 = $this->createMock('\OCP\Files\File');
 		$node1->expects($this->any())
 			->method('getId')
 			->will($this->returnValue(111));
-		$node2 = $this->getMock('\OCP\Files\File');
+		$node2 = $this->createMock('\OCP\Files\File');
 		$node2->expects($this->any())
 			->method('getId')
 			->will($this->returnValue(222));

apps/encryption/tests/RecoveryTest.php

@@ -267,13 +267,13 @@ protected function setUp() {
 
 		$this->cryptMock = $this->getMockBuilder('OCA\Encryption\Crypto\Crypt')->disableOriginalConstructor()->getMock();
 		/** @var \OCP\Security\ISecureRandom $randomMock */
-		$randomMock = $this->getMock('OCP\Security\ISecureRandom');
+		$randomMock = $this->createMock('OCP\Security\ISecureRandom');
 		$this->keyManagerMock = $this->getMockBuilder('OCA\Encryption\KeyManager')->disableOriginalConstructor()->getMock();
-		$this->configMock = $this->getMock('OCP\IConfig');
+		$this->configMock = $this->createMock('OCP\IConfig');
 		/** @var \OCP\Encryption\Keys\IStorage $keyStorageMock */
-		$keyStorageMock = $this->getMock('OCP\Encryption\Keys\IStorage');
-		$this->fileMock = $this->getMock('OCP\Encryption\IFile');
-		$this->viewMock = $this->getMock('OC\Files\View');
+		$keyStorageMock = $this->createMock('OCP\Encryption\Keys\IStorage');
+		$this->fileMock = $this->createMock('OCP\Encryption\IFile');
+		$this->viewMock = $this->createMock('OC\Files\View');
 
 		$this->configMock->expects($this->any())
 			->method('setAppValue')

apps/federatedfilesharing/tests/AddressHandlerTest.php

@@ -42,8 +42,8 @@ class AddressHandlerTest extends \Test\TestCase {
 	public function setUp() {
 		parent::setUp();
 
-		$this->urlGenerator = $this->getMock('OCP\IURLGenerator');
-		$this->il10n = $this->getMock('OCP\IL10N');
+		$this->urlGenerator = $this->createMock('OCP\IURLGenerator');
+		$this->il10n = $this->createMock('OCP\IL10N');
 
 		$this->addressHandler = new AddressHandler($this->urlGenerator, $this->il10n);
 	}

apps/federatedfilesharing/tests/DiscoveryManagerTest.php

@@ -38,7 +38,7 @@ class DiscoveryManagerTest extends \Test\TestCase {
 
 	public function setUp() {
 		parent::setUp();
-		$this->cache = $this->getMock('\OCP\ICache');
+		$this->cache = $this->createMock('\OCP\ICache');
 		/** @var ICacheFactory $cacheFactory */
 		$cacheFactory = $this->getMockBuilder('\OCP\ICacheFactory')
 			->disableOriginalConstructor()->getMock();
@@ -65,7 +65,7 @@ public function setUp() {
 	}
 
 	public function testWithMalformedFormattedEndpointCached() {
-		$response = $this->getMock('\OCP\Http\Client\IResponse');
+		$response = $this->createMock('\OCP\Http\Client\IResponse');
 		$response
 			->expects($this->once())
 			->method('getStatusCode')
@@ -102,7 +102,7 @@ public function testWithMalformedFormattedEndpointCached() {
 	}
 
 	public function testGetWebDavEndpointWithValidFormattedEndpointAndNotCached() {
-		$response = $this->getMock('\OCP\Http\Client\IResponse');
+		$response = $this->createMock('\OCP\Http\Client\IResponse');
 		$response
 			->expects($this->once())
 			->method('getStatusCode')
@@ -125,7 +125,7 @@ public function testGetWebDavEndpointWithValidFormattedEndpointAndNotCached() {
 	}
 
 	public function testGetWebDavEndpointWithValidFormattedEndpointWithoutDataAndNotCached() {
-		$response = $this->getMock('\OCP\Http\Client\IResponse');
+		$response = $this->createMock('\OCP\Http\Client\IResponse');
 		$response
 			->expects($this->once())
 			->method('getStatusCode')
@@ -148,7 +148,7 @@ public function testGetWebDavEndpointWithValidFormattedEndpointWithoutDataAndNot
 	}
 
 	public function testGetShareEndpointWithValidFormattedEndpointAndNotCached() {
-		$response = $this->getMock('\OCP\Http\Client\IResponse');
+		$response = $this->createMock('\OCP\Http\Client\IResponse');
 		$response
 			->expects($this->once())
 			->method('getStatusCode')
@@ -171,7 +171,7 @@ public function testGetShareEndpointWithValidFormattedEndpointAndNotCached() {
 	}
 
 	public function testWithMaliciousEndpointCached() {
-		$response = $this->getMock('\OCP\Http\Client\IResponse');
+		$response = $this->createMock('\OCP\Http\Client\IResponse');
 		$response
 			->expects($this->once())
 			->method('getStatusCode')