Skip to content

Commit 07a918a

Browse files
Fatme HavaluovaFatme Havaluova
Fatme Havaluova
authored and
Fatme Havaluova
committed
Add unit tests
1 parent d600317 commit 07a918a

File tree

9 files changed

+252
-39
lines changed

9 files changed

+252
-39
lines changed

‎lib/definitions/plugins.d.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@ interface IPluginsService {
44
prepare(pluginData: IDependencyData): IFuture<void>;
55
getAllInstalledPlugins(): IFuture<IPluginData[]>;
66
ensureAllDependenciesAreInstalled(): IFuture<void>;
7+
afterPrepareAllPlugins(): IFuture<void>;
78
}
89

910
interface IPluginData extends INodeModuleData {

‎lib/definitions/project.d.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,7 @@ interface IPlatformProjectService {
4040
updatePlatform(currentVersion: string, newVersion: string): IFuture<void>;
4141
preparePluginNativeCode(pluginData: IPluginData): IFuture<void>;
4242
removePluginNativeCode(pluginData: IPluginData): IFuture<void>;
43+
afterPrepareAllPlugins(): IFuture<void>;
4344
}
4445

4546
interface IAndroidProjectPropertiesManager {

‎lib/services/android-project-service.ts

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -296,6 +296,10 @@ class AndroidProjectService extends projectServiceBaseLib.PlatformProjectService
296296
}).future<void>()();
297297
}
298298

299+
public afterPrepareAllPlugins(): IFuture<void> {
300+
return Future.fromResult();
301+
}
302+
299303
private getLibraryRelativePath(basePath: string, libraryPath: string): string {
300304
return path.relative(basePath, libraryPath).split("\\").join("/");
301305
}

‎lib/services/ios-project-service.ts

Lines changed: 77 additions & 36 deletions
Original file line numberDiff line numberDiff line change
@@ -2,15 +2,16 @@
22
"use strict";
33

44
import Future = require("fibers/future");
5-
import path = require("path");
6-
import shell = require("shelljs");
7-
import util = require("util");
8-
import xcode = require("xcode");
5+
import * as path from "path";
6+
import * as shell from "shelljs";
7+
import * as util from "util";
8+
import * as os from "os";
9+
import * as xcode from "xcode";
910
import constants = require("./../constants");
1011
import helpers = require("./../common/helpers");
1112
import projectServiceBaseLib = require("./platform-project-service-base");
1213

13-
class IOSProjectService extends projectServiceBaseLib.PlatformProjectServiceBase implements IPlatformProjectService {
14+
export class IOSProjectService extends projectServiceBaseLib.PlatformProjectServiceBase implements IPlatformProjectService {
1415
private static XCODE_PROJECT_EXT_NAME = ".xcodeproj";
1516
private static XCODEBUILD_MIN_VERSION = "6.0";
1617
private static IOS_PROJECT_NAME_PLACEHOLDER = "__PROJECT_NAME__";
@@ -262,14 +263,6 @@ class IOSProjectService extends projectServiceBaseLib.PlatformProjectServiceBase
262263

263264
this.savePbxProj(project).wait();
264265
}
265-
266-
if(this.$fs.exists(this.projectPodFilePath).wait() && this.$fs.getFsStats(this.projectPodFilePath).wait().mtime < this.$fs.getFsStats(this.$projectData.projectFilePath).wait().mtime) {
267-
try {
268-
this.$childProcess.exec("pod install", { cwd: this.platformData.projectRoot }).wait();
269-
} catch(e) {
270-
this.$errors.failWithoutHelp("CocoaPods are not installed. Run `sudo gem install cocoapods` and try again.");
271-
}
272-
}
273266
}).future<void>()();
274267
}
275268

@@ -314,35 +307,31 @@ class IOSProjectService extends projectServiceBaseLib.PlatformProjectServiceBase
314307
public preparePluginNativeCode(pluginData: IPluginData): IFuture<void> {
315308
return (() => {
316309
let pluginPlatformsFolderPath = pluginData.pluginPlatformsFolderPath(IOSProjectService.IOS_PLATFORM_NAME);
317-
_.each(this.getAllDynamicFrameworksForPlugin(pluginData).wait(), fileName => this.addLibrary(path.join(pluginPlatformsFolderPath, fileName)).wait());
318-
319-
let pluginPodFilePath = path.join(pluginPlatformsFolderPath, "Podfile");
320-
if(this.$fs.exists(pluginPodFilePath).wait()) {
321-
let pluginPodFileContent = this.$fs.readText(pluginPodFilePath).wait();
322-
this.$fs.appendFile(this.projectPodFilePath, pluginPodFileContent).wait();
323-
}
310+
this.prepareDynamicFrameworks(pluginPlatformsFolderPath, pluginData).wait();
311+
this.prepareCocoapods(pluginPlatformsFolderPath).wait();
324312
}).future<void>()();
325313
}
326314

327315
public removePluginNativeCode(pluginData: IPluginData): IFuture<void> {
328316
return (() => {
329317
let pluginPlatformsFolderPath = pluginData.pluginPlatformsFolderPath(IOSProjectService.IOS_PLATFORM_NAME);
330-
let project = this.createPbxProj();
331-
332-
_.each(this.getAllDynamicFrameworksForPlugin(pluginData).wait(), fileName => {
333-
let fullFrameworkPath = path.join(pluginPlatformsFolderPath, fileName);
334-
let relativeFrameworkPath = this.getFrameworkRelativePath(fullFrameworkPath);
335-
project.removeFramework(relativeFrameworkPath, { customFramework: true, embed: true })
336-
});
337-
338-
this.savePbxProj(project).wait();
339-
340-
let pluginPodFilePath = path.join(pluginPlatformsFolderPath, "Podfile");
341-
if(this.$fs.exists(pluginPodFilePath).wait()) {
342-
let pluginPodFileContent = this.$fs.readText(pluginPodFilePath).wait();
343-
let projectPodFileContent = this.$fs.readText(this.projectPodFilePath).wait();
344-
projectPodFileContent = helpers.stringReplaceAll(projectPodFileContent, pluginPodFileContent, "");
345-
this.$fs.writeFile(this.projectPodFilePath, projectPodFileContent).wait();
318+
this.removeDynamicFrameworks(pluginPlatformsFolderPath, pluginData).wait();
319+
this.removeCocoapods(pluginPlatformsFolderPath).wait();
320+
}).future<void>()();
321+
}
322+
323+
public afterPrepareAllPlugins(): IFuture<void> {
324+
return (() => {
325+
if(this.$fs.exists(this.projectPodFilePath).wait()) {
326+
// Check availability
327+
try {
328+
this.$childProcess.exec("gem which cocoapods").wait();
329+
} catch(e) {
330+
this.$errors.failWithoutHelp("CocoaPods are not installed. Run `sudo gem install cocoapods` and try again.");
331+
}
332+
333+
this.$logger.info("Installing pods...");
334+
this.$childProcess.exec("pod install", { cwd: this.platformData.projectRoot }).wait();
346335
}
347336
}).future<void>()();
348337
}
@@ -404,5 +393,57 @@ class IOSProjectService extends projectServiceBaseLib.PlatformProjectServiceBase
404393
this.$fs.rename(path.join(fileRootLocation, oldFileName), path.join(fileRootLocation, newFileName)).wait();
405394
}).future<void>()();
406395
}
396+
397+
private prepareDynamicFrameworks(pluginPlatformsFolderPath: string, pluginData: IPluginData): IFuture<void> {
398+
return (() => {
399+
_.each(this.getAllDynamicFrameworksForPlugin(pluginData).wait(), fileName => this.addLibrary(path.join(pluginPlatformsFolderPath, fileName)).wait());
400+
}).future<void>()();
401+
}
402+
403+
private prepareCocoapods(pluginPlatformsFolderPath: string): IFuture<void> {
404+
return (() => {
405+
let pluginPodFilePath = path.join(pluginPlatformsFolderPath, "Podfile");
406+
if(this.$fs.exists(pluginPodFilePath).wait()) {
407+
let pluginPodFileContent = this.$fs.readText(pluginPodFilePath).wait();
408+
let contentToWrite = this.buildPodfileContent(pluginPodFilePath, pluginPodFileContent);
409+
this.$fs.appendFile(this.projectPodFilePath, contentToWrite).wait();
410+
}
411+
}).future<void>()();
412+
}
413+
414+
private removeDynamicFrameworks(pluginPlatformsFolderPath: string, pluginData: IPluginData): IFuture<void> {
415+
return (() => {
416+
let project = this.createPbxProj();
417+
418+
_.each(this.getAllDynamicFrameworksForPlugin(pluginData).wait(), fileName => {
419+
let fullFrameworkPath = path.join(pluginPlatformsFolderPath, fileName);
420+
let relativeFrameworkPath = this.getFrameworkRelativePath(fullFrameworkPath);
421+
project.removeFramework(relativeFrameworkPath, { customFramework: true, embed: true })
422+
});
423+
424+
this.savePbxProj(project).wait();
425+
}).future<void>()();
426+
}
427+
428+
private removeCocoapods(pluginPlatformsFolderPath: string): IFuture<void> {
429+
return (() => {
430+
let pluginPodFilePath = path.join(pluginPlatformsFolderPath, "Podfile");
431+
if(this.$fs.exists(pluginPodFilePath).wait()) {
432+
let pluginPodFileContent = this.$fs.readText(pluginPodFilePath).wait();
433+
let projectPodFileContent = this.$fs.readText(this.projectPodFilePath).wait();
434+
let contentToRemove= this.buildPodfileContent(pluginPodFilePath, pluginPodFileContent);
435+
projectPodFileContent = helpers.stringReplaceAll(projectPodFileContent, contentToRemove, "");
436+
if(_.isEmpty(projectPodFileContent)) {
437+
this.$fs.deleteFile(this.projectPodFilePath).wait();
438+
} else {
439+
this.$fs.writeFile(this.projectPodFilePath, projectPodFileContent).wait();
440+
}
441+
}
442+
}).future<void>()();
443+
}
444+
445+
private buildPodfileContent(pluginPodFilePath: string, pluginPodFileContent: string): string {
446+
return `# Begin Podfile - ${pluginPodFilePath} ${os.EOL} ${pluginPodFileContent} ${os.EOL} # End Podfile ${os.EOL}`;
447+
}
407448
}
408449
$injector.register("iOSProjectService", IOSProjectService);

‎lib/services/plugins-service.ts

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -144,6 +144,14 @@ export class PluginsService implements IPluginsService {
144144
}).future<IPluginData[]>()();
145145
}
146146

147+
public afterPrepareAllPlugins(): IFuture<void> {
148+
let action = (pluginDestinationPath: string, platform: string, platformData: IPlatformData) => {
149+
return platformData.platformProjectService.afterPrepareAllPlugins();
150+
};
151+
152+
return this.executeForAllInstalledPlatforms(action);
153+
}
154+
147155
private get nodeModulesPath(): string {
148156
return path.join(this.$projectData.projectDir, "node_modules");
149157
}

‎lib/tools/broccoli/node-modules-dest-copy.ts

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -76,6 +76,10 @@ export class DestCopy implements IBroccoliPlugin {
7676
this.$pluginsService.prepare(dependency).wait();
7777
}
7878
});
79+
80+
if(!_.isEmpty(this.dependencies)) {
81+
this.$pluginsService.afterPrepareAllPlugins().wait();
82+
}
7983
}
8084

8185
public rebuild(treeDiff: IDiffResult): void {

‎test/ios-project-service.ts

Lines changed: 149 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,149 @@
1+
/// <reference path=".d.ts" />
2+
"use strict";
3+
4+
import Future = require("fibers/future");
5+
import * as path from "path";
6+
import temp = require("temp");
7+
temp.track();
8+
9+
import ChildProcessLib = require("../lib/common/child-process");
10+
import ConfigLib = require("../lib/config");
11+
import ErrorsLib = require("../lib/common/errors");
12+
import FileSystemLib = require("../lib/common/file-system");
13+
import HostInfoLib = require("../lib/common/host-info");
14+
import iOSProjectServiceLib = require("../lib/services/ios-project-service");
15+
import LoggerLib = require("../lib/common/logger");
16+
import OptionsLib = require("../lib/options");
17+
import ProjectDataLib = require("../lib/project-data");
18+
19+
import yok = require("../lib/common/yok");
20+
21+
import { assert } from "chai";
22+
23+
function createTestInjector(projectPath: string, projectName: string): IInjector {
24+
let testInjector = new yok.Yok();
25+
testInjector.register("childProcess", ChildProcessLib.ChildProcess);
26+
testInjector.register("config", ConfigLib.Configuration);
27+
testInjector.register("errors", ErrorsLib.Errors);
28+
testInjector.register("fs", FileSystemLib.FileSystem);
29+
testInjector.register("hostInfo", HostInfoLib.HostInfo);
30+
testInjector.register("injector", testInjector);
31+
testInjector.register("iOSEmulatorServices", {});
32+
testInjector.register("iOSProjectService", iOSProjectServiceLib.IOSProjectService);
33+
testInjector.register("logger", LoggerLib.Logger);
34+
testInjector.register("options", OptionsLib.Options);
35+
testInjector.register("projectData", {
36+
platformsDir: projectPath,
37+
projectName: projectName
38+
});
39+
testInjector.register("projectHelper", {});
40+
testInjector.register("staticConfig", ConfigLib.StaticConfig);
41+
42+
return testInjector;
43+
}
44+
45+
describe("Cocoapods support", () => {
46+
it("adds plugin with Podfile", () => {
47+
let projectName = "projectDirectory";
48+
let projectPath = temp.mkdirSync(projectName);
49+
50+
let testInjector = createTestInjector(projectPath, projectName);
51+
let fs: IFileSystem = testInjector.resolve("fs");
52+
53+
let packageJsonData = {
54+
"name": "myProject",
55+
"version": "0.1.0",
56+
"nativescript": {
57+
"id": "org.nativescript.myProject",
58+
"tns-android": {
59+
"version": "1.0.0"
60+
}
61+
}
62+
};
63+
fs.writeJson(path.join(projectPath, "package.json"), packageJsonData).wait();
64+
65+
let platformsFolderPath = path.join(projectPath, "ios");
66+
fs.createDirectory(platformsFolderPath).wait();
67+
68+
let iOSProjectService = testInjector.resolve("iOSProjectService");
69+
iOSProjectService.prepareDynamicFrameworks = (pluginPlatformsFolderPath: string, pluginData: IPluginData): IFuture<void> => {
70+
return Future.fromResult();
71+
};
72+
73+
let pluginPath = temp.mkdirSync("pluginDirectory");
74+
let pluginPlatformsFolderPath = path.join(pluginPath, "platforms", "ios");
75+
let pluginPodfilePath = path.join(pluginPlatformsFolderPath, "Podfile");
76+
let pluginPodfileContent = ["source 'https://github.com/CocoaPods/Specs.git'", "platform :ios, '8.1'", "pod 'GoogleMaps'"].join("\n");
77+
fs.writeFile(pluginPodfilePath, pluginPodfileContent).wait();
78+
79+
let pluginData = {
80+
pluginPlatformsFolderPath(platform: string): string {
81+
return pluginPlatformsFolderPath;
82+
}
83+
};
84+
85+
iOSProjectService.preparePluginNativeCode(pluginData).wait();
86+
87+
let projectPodfilePath = path.join(platformsFolderPath, "Podfile");
88+
assert.isTrue(fs.exists(projectPodfilePath).wait());
89+
90+
let actualProjectPodfileContent = fs.readText(projectPodfilePath).wait();
91+
let expectedProjectPodfileContent = [`# Begin Podfile - ${pluginPodfilePath} `, ` ${pluginPodfileContent} `, " # End Podfile \n"].join("\n");
92+
assert.equal(actualProjectPodfileContent, expectedProjectPodfileContent);
93+
});
94+
it("adds and removes plugin with Podfile", () => {
95+
let projectName = "projectDirectory2";
96+
let projectPath = temp.mkdirSync(projectName);
97+
98+
let testInjector = createTestInjector(projectPath, projectName);
99+
let fs: IFileSystem = testInjector.resolve("fs");
100+
101+
let packageJsonData = {
102+
"name": "myProject2",
103+
"version": "0.1.0",
104+
"nativescript": {
105+
"id": "org.nativescript.myProject2",
106+
"tns-android": {
107+
"version": "1.0.0"
108+
}
109+
}
110+
};
111+
fs.writeJson(path.join(projectPath, "package.json"), packageJsonData).wait();
112+
113+
let platformsFolderPath = path.join(projectPath, "ios");
114+
fs.createDirectory(platformsFolderPath).wait();
115+
116+
let iOSProjectService = testInjector.resolve("iOSProjectService");
117+
iOSProjectService.prepareDynamicFrameworks = (pluginPlatformsFolderPath: string, pluginData: IPluginData): IFuture<void> => {
118+
return Future.fromResult();
119+
};
120+
iOSProjectService.removeDynamicFrameworks = (pluginPlatformsFolderPath: string, pluginData: IPluginData): IFuture<void> => {
121+
return Future.fromResult();
122+
}
123+
124+
let pluginPath = temp.mkdirSync("pluginDirectory");
125+
let pluginPlatformsFolderPath = path.join(pluginPath, "platforms", "ios");
126+
let pluginPodfilePath = path.join(pluginPlatformsFolderPath, "Podfile");
127+
let pluginPodfileContent = ["source 'https://github.com/CocoaPods/Specs.git'", "platform :ios, '8.1'", "pod 'GoogleMaps'"].join("\n");
128+
fs.writeFile(pluginPodfilePath, pluginPodfileContent).wait();
129+
130+
let pluginData = {
131+
pluginPlatformsFolderPath(platform: string): string {
132+
return pluginPlatformsFolderPath;
133+
}
134+
};
135+
136+
iOSProjectService.preparePluginNativeCode(pluginData).wait();
137+
138+
let projectPodfilePath = path.join(platformsFolderPath, "Podfile");
139+
assert.isTrue(fs.exists(projectPodfilePath).wait());
140+
141+
let actualProjectPodfileContent = fs.readText(projectPodfilePath).wait();
142+
let expectedProjectPodfileContent = [`# Begin Podfile - ${pluginPodfilePath} `, ` ${pluginPodfileContent} `, " # End Podfile \n"].join("\n");
143+
assert.equal(actualProjectPodfileContent, expectedProjectPodfileContent);
144+
145+
iOSProjectService.removePluginNativeCode(pluginData).wait();
146+
147+
assert.isFalse(fs.exists(projectPodfilePath).wait());
148+
});
149+
});

‎test/npm-support.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -120,7 +120,8 @@ function setupProject(): IFuture<any> {
120120
normalizedPlatformName: "Android",
121121
platformProjectService: {
122122
prepareProject: () => Future.fromResult(),
123-
prepareAppResources: () => Future.fromResult()
123+
prepareAppResources: () => Future.fromResult(),
124+
afterPrepareAllPlugins: () => Future.fromResult()
124125
}
125126
}
126127
};

‎test/stubs.ts

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -250,7 +250,8 @@ export class PlatformsDataStub implements IPlatformsData {
250250
appDestinationDirectoryPath: "",
251251
appResourcesDestinationDirectoryPath: "",
252252
preparePluginNativeCode: () => Future.fromResult(),
253-
removePluginNativeCode: () => Future.fromResult()
253+
removePluginNativeCode: () => Future.fromResult(),
254+
afterPrepareAllPlugins: () => Future.fromResult()
254255
};
255256
}
256257

@@ -271,7 +272,7 @@ export class PlatformProjectServiceStub implements IPlatformProjectService {
271272
validPackageNamesForDevice: [],
272273
frameworkFilesExtensions: [],
273274
appDestinationDirectoryPath: "",
274-
appResourcesDestinationDirectoryPath: ""
275+
appResourcesDestinationDirectoryPath: "",
275276
};
276277
}
277278
validate(): IFuture<void> {
@@ -316,6 +317,9 @@ export class PlatformProjectServiceStub implements IPlatformProjectService {
316317
removePluginNativeCode(pluginData: IPluginData): IFuture<void> {
317318
return Future.fromResult();
318319
}
320+
afterPrepareAllPlugins(): IFuture<void> {
321+
return Future.fromResult();
322+
}
319323
}
320324

321325
export class ProjectDataService implements IProjectDataService {

0 commit comments

Comments
 (0)