Inject OCSClient

Fixes https://github.com/owncloud/core/issues/21451

1 files changed, 16 insertions, 14 deletions

diff --git a/apps/provisioning_api/tests/appstest.php b/apps/provisioning_api/tests/appstest.php
index 4ccba704a3a..871158c5545 100644
--- a/apps/provisioning_api/tests/appstest.php
+++ b/apps/provisioning_api/tests/appstest.php
@@ -23,6 +23,7 @@
  */
 
 namespace OCA\Provisioning_API\Tests;
+use OC\OCSClient;
 use OCA\Provisioning_API\Apps;
 use OCP\API;
 use OCP\App\IAppManager;
@@ -36,42 +37,43 @@ use OCP\IUserSession;
  * @package OCA\Provisioning_API\Tests
  */
 class AppsTest extends TestCase {
-
 	/** @var IAppManager */
 	private $appManager;
-
 	/** @var Apps */
 	private $api;
-
 	/** @var IUserSession */
 	private $userSession;
+	/** @var OCSClient */
+	private $ocsClient;
 
 	public function setup() {
 		parent::setup();
 		$this->appManager = \OC::$server->getAppManager();
 		$this->groupManager = \OC::$server->getGroupManager();
 		$this->userSession = \OC::$server->getUserSession();
-		$this->api = new Apps($this->appManager);
+		$this->ocsClient = $this->getMockBuilder('\OC\OCSClient')
+				->disableOriginalConstructor()->getMock();
+		$this->api = new Apps($this->appManager, $this->ocsClient);
 	}
 
 	public function testGetAppInfo() {
 		$result = $this->api->getAppInfo(['appid' => 'provisioning_api']);
 		$this->assertInstanceOf('OC_OCS_Result', $result);
 		$this->assertTrue($result->succeeded());
-
 	}
 
 	public function testGetAppInfoOnBadAppID() {
-
 		$result = $this->api->getAppInfo(['appid' => 'not_provisioning_api']);
 		$this->assertInstanceOf('OC_OCS_Result', $result);
 		$this->assertFalse($result->succeeded());
 		$this->assertEquals(API::RESPOND_NOT_FOUND, $result->getStatusCode());
-
 	}
 
 	public function testGetApps() {
-
+		$this->ocsClient
+				->expects($this->any())
+				->method($this->anything())
+				->will($this->returnValue(null));
 		$user = $this->generateUsers();
 		$this->groupManager->get('admin')->addUser($user);
 		$this->userSession->setUser($user);
@@ -80,27 +82,27 @@ class AppsTest extends TestCase {
 
 		$this->assertTrue($result->succeeded());
 		$data = $result->getData();
-		$this->assertEquals(count(\OC_App::listAllApps()), count($data['apps']));
-
+		$this->assertEquals(count(\OC_App::listAllApps(false, true, $this->ocsClient)), count($data['apps']));
 	}
 
 	public function testGetAppsEnabled() {
-
 		$_GET['filter'] = 'enabled';
 		$result = $this->api->getApps(['filter' => 'enabled']);
 		$this->assertTrue($result->succeeded());
 		$data = $result->getData();
 		$this->assertEquals(count(\OC_App::getEnabledApps()), count($data['apps']));
-
 	}
 
 	public function testGetAppsDisabled() {
-
+		$this->ocsClient
+				->expects($this->any())
+				->method($this->anything())
+				->will($this->returnValue(null));
 		$_GET['filter'] = 'disabled';
 		$result = $this->api->getApps(['filter' => 'disabled']);
 		$this->assertTrue($result->succeeded());
 		$data = $result->getData();
-		$apps = \OC_App::listAllApps();
+		$apps = \OC_App::listAllApps(false, true, $this->ocsClient);
 		$list =  array();
 		foreach($apps as $app) {
 			$list[] = $app['id'];