Skip to content

Commit 15517f0

Browse files
authored
Migrate hot_test.dart to explicit-package-dependencies. (#160277)
Work towards flutter/flutter#160257.
1 parent 6bcb15d commit 15517f0

File tree

1 file changed

+15
-0
lines changed

1 file changed

+15
-0
lines changed

packages/flutter_tools/test/general.shard/hot_test.dart

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,8 +8,10 @@ import 'package:flutter_tools/src/base/file_system.dart';
88
import 'package:flutter_tools/src/base/io.dart';
99
import 'package:flutter_tools/src/base/platform.dart';
1010
import 'package:flutter_tools/src/build_info.dart';
11+
import 'package:flutter_tools/src/dart/pub.dart';
1112
import 'package:flutter_tools/src/devfs.dart';
1213
import 'package:flutter_tools/src/device.dart';
14+
import 'package:flutter_tools/src/features.dart';
1315
import 'package:flutter_tools/src/reporting/reporting.dart';
1416
import 'package:flutter_tools/src/resident_devtools_handler.dart';
1517
import 'package:flutter_tools/src/resident_runner.dart';
@@ -20,10 +22,19 @@ import 'package:vm_service/vm_service.dart' as vm_service;
2022

2123
import '../src/common.dart';
2224
import '../src/context.dart';
25+
import '../src/fake_pub_deps.dart';
2326
import '../src/fakes.dart';
2427
import 'hot_shared.dart';
2528

2629
void main() {
30+
// TODO(matanlurey): Remove after `explicit-package-dependencies` is enabled by default.
31+
// See https://github.com/flutter/flutter/issues/160257 for details.
32+
FeatureFlags enableExplicitPackageDependencies() {
33+
return TestFeatureFlags(
34+
isExplicitPackageDependenciesEnabled: true,
35+
);
36+
}
37+
2738
group('validateReloadReport', () {
2839
testUsingContext('invalid', () async {
2940
expect(HotRunner.validateReloadReport(vm_service.ReloadReport.parse(<String, dynamic>{
@@ -160,6 +171,8 @@ void main() {
160171
FileSystem: () => fileSystem,
161172
Platform: () => FakePlatform(),
162173
ProcessManager: () => FakeProcessManager.any(),
174+
FeatureFlags: enableExplicitPackageDependencies,
175+
Pub: FakePubWithPrimedDeps.new,
163176
});
164177

165178
testUsingContext('setupHotReload function fails', () async {
@@ -202,6 +215,8 @@ void main() {
202215
FileSystem: () => fileSystem,
203216
Platform: () => FakePlatform(),
204217
ProcessManager: () => FakeProcessManager.any(),
218+
FeatureFlags: enableExplicitPackageDependencies,
219+
Pub: FakePubWithPrimedDeps.new,
205220
});
206221
});
207222

0 commit comments

Comments
 (0)