Skip to content

Commit fdbd80a

Browse files
authored
Equalize the non-existent device error message #2446 (#2597)
1 parent e458230 commit fdbd80a

File tree

4 files changed

+10
-2
lines changed

4 files changed

+10
-2
lines changed

lib/services/platform-service.ts

+4-2
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,9 @@ export class PlatformService extends EventEmitter implements IPlatformService {
4040
private $deviceAppDataFactory: Mobile.IDeviceAppDataFactory,
4141
private $projectChangesService: IProjectChangesService,
4242
private $emulatorPlatformService: IEmulatorPlatformService,
43-
private $analyticsService: IAnalyticsService) {
43+
private $analyticsService: IAnalyticsService,
44+
private $messages: IMessages,
45+
private $staticConfig: Config.IStaticConfig) {
4446
super();
4547
}
4648

@@ -504,7 +506,7 @@ export class PlatformService extends EventEmitter implements IPlatformService {
504506
}
505507

506508
if (found.length === 0) {
507-
this.$errors.fail("Cannot find device with name: %s", emulateOptions.device);
509+
this.$errors.fail(this.$messages.Devices.NotFoundDeviceByIdentifierErrorMessage, this.$staticConfig.CLIENT_NAME.toLowerCase());
508510
}
509511
}
510512
}

test/npm-support.ts

+2
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,7 @@ import { DevicePlatformsConstants } from "../lib/common/mobile/device-platforms-
2626
import { XmlValidator } from "../lib/xml-validator";
2727
import { LockFile } from "../lib/lockfile";
2828
import ProjectChangesLib = require("../lib/services/project-changes-service");
29+
import { Messages } from "../lib/common/messages/messages";
2930

3031
import path = require("path");
3132
import temp = require("temp");
@@ -82,6 +83,7 @@ function createTestInjector(): IInjector {
8283
testInjector.register("analyticsService", {
8384
track: async () => undefined
8485
});
86+
testInjector.register("messages", Messages);
8587

8688
return testInjector;
8789
}

test/platform-commands.ts

+2
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@ import { DevicePlatformsConstants } from "../lib/common/mobile/device-platforms-
2121
import { XmlValidator } from "../lib/xml-validator";
2222
import * as ChildProcessLib from "../lib/common/child-process";
2323
import ProjectChangesLib = require("../lib/services/project-changes-service");
24+
import { Messages } from "../lib/common/messages/messages";
2425

2526
let isCommandExecuted = true;
2627

@@ -140,6 +141,7 @@ function createTestInjector() {
140141
testInjector.register("analyticsService", {
141142
track: async () => undefined
142143
});
144+
testInjector.register("messages", Messages);
143145

144146
return testInjector;
145147
}

test/platform-service.ts

+2
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@ import { DevicePlatformsConstants } from "../lib/common/mobile/device-platforms-
1919
import { XmlValidator } from "../lib/xml-validator";
2020
import * as ChildProcessLib from "../lib/common/child-process";
2121
import ProjectChangesLib = require("../lib/services/project-changes-service");
22+
import { Messages } from "../lib/common/messages/messages";
2223

2324
require("should");
2425
let temp = require("temp");
@@ -80,6 +81,7 @@ function createTestInjector() {
8081
testInjector.register("analyticsService", {
8182
track: async () => undefined
8283
});
84+
testInjector.register("messages", Messages);
8385

8486
return testInjector;
8587
}

0 commit comments

Comments
 (0)