moved caching logic to npminstallation manager

the logic for dealing with npm is in the wrong place (ios-debug-service) so it's moved to npminstallation manager where it should be

Author: Plamen5kov

lib/declarations.ts

@@ -10,6 +10,7 @@ interface INpmInstallationManager {
 	getLatestVersion(packageName: string): IFuture<string>;
 	getNextVersion(packageName: string): IFuture<string>;
 	getLatestCompatibleVersion(packageName: string): IFuture<string>;
+	getInspectorFromCache(inspectorNpmPackageName: string): IFuture<string>;
 }
 
 interface INpmInstallOptions {

lib/npm-installation-manager.ts

@@ -6,6 +6,8 @@ export class NpmInstallationManager implements INpmInstallationManager {
 	private static NPM_LOAD_FAILED = "Failed to retrieve data from npm. Please try again a little bit later.";
 
 	constructor(private $npm: INodePackageManager,
+		private $projectData: IProjectData,
+		private $childProcess: IChildProcess,
 		private $logger: ILogger,
 		private $errors: IErrors,
 		private $options: IOptions,
@@ -57,6 +59,39 @@ export class NpmInstallationManager implements INpmInstallationManager {
 		}).future<string>()();
 	}
 
+	public getInspectorFromCache(inspectorNpmPackageName: string) : IFuture<string> {
+		return (() => {
+			let inspectorPath = path.join(this.$projectData.projectDir, "node_modules", inspectorNpmPackageName);
+
+			// local installation takes precedence over cache
+			if(!this.inspectorAlreadyInstalled(inspectorPath).wait()) {
+				let cachepath = this.$childProcess.exec("npm get cache").wait().trim();
+				let version = this.getLatestCompatibleVersion(inspectorNpmPackageName).wait();
+				let pathToPackageInCache = path.join(cachepath, inspectorNpmPackageName, version);
+				let pathToUnzippedInspector = path.join(pathToPackageInCache, "package");
+
+				if(!this.$fs.exists(pathToPackageInCache).wait()) {
+					this.$childProcess.exec(`npm cache add ${inspectorNpmPackageName}@${version}`).wait();
+					let inspectorTgzPathInCache = path.join(pathToPackageInCache, "package.tgz");
+					this.$childProcess.exec(`tar -xf ${inspectorTgzPathInCache} -C ${pathToPackageInCache}`).wait();
+					this.$childProcess.exec(`npm install --prefix ${pathToUnzippedInspector}`).wait();
+				}
+				this.$logger.out("Using inspector from cache.");
+				return pathToUnzippedInspector;
+			}
+			return inspectorPath;
+		}).future<string>()();
+	}
+
+	private inspectorAlreadyInstalled(pathToInspector: string): IFuture<Boolean> {
+		return (() => {
+			if(this.$fs.exists(pathToInspector).wait()) {
+				return true;
+			}
+			return false;
+		}).future<Boolean>()();
+	}
+
 	private installCore(packageName: string, pathToSave: string, version: string, dependencyType: string): IFuture<string> {
 		return (() => {
 			const possiblePackageName= path.resolve(packageName);

lib/services/ios-debug-service.ts

@@ -214,24 +214,7 @@ class IOSDebugService implements IDebugService {
 	private openAppInspector(fileDescriptor: string): IFuture<void> {
 		if (this.$options.client) {
 			return (() => {
-				let inspectorPath = path.join(this.$projectData.projectDir, "node_modules", inspectorNpmPackageName);
-
-				// local installation takes precedence over cache
-				if(!this.inspectorAlreadyInstalled(inspectorPath).wait()) {
-					let cachepath = this.$childProcess.exec("npm get cache").wait().trim();
-					let version = this.$npmInstallationManager.getLatestCompatibleVersion(inspectorNpmPackageName).wait();
-					let pathToPackageInCache = path.join(cachepath, inspectorNpmPackageName, version);
-					let pathToUnzippedInspector = path.join(pathToPackageInCache, "package");
-
-					if(!this.$fs.exists(pathToPackageInCache).wait()) {
-						this.$childProcess.exec(`npm cache add ${inspectorNpmPackageName}@${version}`).wait();
-						let inspectorTgzPathInCache = path.join(pathToPackageInCache, "package.tgz");
-						this.$childProcess.exec(`tar -xf ${inspectorTgzPathInCache} -C ${pathToPackageInCache}`).wait();
-						this.$childProcess.exec(`npm install --prefix ${pathToUnzippedInspector}`).wait();
-					}
-					this.$logger.out("Using inspector from cache.");
-					inspectorPath = pathToUnzippedInspector;
-				}
+				let  inspectorPath = this.$npmInstallationManager.getInspectorFromCache(inspectorNpmPackageName).wait();
 
 				let inspectorSourceLocation = path.join(inspectorPath, inspectorUiDir, "Main.html");
 				let inspectorApplicationPath = path.join(inspectorPath, inspectorAppName);
@@ -245,14 +228,5 @@ class IOSDebugService implements IDebugService {
 			}).future<void>()();
 		}
 	}
-
-	private inspectorAlreadyInstalled(pathToInspector: string): IFuture<Boolean> {
-		return (() => {
-			if(this.$fs.exists(pathToInspector).wait()) {
-				return true;
-			}
-			return false;
-		}).future<Boolean>()();
-	}
 }
 $injector.register("iOSDebugService", IOSDebugService);