chore: Remove canStartApplication method

Remove `canStartApplication` method from application managers as it does not have a useful usage. Previously we've used it on Windows and Linux as we did not have a way to start iOS applications.
Now, as we have implemented such way through Safari, the method has lost its purpose.

Author: rosen-vladimirov

definitions/mobile.d.ts

@@ -270,7 +270,6 @@ declare module Mobile {
 		startApplication(appData: IApplicationData): Promise<void>;
 		stopApplication(appData: IApplicationData): Promise<void>;
 		restartApplication(appData: IApplicationData): Promise<void>;
-		canStartApplication(): boolean;
 		checkForApplicationUpdates(): Promise<void>;
 		isLiveSyncSupported(appIdentifier: string): Promise<boolean>;
 		getApplicationInfo(applicationIdentifier: string): Promise<Mobile.IApplicationInfo>;

mobile/android/android-application-manager.ts

@@ -92,10 +92,6 @@ export class AndroidApplicationManager extends ApplicationManagerBase {
 		return Promise.resolve(null);
 	}
 
-	public canStartApplication(): boolean {
-		return true;
-	}
-
 	public async isLiveSyncSupported(appIdentifier: string): Promise<boolean> {
 		const liveSyncVersion = await this.adb.sendBroadcastToDevice(LiveSyncConstants.CHECK_LIVESYNC_INTENT_NAME, { "app-id": appIdentifier });
 		return liveSyncVersion === LiveSyncConstants.VERSION_2 || liveSyncVersion === LiveSyncConstants.VERSION_3;

mobile/application-manager-base.ts

@@ -84,7 +84,6 @@ export abstract class ApplicationManagerBase extends EventEmitter implements Mob
 	public abstract async stopApplication(appData: Mobile.IApplicationData): Promise<void>;
 	public abstract async getInstalledApplications(): Promise<string[]>;
 	public abstract async getApplicationInfo(applicationIdentifier: string): Promise<Mobile.IApplicationInfo>;
-	public abstract canStartApplication(): boolean;
 	public abstract async getDebuggableApps(): Promise<Mobile.IDeviceApplicationInformation[]>;
 	public abstract async getDebuggableAppViews(appIdentifiers: string[]): Promise<IDictionary<Mobile.IDebugWebViewInfo[]>>;
 

mobile/ios/device/ios-application-manager.ts

@@ -11,8 +11,6 @@ export class IOSApplicationManager extends ApplicationManagerBase {
 		private device: Mobile.IDevice,
 		private $errors: IErrors,
 		private $iOSNotificationService: IiOSNotificationService,
-		private $hostInfo: IHostInfo,
-		private $staticConfig: Config.IStaticConfig,
 		private $iosDeviceOperations: IIOSDeviceOperations,
 		private $options: ICommonOptions,
 		private $deviceLogProvider: Mobile.IDeviceLogProvider) {
@@ -124,10 +122,6 @@ export class IOSApplicationManager extends ApplicationManagerBase {
 		await this.device.openDeviceLogStream();
 	}
 
-	public canStartApplication(): boolean {
-		return this.$hostInfo.isDarwin || (this.$hostInfo.isWindows && this.$staticConfig.enableDeviceRunCommandOnWindows);
-	}
-
 	public getDebuggableApps(): Promise<Mobile.IDeviceApplicationInformation[]> {
 		// Implement when we can find debuggable applications for iOS.
 		return Promise.resolve([]);

mobile/ios/simulator/ios-simulator-application-manager.ts

@@ -54,10 +54,6 @@ export class IOSSimulatorApplicationManager extends ApplicationManagerBase {
 		return this.iosSim.stopApplication(this.identifier, appData.appId, appData.projectName);
 	}
 
-	public canStartApplication(): boolean {
-		return true;
-	}
-
 	public async getApplicationInfo(applicationIdentifier: string): Promise<Mobile.IApplicationInfo> {
 		let result: Mobile.IApplicationInfo = null;
 		const plistContent = await this.getParsedPlistContent(applicationIdentifier);

services/livesync-service-base.ts

@@ -183,7 +183,7 @@ class LiveSyncServiceBase implements ILiveSyncServiceBase {
 						shouldRefreshApplication = false;
 					}
 
-					if (device.applicationManager.canStartApplication() && !shouldRefreshApplication) {
+					if (!shouldRefreshApplication) {
 						await device.applicationManager.startApplication({ appId: appIdentifier, projectName: "" });
 					}
 					wasInstalled = false;

test/unit-tests/mobile/application-manager-base.ts

@@ -40,10 +40,6 @@ class ApplicationManager extends ApplicationManagerBase {
 		return null;
 	}
 
-	public canStartApplication(): boolean {
-		return true;
-	}
-
 	public async getDebuggableApps(): Promise<Mobile.IDeviceApplicationInformation[]> {
 		return currentlyAvailableAppsForDebugging;
 	}

test/unit-tests/mobile/devices-service.ts

@@ -78,7 +78,6 @@ const iOSSimulator = {
 	},
 	applicationManager: {
 		getInstalledApplications: () => Promise.resolve(["com.telerik.unitTest1", "com.telerik.unitTest2"]),
-		canStartApplication: () => true,
 		startApplication: (packageName: string, framework: string) => Promise.resolve(),
 		tryStartApplication: (packageName: string, framework: string) => Promise.resolve(),
 		reinstallApplication: (packageName: string, packageFile: string) => Promise.resolve(),
@@ -218,7 +217,6 @@ describe("devicesService", () => {
 		},
 		applicationManager: {
 			getInstalledApplications: () => Promise.resolve(["com.telerik.unitTest1", "com.telerik.unitTest2"]),
-			canStartApplication: () => true,
 			startApplication: (appData: Mobile.IApplicationData) => Promise.resolve(),
 			tryStartApplication: (appData: Mobile.IApplicationData) => Promise.resolve(),
 			reinstallApplication: (packageName: string, packageFile: string) => Promise.resolve(),
@@ -238,7 +236,6 @@ describe("devicesService", () => {
 		},
 		applicationManager: {
 			getInstalledApplications: () => Promise.resolve(["com.telerik.unitTest1", "com.telerik.unitTest2", "com.telerik.unitTest3"]),
-			canStartApplication: () => true,
 			startApplication: (appData: Mobile.IApplicationData) => Promise.resolve(),
 			tryStartApplication: (appData: Mobile.IApplicationData) => Promise.resolve(),
 			reinstallApplication: (packageName: string, packageFile: string) => Promise.resolve(),
@@ -935,18 +932,6 @@ describe("devicesService", () => {
 			});
 		});
 
-		it("does not call startApplication when canStartApplication returns false", async () => {
-			iOSDevice.applicationManager.canStartApplication = () => false;
-			iOSDevice.applicationManager.startApplication = (): Promise<void> => {
-				throw new Error("Start application must not be called for iOSDevice when canStartApplication returns false.");
-			};
-
-			const results = devicesService.deployOnDevices([androidDevice.deviceInfo.identifier, iOSDevice.deviceInfo.identifier], "path", "packageName", "cordova");
-			assert.isTrue(results.length > 0);
-			const deployOnDevicesResults = await Promise.all(results);
-			assert.deepEqual(deployOnDevicesResults, [undefined, undefined]);
-		});
-
 		it("throws error when invalid identifier is passed", async () => {
 			const results = devicesService.deployOnDevices(["invalidDeviceId", iOSDevice.deviceInfo.identifier], "path", "packageName", "cordova");
 			const expectedErrorMessage = getErrorMessage(testInjector, "NotFoundDeviceByIdentifierErrorMessageWithIdentifier", "invalidDeviceId");