Unverified Commit cf8fbc36 authored by James D. Lin's avatar James D. Lin Committed by GitHub

[flutter tools] Revert desktop device name changes and print the category instead (#60395)

parent ca713557
...@@ -559,7 +559,7 @@ abstract class Device { ...@@ -559,7 +559,7 @@ abstract class Device {
supportIndicator += ' ($type)'; supportIndicator += ' ($type)';
} }
table.add(<String>[ table.add(<String>[
device.name, '${device.name} (${device.category})',
device.id, device.id,
getNameForTargetPlatform(targetPlatform), getNameForTargetPlatform(targetPlatform),
'${await device.sdkNameAndVersion}$supportIndicator', '${await device.sdkNameAndVersion}$supportIndicator',
......
...@@ -26,7 +26,7 @@ class LinuxDevice extends DesktopDevice { ...@@ -26,7 +26,7 @@ class LinuxDevice extends DesktopDevice {
bool isSupported() => true; bool isSupported() => true;
@override @override
String get name => 'Linux desktop'; String get name => 'Linux';
@override @override
Future<TargetPlatform> get targetPlatform async => TargetPlatform.linux_x64; Future<TargetPlatform> get targetPlatform async => TargetPlatform.linux_x64;
......
...@@ -24,7 +24,7 @@ class MacOSDevice extends DesktopDevice { ...@@ -24,7 +24,7 @@ class MacOSDevice extends DesktopDevice {
bool isSupported() => true; bool isSupported() => true;
@override @override
String get name => 'macOS desktop'; String get name => 'macOS';
@override @override
Future<TargetPlatform> get targetPlatform async => TargetPlatform.darwin_x64; Future<TargetPlatform> get targetPlatform async => TargetPlatform.darwin_x64;
......
...@@ -27,7 +27,7 @@ class WindowsDevice extends DesktopDevice { ...@@ -27,7 +27,7 @@ class WindowsDevice extends DesktopDevice {
bool isSupported() => true; bool isSupported() => true;
@override @override
String get name => 'Windows desktop'; String get name => 'Windows';
@override @override
Future<TargetPlatform> get targetPlatform async => TargetPlatform.windows_x64; Future<TargetPlatform> get targetPlatform async => TargetPlatform.windows_x64;
......
...@@ -120,8 +120,8 @@ void main() { ...@@ -120,8 +120,8 @@ void main() {
''' '''
2 connected devices: 2 connected devices:
ephemeral • ephemeral • android-arm • Test SDK (1.2.3) (emulator) ephemeral (mobile) • ephemeral • android-arm • Test SDK (1.2.3) (emulator)
webby • webby • web-javascript • Web SDK (1.2.4) (emulator) webby (mobile) • webby • web-javascript • Web SDK (1.2.4) (emulator)
• Cannot connect to device ABC • Cannot connect to device ABC
''' '''
......
...@@ -28,7 +28,7 @@ void main() { ...@@ -28,7 +28,7 @@ void main() {
testWithoutContext('LinuxDevice defaults', () async { testWithoutContext('LinuxDevice defaults', () async {
final PrebuiltLinuxApp linuxApp = PrebuiltLinuxApp(executable: 'foo'); final PrebuiltLinuxApp linuxApp = PrebuiltLinuxApp(executable: 'foo');
expect(await device.targetPlatform, TargetPlatform.linux_x64); expect(await device.targetPlatform, TargetPlatform.linux_x64);
expect(device.name, 'Linux desktop'); expect(device.name, 'Linux');
expect(await device.installApp(linuxApp), true); expect(await device.installApp(linuxApp), true);
expect(await device.uninstallApp(linuxApp), true); expect(await device.uninstallApp(linuxApp), true);
expect(await device.isLatestBuildInstalled(linuxApp), true); expect(await device.isLatestBuildInstalled(linuxApp), true);
......
...@@ -40,7 +40,7 @@ void main() { ...@@ -40,7 +40,7 @@ void main() {
testUsingContext('defaults', () async { testUsingContext('defaults', () async {
final MockMacOSApp mockMacOSApp = MockMacOSApp(); final MockMacOSApp mockMacOSApp = MockMacOSApp();
expect(await device.targetPlatform, TargetPlatform.darwin_x64); expect(await device.targetPlatform, TargetPlatform.darwin_x64);
expect(device.name, 'macOS desktop'); expect(device.name, 'macOS');
expect(await device.installApp(mockMacOSApp), true); expect(await device.installApp(mockMacOSApp), true);
expect(await device.uninstallApp(mockMacOSApp), true); expect(await device.uninstallApp(mockMacOSApp), true);
expect(await device.isLatestBuildInstalled(mockMacOSApp), true); expect(await device.isLatestBuildInstalled(mockMacOSApp), true);
......
...@@ -32,7 +32,7 @@ void main() { ...@@ -32,7 +32,7 @@ void main() {
testUsingContext('defaults', () async { testUsingContext('defaults', () async {
final PrebuiltWindowsApp windowsApp = PrebuiltWindowsApp(executable: 'foo'); final PrebuiltWindowsApp windowsApp = PrebuiltWindowsApp(executable: 'foo');
expect(await device.targetPlatform, TargetPlatform.windows_x64); expect(await device.targetPlatform, TargetPlatform.windows_x64);
expect(device.name, 'Windows desktop'); expect(device.name, 'Windows');
expect(await device.installApp(windowsApp), true); expect(await device.installApp(windowsApp), true);
expect(await device.uninstallApp(windowsApp), true); expect(await device.uninstallApp(windowsApp), true);
expect(await device.isLatestBuildInstalled(windowsApp), true); expect(await device.isLatestBuildInstalled(windowsApp), true);
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment