commit 5141c1e8e944a1474fd324dd7c1bb5133a896fdd
author Chris Bracken <chris@bracken.jp> Mon Jun 26 13:11:49 2017 -0700
committer GitHub <noreply@github.com> Mon Jun 26 13:11:49 2017 -0700
tree 62fa67227b2faec7866a241c472b0ee58d9dab26
parent dbb734ddb5407711ba9e95451f303a4057a9f88a

Add test for unknown Xcode version string (#10979)

Also, correct an existing test description.

packages/flutter_tools/test/ios/mac_test.dart

diff --git a/packages/flutter_tools/test/ios/mac_test.dart b/packages/flutter_tools/test/ios/mac_test.dart
index fbc5603..9313030 100644
--- a/packages/flutter_tools/test/ios/mac_test.dart
+++ b/packages/flutter_tools/test/ios/mac_test.dart
@@ -100,7 +100,7 @@
       ProcessManager: () => mockProcessManager,
     });
 
-    testUsingContext('xcodeVersionText returns null when xcodebuild is not installed', () {
+    testUsingContext('xcodeVersionText returns formatted version text', () {
       when(mockProcessManager.runSync(<String>['/usr/bin/xcodebuild', '-version']))
           .thenReturn(new ProcessResult(1, 0, 'Xcode 8.3.3\nBuild version 8E3004b', ''));
       expect(xcode.xcodeVersionText, 'Xcode 8.3.3, Build version 8E3004b');
@@ -108,6 +108,14 @@
       ProcessManager: () => mockProcessManager,
     });
 
+    testUsingContext('xcodeVersionText handles Xcode version string with unexpected format', () {
+      when(mockProcessManager.runSync(<String>['/usr/bin/xcodebuild', '-version']))
+          .thenReturn(new ProcessResult(1, 0, 'Xcode Ultra5000\nBuild version 8E3004b', ''));
+      expect(xcode.xcodeVersionText, 'Xcode Ultra5000, Build version 8E3004b');
+    }, overrides: <Type, Generator>{
+      ProcessManager: () => mockProcessManager,
+    });
+
     testUsingContext('eulaSigned is false when clang is not installed', () {
       when(mockProcessManager.runSync(<String>['/usr/bin/xcrun', 'clang']))
           .thenThrow(const ProcessException('/usr/bin/xcrun', const <String>['clang']));