Unverified Commit c7cb5f3f authored by Christopher Fujino's avatar Christopher Fujino Committed by GitHub

[flutter_tools] pin package intl and roll pub packages (#117168)

* pin package intl and roll pub packages

* add test

* fix analysis
parent a34e4194
...@@ -7,7 +7,7 @@ environment: ...@@ -7,7 +7,7 @@ environment:
dependencies: dependencies:
flutter: flutter:
sdk: flutter sdk: flutter
camera: 0.10.0+4 camera: 0.10.0+5
camera_android: 0.10.1 # THIS LINE IS AUTOGENERATED - TO UPDATE USE "flutter update-packages --force-upgrade" camera_android: 0.10.1 # THIS LINE IS AUTOGENERATED - TO UPDATE USE "flutter update-packages --force-upgrade"
camera_avfoundation: 0.9.9 # THIS LINE IS AUTOGENERATED - TO UPDATE USE "flutter update-packages --force-upgrade" camera_avfoundation: 0.9.9 # THIS LINE IS AUTOGENERATED - TO UPDATE USE "flutter update-packages --force-upgrade"
...@@ -31,4 +31,4 @@ dependencies: ...@@ -31,4 +31,4 @@ dependencies:
flutter: flutter:
uses-material-design: true uses-material-design: true
# PUBSPEC CHECKSUM: 7ef1 # PUBSPEC CHECKSUM: 6cf2
...@@ -41,6 +41,8 @@ const Map<String, String> kManuallyPinnedDependencies = <String, String>{ ...@@ -41,6 +41,8 @@ const Map<String, String> kManuallyPinnedDependencies = <String, String>{
'archive': '3.3.2', 'archive': '3.3.2',
// https://github.com/flutter/flutter/issues/116376 // https://github.com/flutter/flutter/issues/116376
'path_provider_android': '2.0.21', 'path_provider_android': '2.0.21',
// https://github.com/flutter/flutter/issues/117163
'intl': '0.17.0',
}; };
class UpdatePackagesCommand extends FlutterCommand { class UpdatePackagesCommand extends FlutterCommand {
......
...@@ -12,7 +12,7 @@ dependencies: ...@@ -12,7 +12,7 @@ dependencies:
browser_launcher: 1.1.1 browser_launcher: 1.1.1
dds: 2.5.0 dds: 2.5.0
dwds: 16.0.1 dwds: 16.0.1
completion: 1.0.0 completion: 1.0.1
coverage: 1.6.1 coverage: 1.6.1
crypto: 3.0.2 crypto: 3.0.2
file: 6.1.4 file: 6.1.4
...@@ -102,4 +102,4 @@ dartdoc: ...@@ -102,4 +102,4 @@ dartdoc:
# Exclude this package from the hosted API docs. # Exclude this package from the hosted API docs.
nodoc: true nodoc: true
# PUBSPEC CHECKSUM: 1fee # PUBSPEC CHECKSUM: 59ef
...@@ -13,7 +13,6 @@ import 'package:flutter_tools/src/project.dart'; ...@@ -13,7 +13,6 @@ import 'package:flutter_tools/src/project.dart';
import 'package:test/fake.dart'; import 'package:test/fake.dart';
import 'package:yaml/yaml.dart'; import 'package:yaml/yaml.dart';
import '../../src/common.dart';
import '../../src/context.dart'; import '../../src/context.dart';
import '../../src/test_flutter_command_runner.dart'; import '../../src/test_flutter_command_runner.dart';
...@@ -75,14 +74,6 @@ dependencies: ...@@ -75,14 +74,6 @@ dependencies:
'''; ''';
void main() { void main() {
testWithoutContext('kManuallyPinnedDependencies pins are actually pins', () {
expect(
kManuallyPinnedDependencies.values,
isNot(contains(anyOf('any', startsWith('^'), startsWith('>'), startsWith('<')))),
reason: 'Version pins in kManuallyPinnedDependencies must be specific pins, not ranges.',
);
});
group('update-packages', () { group('update-packages', () {
late FileSystem fileSystem; late FileSystem fileSystem;
late Directory flutterSdk; late Directory flutterSdk;
......
...@@ -93,6 +93,27 @@ void main() { ...@@ -93,6 +93,27 @@ void main() {
flutter.childFile('pubspec.yaml').writeAsStringSync(kFlutterPubspecYaml); flutter.childFile('pubspec.yaml').writeAsStringSync(kFlutterPubspecYaml);
}); });
testWithoutContext('kManuallyPinnedDependencies pins are actually pins', () {
expect(
kManuallyPinnedDependencies.values,
isNot(contains(anyOf('any', startsWith('^'), startsWith('>'), startsWith('<')))),
reason: 'Version pins in kManuallyPinnedDependencies must be specific pins, not ranges.',
);
expect(
kManuallyPinnedDependencies.keys,
unorderedEquals(const <String>[
'flutter_gallery_assets',
'flutter_template_images',
'video_player',
'material_color_utilities',
'url_launcher_android',
'archive',
'path_provider_android',
'intl',
]),
);
});
testWithoutContext( testWithoutContext(
'createTemporaryFlutterSdk creates an unpinned flutter SDK', () { 'createTemporaryFlutterSdk creates an unpinned flutter SDK', () {
// A stray extra package should not cause a crash. // A stray extra package should not cause a crash.
......
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