Skip to content
This repository was archived by the owner on Feb 2, 2021. It is now read-only.

Commit ef32193

Browse files
committed
Add android platform check inside android-device-discovery.
1 parent 592dbeb commit ef32193

File tree

2 files changed

+11
-2
lines changed

2 files changed

+11
-2
lines changed

mobile/mobile-core/android-device-discovery.ts

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,8 @@ export class AndroidDeviceDiscovery extends DeviceDiscovery implements Mobile.IA
1313
private isStarted: boolean;
1414

1515
constructor(private $injector: IInjector,
16-
private $adb: Mobile.IAndroidDebugBridge) {
16+
private $adb: Mobile.IAndroidDebugBridge,
17+
private $mobileHelper: Mobile.IMobileHelper) {
1718
super();
1819
}
1920

@@ -29,7 +30,10 @@ export class AndroidDeviceDiscovery extends DeviceDiscovery implements Mobile.IA
2930
this.removeDevice(deviceIdentifier);
3031
}
3132

32-
public async startLookingForDevices(): Promise<void> {
33+
public async startLookingForDevices(options?: Mobile.IDeviceLookingOptions): Promise<void> {
34+
if (options && options.platform && !this.$mobileHelper.isAndroidPlatform(options.platform)) {
35+
return;
36+
}
3337
await this.ensureAdbServerStarted();
3438
await this.checkForDevices();
3539
}

test/unit-tests/mobile/android-device-discovery.ts

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,11 @@ function createTestInjector(): IInjector {
4040
injector.register("errors", {});
4141
injector.register("logger", {});
4242
injector.register("androidDebugBridgeResultHandler", AndroidDebugBridgeResultHandler);
43+
injector.register("mobileHelper", {
44+
isAndroidPlatform: () => {
45+
return true;
46+
}
47+
});
4348
injector.register("childProcess", {
4449
spawn: (command: string, args?: string[], options?: any) => {
4550
mockChildProcess = new MockEventEmitter();

0 commit comments

Comments
 (0)