From a14ac4e3c8b7d9ee0147fd2cebc0fae5bc4011cf Mon Sep 17 00:00:00 2001 From: Jenn Magder Date: Fri, 6 Mar 2020 12:14:42 -0800 Subject: [PATCH] Bump recommended CocoaPods version to 1.8 (#52071) --- .../flutter_tools/lib/src/macos/cocoapods.dart | 2 +- .../test/general.shard/macos/cocoapods_test.dart | 14 +++++++++++--- 2 files changed, 12 insertions(+), 4 deletions(-) diff --git a/packages/flutter_tools/lib/src/macos/cocoapods.dart b/packages/flutter_tools/lib/src/macos/cocoapods.dart index 4283826dc1a..121e846fb97 100644 --- a/packages/flutter_tools/lib/src/macos/cocoapods.dart +++ b/packages/flutter_tools/lib/src/macos/cocoapods.dart @@ -69,7 +69,7 @@ class CocoaPods { Future _versionText; String get cocoaPodsMinimumVersion => '1.6.0'; - String get cocoaPodsRecommendedVersion => '1.6.0'; + String get cocoaPodsRecommendedVersion => '1.8.0'; Future get isInstalled => processUtils.exitsHappy(['which', 'pod']); diff --git a/packages/flutter_tools/test/general.shard/macos/cocoapods_test.dart b/packages/flutter_tools/test/general.shard/macos/cocoapods_test.dart index 43afe85763d..b1e78a48e9c 100644 --- a/packages/flutter_tools/test/general.shard/macos/cocoapods_test.dart +++ b/packages/flutter_tools/test/general.shard/macos/cocoapods_test.dart @@ -68,7 +68,7 @@ void main() { projectUnderTest = FlutterProject.fromDirectory(fs.directory('project')); projectUnderTest.ios.xcodeProject.createSync(recursive: true); cocoaPodsUnderTest = CocoaPods(); - pretendPodVersionIs('1.6.0'); + pretendPodVersionIs('1.8.0'); fs.file(fs.path.join( Cache.flutterRoot, 'packages', 'flutter_tools', 'templates', 'cocoapods', 'Podfile-ios-objc', )) @@ -173,9 +173,17 @@ void main() { ProcessManager: () => mockProcessManager, }); - testUsingContext('detects at recommended version', () async { + testUsingContext('detects below recommended version', () async { pretendPodIsInstalled(); pretendPodVersionIs('1.6.0'); + expect(await cocoaPodsUnderTest.evaluateCocoaPodsInstallation, CocoaPodsStatus.belowRecommendedVersion); + }, overrides: { + ProcessManager: () => mockProcessManager, + }); + + testUsingContext('detects at recommended version', () async { + pretendPodIsInstalled(); + pretendPodVersionIs('1.8.0'); expect(await cocoaPodsUnderTest.evaluateCocoaPodsInstallation, CocoaPodsStatus.recommended); }, overrides: { ProcessManager: () => mockProcessManager, @@ -183,7 +191,7 @@ void main() { testUsingContext('detects above recommended version', () async { pretendPodIsInstalled(); - pretendPodVersionIs('1.6.1'); + pretendPodVersionIs('1.8.1'); expect(await cocoaPodsUnderTest.evaluateCocoaPodsInstallation, CocoaPodsStatus.recommended); }, overrides: { ProcessManager: () => mockProcessManager,