Skip to content
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.

Commit f3080d9

Browse files
committedSep 29, 2017
Remove getting cloud lib version
Remove getting cloud lib version through spawning command as this is no longer required. CLI has API to get the installed extensions, so it should be used instead.
1 parent d36bdb4 commit f3080d9

File tree

4 files changed

+0
-35
lines changed

4 files changed

+0
-35
lines changed
 

‎README.md

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -414,12 +414,6 @@ Library that helps identifying if the environment can be used for development of
414414
*/
415415
nativeScriptCliVersion: string;
416416

417-
/**
418-
* The version of `nativescript-cloud` library, as returned by `tns cloud lib version`.
419-
* @type {string}
420-
*/
421-
nativeScriptCloudVersion: string;
422-
423417
/**
424418
* Information about xcproj.
425419
* @type {string}

‎lib/sys-info.ts

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,6 @@ export class SysInfo implements NativeScriptDoctor.ISysInfo {
4141
private sysInfoCache: NativeScriptDoctor.ISysInfoData;
4242
private isCocoaPodsWorkingCorrectlyCache: boolean;
4343
private nativeScriptCliVersionCache: string;
44-
private nativeScriptCloudVersionCache: string;
4544
private xcprojInfoCache: NativeScriptDoctor.IXcprojInfo;
4645
private isCocoaPodsUpdateRequiredCache: boolean;
4746
private shouldCache: boolean = true;
@@ -260,7 +259,6 @@ export class SysInfo implements NativeScriptDoctor.ISysInfo {
260259
result.isCocoaPodsWorkingCorrectly = await this.isCocoaPodsWorkingCorrectly();
261260

262261
result.nativeScriptCliVersion = await this.getNativeScriptCliVersion();
263-
result.nativeScriptCloudVersion = await this.getNativeScriptCloudVersion();
264262

265263
result.isCocoaPodsUpdateRequired = await this.isCocoaPodsUpdateRequired();
266264
result.isAndroidSdkConfiguredCorrectly = await this.isAndroidSdkConfiguredCorrectly();
@@ -303,13 +301,6 @@ export class SysInfo implements NativeScriptDoctor.ISysInfo {
303301
});
304302
}
305303

306-
public getNativeScriptCloudVersion(): Promise<string> {
307-
return this.getValueForProperty(() => this.nativeScriptCloudVersionCache, async (): Promise<string> => {
308-
const output = await this.execCommand("tns cloud lib version");
309-
return output ? this.getVersionFromCLIOutput(output.trim()) : output;
310-
});
311-
}
312-
313304
public getXcprojInfo(): Promise<NativeScriptDoctor.IXcprojInfo> {
314305
return this.getValueForProperty(() => this.xcprojInfoCache, async (): Promise<NativeScriptDoctor.IXcprojInfo> => {
315306
const cocoaPodsVersion = await this.getCocoaPodsVersion();

‎test/sys-info.ts

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,6 @@ interface IChildProcessResults {
3737
podVersion: IChildProcessResultDescription;
3838
pod: IChildProcessResultDescription;
3939
nativeScriptCliVersion: IChildProcessResultDescription;
40-
nativeScriptCloudVersion: IChildProcessResultDescription;
4140
git: IChildProcessResultDescription;
4241
}
4342

@@ -94,7 +93,6 @@ function createChildProcessResults(childProcessResult: IChildProcessResults): ID
9493
'"C:\\Program Files/Git/cmd/git.exe" --version': childProcessResult.gitVersion, // When running Windows test on the Non-Windows platform
9594
"gradle -v": childProcessResult.gradleVersion,
9695
"tns --version": childProcessResult.nativeScriptCliVersion,
97-
"tns cloud lib version": childProcessResult.nativeScriptCloudVersion,
9896
"emulator": { shouldThrowError: false },
9997
"which git": childProcessResult.git
10098
};
@@ -240,7 +238,6 @@ describe("SysInfo unit tests", () => {
240238
podVersion: { result: setStdOut("0.38.2") },
241239
pod: { result: setStdOut("success") },
242240
nativeScriptCliVersion: { result: setStdOut("2.5.0") },
243-
nativeScriptCloudVersion: { result: setStdOut("0.1.0") },
244241
git: { result: setStdOut("git") }
245242
};
246243

@@ -347,10 +344,6 @@ describe("SysInfo unit tests", () => {
347344
{
348345
testedProperty: "nativeScriptCliVersion",
349346
method: (currentSysInfo: SysInfo) => currentSysInfo.getNativeScriptCliVersion()
350-
},
351-
{
352-
testedProperty: "nativeScriptCloudVersion",
353-
method: (currentSysInfo: SysInfo) => currentSysInfo.getNativeScriptCloudVersion()
354347
}];
355348

356349
testData.forEach((testCase) => {
@@ -419,7 +412,6 @@ ${expectedCliVersion}`;
419412
podVersion: { shouldThrowError: true },
420413
pod: { shouldThrowError: true },
421414
nativeScriptCliVersion: { shouldThrowError: true },
422-
nativeScriptCloudVersion: { shouldThrowError: true },
423415
git: { shouldThrowError: false }
424416
};
425417
androidToolsInfo.validateAndroidHomeEnvVariable = (): any[] => [1];

‎typings/interfaces.ts

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -105,12 +105,6 @@ declare module NativeScriptDoctor {
105105
*/
106106
getNativeScriptCliVersion(): Promise<string>;
107107

108-
/**
109-
* Returns the version of the installed `nativescript-cloud` version.
110-
* @return {Promise<string>} Returns the version of the installed `nativescript-cloud` version.
111-
*/
112-
getNativeScriptCloudVersion(): Promise<string>;
113-
114108
/**
115109
* Checks if xcproj is required to build projects and if it is installed.
116110
* @return {Promise<IXcprojInfo>} Returns the collected information aboud xcproj.
@@ -291,12 +285,6 @@ declare module NativeScriptDoctor {
291285
*/
292286
nativeScriptCliVersion: string;
293287

294-
/**
295-
* The version of `nativescript-cloud` library, as returned by `tns cloud lib version`.
296-
* @type {string}
297-
*/
298-
nativeScriptCloudVersion: string;
299-
300288
/**
301289
* Information about xcproj.
302290
* @type {string}

0 commit comments

Comments
 (0)
Please sign in to comment.