github/flutter
7
0
Fork 0
mirror of https://github.com/flutter/flutter.git synced 2025-06-03 00:51:18 +00:00
Code Issues Packages Projects Releases Wiki Activity

Remove left-over traces of "link-dry-run" - which isn't used anywhere in flutter (#155820)

Browse Source 
The "link-dry-run" functionality was never used in flutter (even before
the recent refactoring).
I think we can remove this "link-dry-run" concept everywhere.

PR to remove this in dart-lang/native:
https://github.com/dart-lang/native/pull/1613
This commit is contained in:
Martin Kustermann 2024-09-27 16:45:35 +02:00 committed by GitHub
parent 9be22b58c4
commit ead6b0d17c
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
3 changed files with 3 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
packages/flutter_tools/test/general.shard/isolated/fake_native_assets_build_runner.dart
View File

@ -15,7 +15,8 @@ import 'package:package_config/package_config_types.dart';
/// Mocks all logic instead of using `package:native_assets_builder`, which /// Mocks all logic instead of using `package:native_assets_builder`, which
/// relies on doing process calls to `pub` and the local file system. /// relies on doing process calls to `pub` and the local file system.
class FakeFlutterNativeAssetsBuildRunner implements FlutterNativeAssetsBuildRunner { class FakeFlutterNativeAssetsBuildRunner
implements FlutterNativeAssetsBuildRunner {
FakeFlutterNativeAssetsBuildRunner({ FakeFlutterNativeAssetsBuildRunner({
this.hasPackageConfigResult = true, this.hasPackageConfigResult = true,
this.packagesWithNativeAssetsResult = const <Package>[], this.packagesWithNativeAssetsResult = const <Package>[],
@ -23,7 +24,6 @@ class FakeFlutterNativeAssetsBuildRunner implements FlutterNativeAssetsBuildRunn
this.buildDryRunResult = const FakeFlutterNativeAssetsBuilderResult(), this.buildDryRunResult = const FakeFlutterNativeAssetsBuilderResult(),
this.buildResult = const FakeFlutterNativeAssetsBuilderResult(), this.buildResult = const FakeFlutterNativeAssetsBuilderResult(),
this.linkResult = const FakeFlutterNativeAssetsBuilderResult(), this.linkResult = const FakeFlutterNativeAssetsBuilderResult(),
this.linkDryRunResult = const FakeFlutterNativeAssetsBuilderResult(),
CCompilerConfigImpl? cCompilerConfigResult, CCompilerConfigImpl? cCompilerConfigResult,
CCompilerConfigImpl? ndkCCompilerConfigImplResult, CCompilerConfigImpl? ndkCCompilerConfigImplResult,
}) : cCompilerConfigResult = cCompilerConfigResult ?? CCompilerConfigImpl(), }) : cCompilerConfigResult = cCompilerConfigResult ?? CCompilerConfigImpl(),
@ -34,7 +34,6 @@ class FakeFlutterNativeAssetsBuildRunner implements FlutterNativeAssetsBuildRunn
final native_assets_builder.BuildResult buildResult; final native_assets_builder.BuildResult buildResult;
final native_assets_builder.LinkResult linkResult; final native_assets_builder.LinkResult linkResult;
final native_assets_builder.BuildDryRunResult buildDryRunResult; final native_assets_builder.BuildDryRunResult buildDryRunResult;
final native_assets_builder.LinkDryRunResult linkDryRunResult;
final bool hasPackageConfigResult; final bool hasPackageConfigResult;
final List<Package> packagesWithNativeAssetsResult; final List<Package> packagesWithNativeAssetsResult;
final CCompilerConfigImpl cCompilerConfigResult; final CCompilerConfigImpl cCompilerConfigResult;
@ -43,7 +42,6 @@ class FakeFlutterNativeAssetsBuildRunner implements FlutterNativeAssetsBuildRunn
int buildInvocations = 0; int buildInvocations = 0;
int buildDryRunInvocations = 0; int buildDryRunInvocations = 0;
int linkInvocations = 0; int linkInvocations = 0;
int linkDryRunInvocations = 0;
int hasPackageConfigInvocations = 0; int hasPackageConfigInvocations = 0;
int packagesWithNativeAssetsInvocations = 0; int packagesWithNativeAssetsInvocations = 0;
BuildModeImpl? lastBuildMode; BuildModeImpl? lastBuildMode;
@ -122,8 +120,7 @@ final class FakeFlutterNativeAssetsBuilderResult
implements implements
native_assets_builder.BuildResult, native_assets_builder.BuildResult,
native_assets_builder.BuildDryRunResult, native_assets_builder.BuildDryRunResult,
native_assets_builder.LinkResult, native_assets_builder.LinkResult {
native_assets_builder.LinkDryRunResult {
const FakeFlutterNativeAssetsBuilderResult({ const FakeFlutterNativeAssetsBuilderResult({
this.assets = const <AssetImpl>[], this.assets = const <AssetImpl>[],
this.assetsForLinking = const <String, List<AssetImpl>>{}, this.assetsForLinking = const <String, List<AssetImpl>>{},

1
packages/flutter_tools/test/general.shard/isolated/hot_test.dart
View File

@ -97,7 +97,6 @@ void main() {
expect(buildRunner.buildInvocations, 0); expect(buildRunner.buildInvocations, 0);
expect(buildRunner.buildDryRunInvocations, 0); expect(buildRunner.buildDryRunInvocations, 0);
expect(buildRunner.linkInvocations, 0); expect(buildRunner.linkInvocations, 0);
expect(buildRunner.linkDryRunInvocations, 0);
expect(buildRunner.hasPackageConfigInvocations, 0); expect(buildRunner.hasPackageConfigInvocations, 0);
expect(buildRunner.packagesWithNativeAssetsInvocations, 0); expect(buildRunner.packagesWithNativeAssetsInvocations, 0);
}, overrides: <Type, Generator>{ }, overrides: <Type, Generator>{

1
packages/flutter_tools/test/general.shard/isolated/resident_runner_test.dart
View File

@ -82,7 +82,6 @@ void main() {
expect(buildRunner.buildInvocations, 0); expect(buildRunner.buildInvocations, 0);
expect(buildRunner.buildDryRunInvocations, 0); expect(buildRunner.buildDryRunInvocations, 0);
expect(buildRunner.linkInvocations, 0); expect(buildRunner.linkInvocations, 0);
expect(buildRunner.linkDryRunInvocations, 0);
expect(buildRunner.hasPackageConfigInvocations, 0); expect(buildRunner.hasPackageConfigInvocations, 0);
expect(buildRunner.packagesWithNativeAssetsInvocations, 0); expect(buildRunner.packagesWithNativeAssetsInvocations, 0);