1.17.5 CP: Fix daemon device discovery crash when Xcode isn't installed (#60546) (#60611)
* Fix daemon device discovery crash when Xcode isn't installed (#60546)
* getAvailableIOSDevices -> getAvailableTetheredIOSDevices
Co-authored-by: Jenn Magder <magder@google.com>
diff --git a/packages/flutter_tools/lib/src/ios/devices.dart b/packages/flutter_tools/lib/src/ios/devices.dart
index d23fd6c..678f5d3 100644
--- a/packages/flutter_tools/lib/src/ios/devices.dart
+++ b/packages/flutter_tools/lib/src/ios/devices.dart
@@ -69,6 +69,9 @@
'Control of iOS devices or simulators only supported on macOS.'
);
}
+ if (!_xcdevice.isInstalled) {
+ return;
+ }
deviceNotifier ??= ItemListNotifier<Device>();
@@ -77,7 +80,7 @@
// cancel any outstanding subscriptions.
await _observedDeviceEventsSubscription?.cancel();
- _observedDeviceEventsSubscription = _xcdevice.observedDeviceEvents().listen(
+ _observedDeviceEventsSubscription = _xcdevice.observedDeviceEvents()?.listen(
_onDeviceEvent,
onError: (dynamic error, StackTrace stack) {
_logger.printTrace('Process exception running xcdevice observe:\n$error\n$stack');
diff --git a/packages/flutter_tools/test/general.shard/ios/devices_test.dart b/packages/flutter_tools/test/general.shard/ios/devices_test.dart
index 04c5039..b6a123a 100644
--- a/packages/flutter_tools/test/general.shard/ios/devices_test.dart
+++ b/packages/flutter_tools/test/general.shard/ios/devices_test.dart
@@ -318,6 +318,19 @@
);
});
+ testWithoutContext('start polling without Xcode', () async {
+ final IOSDevices iosDevices = IOSDevices(
+ platform: macPlatform,
+ xcdevice: mockXcdevice,
+ iosWorkflow: mockIosWorkflow,
+ logger: logger,
+ );
+ when(mockXcdevice.isInstalled).thenReturn(false);
+
+ await iosDevices.startPolling();
+ verifyNever(mockXcdevice.getAvailableTetheredIOSDevices());
+ });
+
testWithoutContext('start polling', () async {
final IOSDevices iosDevices = IOSDevices(
platform: macPlatform,
