Skip to content

Commit 0875988

Browse files
authored
Migrate flutter_command_test.dart to explicit-package-dependencies. (#160274)
Work towards flutter/flutter#160257.
1 parent 5e7c856 commit 0875988

File tree

1 file changed

+16
-0
lines changed

1 file changed

+16
-0
lines changed

packages/flutter_tools/test/general.shard/runner/flutter_command_test.dart

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@ import 'package:flutter_tools/src/cache.dart';
2121
import 'package:flutter_tools/src/commands/run.dart';
2222
import 'package:flutter_tools/src/dart/pub.dart';
2323
import 'package:flutter_tools/src/device.dart';
24+
import 'package:flutter_tools/src/features.dart';
2425
import 'package:flutter_tools/src/globals.dart' as globals;
2526
import 'package:flutter_tools/src/pre_run_validator.dart';
2627
import 'package:flutter_tools/src/project.dart';
@@ -33,11 +34,20 @@ import 'package:unified_analytics/unified_analytics.dart';
3334
import '../../src/common.dart';
3435
import '../../src/context.dart';
3536
import '../../src/fake_devices.dart';
37+
import '../../src/fake_pub_deps.dart';
3638
import '../../src/fakes.dart';
3739
import '../../src/test_flutter_command_runner.dart';
3840
import 'utils.dart';
3941

4042
void main() {
43+
// TODO(matanlurey): Remove after `explicit-package-dependencies` is enabled by default.
44+
// See https://github.com/flutter/flutter/issues/160257 for details.
45+
FeatureFlags enableExplicitPackageDependencies() {
46+
return TestFeatureFlags(
47+
isExplicitPackageDependenciesEnabled: true,
48+
);
49+
}
50+
4151
group('Flutter Command', () {
4252
late FakeCache cache;
4353
late TestUsage usage;
@@ -638,6 +648,7 @@ void main() {
638648
Usage: () => usage,
639649
FileSystem: () => fileSystem,
640650
ProcessManager: () => FakeProcessManager.any(),
651+
FeatureFlags: enableExplicitPackageDependencies,
641652
});
642653

643654
testUsingContext('use packagesPath to generate BuildInfo', () async {
@@ -1410,6 +1421,11 @@ class FakePub extends Fake implements Pub {
14101421
bool shouldSkipThirdPartyGenerator = true,
14111422
PubOutputMode outputMode = PubOutputMode.all,
14121423
}) async { }
1424+
1425+
@override
1426+
Future<Map<String, Object?>> deps(FlutterProject project) {
1427+
return FakePubWithPrimedDeps().deps(project);
1428+
}
14131429
}
14141430

14151431
class _TestDeviceManager extends DeviceManager {

0 commit comments

Comments
 (0)