From 1031872784e9373082797e5bf5c815816af2105b Mon Sep 17 00:00:00 2001 From: Antony Chan Date: Mon, 8 Oct 2018 23:14:32 -0700 Subject: [PATCH] check isAvailable key on simulator object (#21557) Summary: With the new xcode command line tools (10.1 beta2), running `xcrun simctl list --json devices` seems to gives us a slightly different format than what we were expecting. More specifically, the `availability` key is changed to `isAvailable` and returns `'YES'` if the simulator is available: Before: ```js devices: { 'iOS 9.2': [ { state: 'Shutdown', availability: '(unavailable, runtime profile not found)', name: 'iPhone 4s', udid: 'B9B5E161-416B-43C4-A78F-729CB96CC8C6', }, { state: 'Shutdown', availability: '(unavailable, runtime profile not found)', name: 'iPhone 5', udid: '1CCBBF8B-5773-4EA6-BD6F-C308C87A1ADB', }, ... ] } ``` After: ```js devices: { 'iOS 12.1': [ { state: 'Shutdown', isAvailable: 'YES', name: 'iPhone 6s', udid: 'D0F29BE7-CC3C-4976-888D-C739B4F50508', }, { state: 'Shutdown', isAvailable: 'YES', name: 'iPhone 6', udid: 'BA0D93BD-07E6-4182-9B0A-F60A2474139C', }, ... ] } ``` Without this fix, `npm run ios` is not able to launch the simulator correctly and returns the following error: ``` Could not find iPhone 6 simulator Error: Could not find iPhone 6 simulator at resolve (/Users/antonyc/Projects/AwesomeProject/node_modules/react-native/local-cli/runIOS/runIOS.js:148:13) at new Promise () at runOnSimulator (/Users/antonyc/Projects/AwesomeProject/node_modules/react-native/local-cli/runIOS/runIOS.js:134:10) at Object.runIOS [as func] (/Users/antonyc/Projects/AwesomeProject/node_modules/react-native/local-cli/runIOS/runIOS.js:106:12) at Promise.resolve.then (/Users/antonyc/Projects/AwesomeProject/node_modules/react-native/local-cli/cliEntry.js:117:22) ``` Pull Request resolved: https://github.com/facebook/react-native/pull/21557 Differential Revision: D10248115 Pulled By: TheSavior fbshipit-source-id: 37197bbf828e4a6e254270d46411a6716a91afe3 --- .../__tests__/findMatchingSimulator-test.js | 52 +++++++++++++++++++ local-cli/runIOS/findMatchingSimulator.js | 5 +- 2 files changed, 56 insertions(+), 1 deletion(-) diff --git a/local-cli/runIOS/__tests__/findMatchingSimulator-test.js b/local-cli/runIOS/__tests__/findMatchingSimulator-test.js index 1729177df51334..645883412b007c 100644 --- a/local-cli/runIOS/__tests__/findMatchingSimulator-test.js +++ b/local-cli/runIOS/__tests__/findMatchingSimulator-test.js @@ -65,6 +65,58 @@ describe('findMatchingSimulator', () => { }); }); + it('should find simulator with new xcrun format', () => { + expect( + findMatchingSimulator( + { + devices: { + 'iOS 12.1': [ + { + state: 'Shutdown', + isAvailable: 'YES', + name: 'iPhone 6s', + udid: 'D0F29BE7-CC3C-4976-888D-C739B4F50508', + }, + { + state: 'Shutdown', + isAvailable: 'YES', + name: 'iPhone 6', + udid: 'BA0D93BD-07E6-4182-9B0A-F60A2474139C', + }, + { + state: 'Shutdown', + isAvailable: 'YES', + name: 'iPhone XS Max', + udid: 'B9B5E161-416B-43C4-A78F-729CB96CC8C6', + availabilityError: '', + }, + { + state: 'Shutdown', + isAvailable: 'YES', + name: 'iPad Air', + udid: '1CCBBF8B-5773-4EA6-BD6F-C308C87A1ADB', + availabilityError: '', + }, + { + state: 'Shutdown', + isAvailable: 'YES', + name: 'iPad (5th generation)', + udid: '9564ABEE-9EC2-4B4A-B443-D3710929A45A', + availabilityError: '', + }, + ], + }, + }, + 'iPhone 6', + ), + ).toEqual({ + udid: 'BA0D93BD-07E6-4182-9B0A-F60A2474139C', + name: 'iPhone 6', + booted: false, + version: 'iOS 12.1', + }); + }); + it('should return null if no simulators available', () => { expect( findMatchingSimulator( diff --git a/local-cli/runIOS/findMatchingSimulator.js b/local-cli/runIOS/findMatchingSimulator.js index c80102e6d7767b..f336f96a6c0f8f 100644 --- a/local-cli/runIOS/findMatchingSimulator.js +++ b/local-cli/runIOS/findMatchingSimulator.js @@ -34,7 +34,10 @@ function findMatchingSimulator(simulators, simulatorName) { for (let i in devices[version]) { let simulator = devices[version][i]; // Skipping non-available simulator - if (simulator.availability !== '(available)') { + if ( + simulator.availability !== '(available)' && + simulator.isAvailable !== 'YES' + ) { continue; } let booted = simulator.state === 'Booted';