Add 'sandbox-pod' command. Implement test.

config/config.json

@@ -5,5 +5,6 @@
 	"PROXY_HOSTNAME": "127.0.0.1",
 	"TYPESCRIPT_COMPILER_OPTIONS": {},
 	"CI_LOGGER": false,
-	"ANDROID_DEBUG_UI_MAC": "Google Chrome"
+	"ANDROID_DEBUG_UI_MAC": "Google Chrome",
+	"USE_POD_SANDBOX": true
 }

lib/config.ts

@@ -11,6 +11,7 @@ export class Configuration extends configBaseLib.ConfigBase implements IConfigur
 	TYPESCRIPT_COMPILER_OPTIONS = {};
 	USE_PROXY = false;
 	ANDROID_DEBUG_UI: string = null;
+	USE_POD_SANDBOX: boolean = true;
 
 	/*don't require logger and everything that has logger as dependency in config.js due to cyclic dependency*/
 	constructor(protected $fs: IFileSystem) {

lib/declarations.ts

@@ -36,6 +36,7 @@ interface IStaticConfig extends Config.IStaticConfig { }
 
 interface IConfiguration extends Config.IConfig {
 	ANDROID_DEBUG_UI: string;
+	USE_POD_SANDBOX: boolean;
 }
 
 interface IApplicationPackage {

lib/services/ios-project-service.ts

@@ -32,7 +32,8 @@ export class IOSProjectService extends projectServiceBaseLib.PlatformProjectServ
 		private $options: IOptions,
 		private $injector: IInjector,
 		$projectDataService: IProjectDataService,
-		private $prompter: IPrompter) {
+		private $prompter: IPrompter,
+		private $config: IConfiguration) {
 			super($fs, $projectData, $projectDataService);
 		}
 
@@ -495,7 +496,8 @@ export class IOSProjectService extends projectServiceBaseLib.PlatformProjectServ
 
 	private executePodInstall(): IFuture<any> {
 		this.$logger.info("Installing pods...");
-		return this.$childProcess.spawnFromEvent("pod",  ["install"], "close", { cwd: this.platformData.projectRoot, stdio: 'inherit' });
+		let podTool = this.$config.USE_POD_SANDBOX ? "sandbox-pod" : "pod";
+		return this.$childProcess.spawnFromEvent(podTool,  ["install"], "close", { cwd: this.platformData.projectRoot, stdio: 'inherit' });
 	}
 
 	private prepareFrameworks(pluginPlatformsFolderPath: string, pluginData: IPluginData): IFuture<void> {

test/project-service.ts

@@ -4,7 +4,7 @@
 import yok = require('../lib/common/yok');
 import stubs = require('./stubs');
 import * as constants from "./../lib/constants";
-import * as ChildProcessLib from "../lib/common/child-process";
+import {ChildProcess} from "../lib/common/child-process";
 import * as ProjectServiceLib from "../lib/services/project-service";
 import * as ProjectDataServiceLib from "../lib/services/project-data-service";
 import * as ProjectDataLib from "../lib/project-data";
@@ -20,6 +20,7 @@ import * as helpers from "../lib/common/helpers";
 import {assert} from "chai";
 import * as optionsLib from "../lib/options";
 import * as hostInfoLib from "../lib/common/host-info";
+import iOSProjectServiceLib = require("../lib/services/ios-project-service");
 
 let mockProjectNameValidator = {
 	validate: () => { return true; }
@@ -39,21 +40,20 @@ class ProjectIntegrationTest {
 		return projectService.createProject(projectName);
 	}
 
-	public getDefaultTemplatePath(): IFuture<string> {
+	public getDefaultTemplatePath(templateName: string): IFuture<string> {
 		return (() => {
 			let npmInstallationManager = this.testInjector.resolve("npmInstallationManager");
 			let fs = this.testInjector.resolve("fs");
 
-			let defaultTemplatePackageName = "tns-template-hello-world";
 			let cacheRoot = npmInstallationManager.getCacheRootPath();
-			let defaultTemplatePath = path.join(cacheRoot, defaultTemplatePackageName);
-			let latestVersion = npmInstallationManager.getLatestVersion(defaultTemplatePackageName).wait();
+			let defaultTemplatePath = path.join(cacheRoot, templateName);
+			let latestVersion = npmInstallationManager.getLatestVersion(templateName).wait();
 
 			if(!fs.exists(path.join(defaultTemplatePath, latestVersion)).wait()) {
-				npmInstallationManager.addToCache(defaultTemplatePackageName, latestVersion).wait();
+				npmInstallationManager.addToCache(templateName, latestVersion).wait();
 			}
 			if(!fs.exists(path.join(defaultTemplatePath, latestVersion, "package", "app")).wait()) {
-				npmInstallationManager.cacheUnpack(defaultTemplatePackageName, latestVersion).wait();
+				npmInstallationManager.cacheUnpack(templateName, latestVersion).wait();
 			}
 
 			return path.join(defaultTemplatePath, latestVersion, "package");
@@ -103,7 +103,7 @@ class ProjectIntegrationTest {
 
 	private createTestInjector(): void {
 		this.testInjector = new yok.Yok();
-		this.testInjector.register("childProcess", ChildProcessLib.ChildProcess);
+		this.testInjector.register("childProcess", ChildProcess);
 		this.testInjector.register("errors", stubs.ErrorsStub);
 		this.testInjector.register('logger', stubs.LoggerStub);
 		this.testInjector.register("projectService", ProjectServiceLib.ProjectService);
@@ -118,7 +118,6 @@ class ProjectIntegrationTest {
 		this.testInjector.register("npmInstallationManager", NpmInstallationManagerLib.NpmInstallationManager);
 		this.testInjector.register("npm", NpmLib.NodePackageManager);
 		this.testInjector.register("httpClient", HttpClientLib.HttpClient);
-		this.testInjector.register("config", {});
 		this.testInjector.register("lockfile", stubs.LockFile);
 
 		this.testInjector.register("options", optionsLib.Options);
@@ -135,7 +134,7 @@ describe("Project Service Tests", () => {
 			let options = projectIntegrationTest.testInjector.resolve("options");
 
 			options.path = tempFolder;
-			options.copyFrom = projectIntegrationTest.getDefaultTemplatePath().wait();
+			options.copyFrom = projectIntegrationTest.getDefaultTemplatePath("tns-template-hello-world").wait();
 
 			projectIntegrationTest.createProject(projectName).wait();
 			projectIntegrationTest.assertProject(tempFolder, projectName, "org.nativescript.myapp").wait();
@@ -147,12 +146,52 @@ describe("Project Service Tests", () => {
 			let options = projectIntegrationTest.testInjector.resolve("options");
 
 			options.path = tempFolder;
-			options.copyFrom = projectIntegrationTest.getDefaultTemplatePath().wait();
+			options.copyFrom = projectIntegrationTest.getDefaultTemplatePath("tns-template-hello-world").wait();
 			options.appid = "my.special.id";
 
 			projectIntegrationTest.createProject(projectName).wait();
 			projectIntegrationTest.assertProject(tempFolder, projectName, options.appid).wait();
 		});
+		it("creates ios project and tests post-install sandboxing of CocoaPods setup", () => {
+			if (require("os").platform() !== "darwin") {
+				console.log("Skipping CocoaPods sandbox test. It works only on darwin.");
+				return;
+			}
+
+			let testInjector = createInjectorForPodsTest();
+
+			let iOSProjectService: IPlatformProjectService = testInjector.resolve("iOSProjectService");
+			let fs: IFileSystem = testInjector.resolve("fs");
+			let childProcess = testInjector.resolve("childProcess");
+			let projectIntegrationTest = new ProjectIntegrationTest();
+			let workingFolderPath = temp.mkdirSync("ios_project");
+
+			let iosTemplatePath = path.join(projectIntegrationTest.getDefaultTemplatePath("tns-ios").wait(), "framework/");
+			childProcess.exec(`cp -R ${iosTemplatePath} ${workingFolderPath}`, { cwd: workingFolderPath }).wait();
+			fs.writeFile("/tmp/Podfile/testFile.txt", "Test content.").wait();
+
+			let postInstallCommmand = `\`cat /tmp/Podfile/testFile.txt > ${workingFolderPath}/copyTestFile.txt && rm -rf /tmp/Podfile\``;
+			let podfileContent = `post_install do |installer_representation| ${postInstallCommmand} end`;
+			fs.writeFile(path.join(workingFolderPath, "Podfile"), podfileContent).wait();
+
+			let platformData =  iOSProjectService.platformData;
+			Object.defineProperty(iOSProjectService, "platformData", {
+				get: () => {
+					return { projectRoot: workingFolderPath };
+				}
+    		 });
+
+			try {
+				iOSProjectService.afterPrepareAllPlugins().wait();
+			} catch(e) {
+				assert.isNotNull(e);
+			} finally {
+				Object.defineProperty(iOSProjectService, "platformData", platformData);	
+			}
+
+			assert.isTrue(fs.exists("/tmp/Podfile").wait());
+			assert.isTrue(fs.exists(path.join(workingFolderPath, "copyTestFile.txt")).wait());
+		});
 	});
 });
 
@@ -173,10 +212,9 @@ function createTestInjector() {
 
 	testInjector.register("npmInstallationManager", NpmInstallationManagerLib.NpmInstallationManager);
 	testInjector.register("httpClient", HttpClientLib.HttpClient);
-	testInjector.register("config", {});
 	testInjector.register("lockfile", stubs.LockFile);
 
-	testInjector.register("childProcess", ChildProcessLib.ChildProcess);
+	testInjector.register("childProcess", ChildProcess);
 
 	testInjector.register('projectData', ProjectDataLib.ProjectData);
 	testInjector.register("options", optionsLib.Options);
@@ -185,6 +223,33 @@ function createTestInjector() {
 	return testInjector;
 }
 
+function createInjectorForPodsTest() {
+	let testInjector = new yok.Yok();
+
+	testInjector.register("errors", stubs.ErrorsStub);
+	testInjector.register('logger', stubs.LoggerStub);
+	testInjector.register("projectHelper", {});
+	testInjector.register("projectData", {
+		projectName: "__PROJECT_NAME__",
+		platformsDir: ""
+	});
+	testInjector.register("projectDataService", {});
+	testInjector.register("iOSEmulatorServices", {});
+	testInjector.register("config", {
+		"USE_POD_SANDBOX": true
+	});
+	testInjector.register("prompter", {});
+	testInjector.register("fs", fsLib.FileSystem);
+	testInjector.register("staticConfig", StaticConfigLib.StaticConfig);
+	testInjector.register("npmInstallationManager", NpmInstallationManagerLib.NpmInstallationManager);
+	testInjector.register("iOSProjectService", iOSProjectServiceLib.IOSProjectService);
+	testInjector.register("childProcess", ChildProcess);
+	testInjector.register("options", optionsLib.Options);
+	testInjector.register("hostInfo", hostInfoLib.HostInfo);
+
+	return testInjector;
+}
+
 describe("project upgrade procedure tests", () => {
 	it("should throw error when no nativescript project folder specified", () => {
 		let testInjector = createTestInjector();