Unverified Commit c48c428e authored by Jenn Magder's avatar Jenn Magder Committed by GitHub

Xcode 13 as minimum recommended version (#90906)

parent afd84ad4
...@@ -22,7 +22,7 @@ Version get xcodeRequiredVersion => Version(12, 0, 1, text: '12.0.1'); ...@@ -22,7 +22,7 @@ Version get xcodeRequiredVersion => Version(12, 0, 1, text: '12.0.1');
/// Diverging this number from the minimum required version will provide a doctor /// Diverging this number from the minimum required version will provide a doctor
/// warning, not error, that users should upgrade Xcode. /// warning, not error, that users should upgrade Xcode.
Version get xcodeRecommendedVersion => xcodeRequiredVersion; Version get xcodeRecommendedVersion => Version(13, 0, 0, text: '13.0.0');
/// SDK name passed to `xcrun --sdk`. Corresponds to undocumented Xcode /// SDK name passed to `xcrun --sdk`. Corresponds to undocumented Xcode
/// SUPPORTED_PLATFORMS values. /// SUPPORTED_PLATFORMS values.
......
...@@ -189,28 +189,28 @@ void main() { ...@@ -189,28 +189,28 @@ void main() {
testWithoutContext('isRecommendedVersionSatisfactory is true when version meets minimum', () { testWithoutContext('isRecommendedVersionSatisfactory is true when version meets minimum', () {
xcodeProjectInterpreter.isInstalled = true; xcodeProjectInterpreter.isInstalled = true;
xcodeProjectInterpreter.version = Version(12, 0, 1); xcodeProjectInterpreter.version = Version(13, 0, 0);
expect(xcode.isRecommendedVersionSatisfactory, isTrue); expect(xcode.isRecommendedVersionSatisfactory, isTrue);
}); });
testWithoutContext('isRecommendedVersionSatisfactory is true when major version exceeds minimum', () { testWithoutContext('isRecommendedVersionSatisfactory is true when major version exceeds minimum', () {
xcodeProjectInterpreter.isInstalled = true; xcodeProjectInterpreter.isInstalled = true;
xcodeProjectInterpreter.version = Version(13, 0, 0); xcodeProjectInterpreter.version = Version(14, 0, 0);
expect(xcode.isRecommendedVersionSatisfactory, isTrue); expect(xcode.isRecommendedVersionSatisfactory, isTrue);
}); });
testWithoutContext('isRecommendedVersionSatisfactory is true when minor version exceeds minimum', () { testWithoutContext('isRecommendedVersionSatisfactory is true when minor version exceeds minimum', () {
xcodeProjectInterpreter.isInstalled = true; xcodeProjectInterpreter.isInstalled = true;
xcodeProjectInterpreter.version = Version(12, 3, 0); xcodeProjectInterpreter.version = Version(13, 3, 0);
expect(xcode.isRecommendedVersionSatisfactory, isTrue); expect(xcode.isRecommendedVersionSatisfactory, isTrue);
}); });
testWithoutContext('isRecommendedVersionSatisfactory is true when patch version exceeds minimum', () { testWithoutContext('isRecommendedVersionSatisfactory is true when patch version exceeds minimum', () {
xcodeProjectInterpreter.isInstalled = true; xcodeProjectInterpreter.isInstalled = true;
xcodeProjectInterpreter.version = Version(12, 0, 2); xcodeProjectInterpreter.version = Version(13, 0, 2);
expect(xcode.isRecommendedVersionSatisfactory, isTrue); expect(xcode.isRecommendedVersionSatisfactory, isTrue);
}); });
......
...@@ -69,8 +69,8 @@ void main() { ...@@ -69,8 +69,8 @@ void main() {
final ValidationResult result = await validator.validate(); final ValidationResult result = await validator.validate();
expect(result.type, ValidationType.partial); expect(result.type, ValidationType.partial);
expect(result.messages.last.type, ValidationMessageType.hint); expect(result.messages.last.type, ValidationMessageType.hint);
expect(result.messages.last.message, contains('Flutter recommends a minimum Xcode version of 12.0.2')); expect(result.messages.last.message, contains('Flutter recommends a minimum Xcode version of 13.0.0'));
}, skip: true); // [intended] Unskip and update when minimum and required check versions diverge. });
testWithoutContext('Emits partial status when Xcode EULA not signed', () async { testWithoutContext('Emits partial status when Xcode EULA not signed', () async {
final ProcessManager processManager = FakeProcessManager.list(<FakeCommand>[ final ProcessManager processManager = FakeProcessManager.list(<FakeCommand>[
......
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