Unverified Commit 4e70bfae authored by Daco Harkes's avatar Daco Harkes Committed by GitHub

Reland "Move native assets to `isolated/` directory" (#143055)

Reland of https://github.com/flutter/flutter/pull/142709.

The revert of the revert is in the first commit, the fix in the commit on top.

The move of the fakes for packages/flutter_tools/test/general.shard/resident_runner_test.dart was erroneous before, as it was trying to use setters instead of a private field. This PR changes the private `_devFS` field in the fake to be a public `fakeDevFS` in line with other fakes.

## Original PR description

Native assets in other build systems are not built with `package:native_assets_builder` invoking `build.dart` scripts. Instead all packages have their own blaze rules. Therefore we'd like to not depend on `package:native_assets_builder` from flutter tools in g3 at all.

This PR aims to move the imports of `native_assets_builder` and `native_assets_cli` into the `isolated/` directory and into the files with a `main` function that are not used in with other build systems.

In order to be able to remove all imports in files used by other build systems, two new interfaces are added `HotRunnerNativeAssetsBuilder` and `TestCompilerNativeAssetsBuilder`. New parameters are then piped all the way through from the entry points:

* bin/fuchsia_tester.dart
* lib/executable.dart

The build_system/targets dir is already excluded in other build systems.

So, after this PR only the two above files and build_system/targets import from `isolated/native_assets/` and only `isolated/native_assets/` import `package:native_assets_cli` and `package:native_assets_builder`.

Context:

* https://github.com/flutter/flutter/issues/142041
parent 38c6a5ac
...@@ -16,6 +16,7 @@ import 'package:flutter_tools/src/cache.dart'; ...@@ -16,6 +16,7 @@ import 'package:flutter_tools/src/cache.dart';
import 'package:flutter_tools/src/context_runner.dart'; import 'package:flutter_tools/src/context_runner.dart';
import 'package:flutter_tools/src/device.dart'; import 'package:flutter_tools/src/device.dart';
import 'package:flutter_tools/src/globals.dart' as globals; import 'package:flutter_tools/src/globals.dart' as globals;
import 'package:flutter_tools/src/isolated/native_assets/test/native_assets.dart';
import 'package:flutter_tools/src/project.dart'; import 'package:flutter_tools/src/project.dart';
import 'package:flutter_tools/src/reporting/reporting.dart'; import 'package:flutter_tools/src/reporting/reporting.dart';
import 'package:flutter_tools/src/test/coverage_collector.dart'; import 'package:flutter_tools/src/test/coverage_collector.dart';
...@@ -153,6 +154,7 @@ Future<void> run(List<String> args) async { ...@@ -153,6 +154,7 @@ Future<void> run(List<String> args) async {
concurrency: math.max(1, globals.platform.numberOfProcessors - 2), concurrency: math.max(1, globals.platform.numberOfProcessors - 2),
icudtlPath: globals.fs.path.absolute(argResults[_kOptionIcudtl] as String), icudtlPath: globals.fs.path.absolute(argResults[_kOptionIcudtl] as String),
coverageDirectory: coverageDirectory, coverageDirectory: coverageDirectory,
nativeAssetsBuilder: const TestCompilerNativeAssetsBuilderImpl(),
); );
if (collector != null) { if (collector != null) {
......
...@@ -50,6 +50,8 @@ import 'src/globals.dart' as globals; ...@@ -50,6 +50,8 @@ import 'src/globals.dart' as globals;
// Files in `isolated` are intentionally excluded from google3 tooling. // Files in `isolated` are intentionally excluded from google3 tooling.
import 'src/isolated/build_targets.dart'; import 'src/isolated/build_targets.dart';
import 'src/isolated/mustache_template.dart'; import 'src/isolated/mustache_template.dart';
import 'src/isolated/native_assets/native_assets.dart';
import 'src/isolated/native_assets/test/native_assets.dart';
import 'src/isolated/resident_web_runner.dart'; import 'src/isolated/resident_web_runner.dart';
import 'src/pre_run_validator.dart'; import 'src/pre_run_validator.dart';
import 'src/project_validator.dart'; import 'src/project_validator.dart';
...@@ -177,6 +179,7 @@ List<FlutterCommand> generateCommands({ ...@@ -177,6 +179,7 @@ List<FlutterCommand> generateCommands({
platform: globals.platform, platform: globals.platform,
processInfo: globals.processInfo, processInfo: globals.processInfo,
fileSystem: globals.fs, fileSystem: globals.fs,
nativeAssetsBuilder: const HotRunnerNativeAssetsBuilderImpl(),
), ),
BuildCommand( BuildCommand(
artifacts: globals.artifacts!, artifacts: globals.artifacts!,
...@@ -237,10 +240,17 @@ List<FlutterCommand> generateCommands({ ...@@ -237,10 +240,17 @@ List<FlutterCommand> generateCommands({
platform: globals.platform, platform: globals.platform,
featureFlags: featureFlags, featureFlags: featureFlags,
), ),
RunCommand(verboseHelp: verboseHelp), RunCommand(
verboseHelp: verboseHelp,
nativeAssetsBuilder: const HotRunnerNativeAssetsBuilderImpl(),
),
ScreenshotCommand(fs: globals.fs), ScreenshotCommand(fs: globals.fs),
ShellCompletionCommand(), ShellCompletionCommand(),
TestCommand(verboseHelp: verboseHelp, verbose: verbose), TestCommand(
verboseHelp: verboseHelp,
verbose: verbose,
nativeAssetsBuilder: const TestCompilerNativeAssetsBuilderImpl(),
),
UpgradeCommand(verboseHelp: verboseHelp), UpgradeCommand(verboseHelp: verboseHelp),
SymbolizeCommand( SymbolizeCommand(
stdio: globals.stdio, stdio: globals.stdio,
......
...@@ -7,18 +7,18 @@ import 'package:native_assets_cli/native_assets_cli_internal.dart' show Asset; ...@@ -7,18 +7,18 @@ import 'package:native_assets_cli/native_assets_cli_internal.dart' show Asset;
import 'package:package_config/package_config_types.dart'; import 'package:package_config/package_config_types.dart';
import '../../android/gradle_utils.dart'; import '../../android/gradle_utils.dart';
import '../../android/native_assets.dart';
import '../../base/common.dart'; import '../../base/common.dart';
import '../../base/file_system.dart'; import '../../base/file_system.dart';
import '../../base/platform.dart'; import '../../base/platform.dart';
import '../../build_info.dart'; import '../../build_info.dart';
import '../../dart/package_map.dart'; import '../../dart/package_map.dart';
import '../../ios/native_assets.dart'; import '../../isolated/native_assets/android/native_assets.dart';
import '../../linux/native_assets.dart'; import '../../isolated/native_assets/ios/native_assets.dart';
import '../../macos/native_assets.dart'; import '../../isolated/native_assets/linux/native_assets.dart';
import '../../isolated/native_assets/macos/native_assets.dart';
import '../../isolated/native_assets/native_assets.dart';
import '../../isolated/native_assets/windows/native_assets.dart';
import '../../macos/xcode.dart'; import '../../macos/xcode.dart';
import '../../native_assets.dart';
import '../../windows/native_assets.dart';
import '../build_system.dart'; import '../build_system.dart';
import '../depfile.dart'; import '../depfile.dart';
import '../exceptions.dart'; import '../exceptions.dart';
......
...@@ -71,6 +71,7 @@ class AttachCommand extends FlutterCommand { ...@@ -71,6 +71,7 @@ class AttachCommand extends FlutterCommand {
required Platform platform, required Platform platform,
required ProcessInfo processInfo, required ProcessInfo processInfo,
required FileSystem fileSystem, required FileSystem fileSystem,
HotRunnerNativeAssetsBuilder? nativeAssetsBuilder,
}) : _hotRunnerFactory = hotRunnerFactory ?? HotRunnerFactory(), }) : _hotRunnerFactory = hotRunnerFactory ?? HotRunnerFactory(),
_stdio = stdio, _stdio = stdio,
_logger = logger, _logger = logger,
...@@ -78,7 +79,8 @@ class AttachCommand extends FlutterCommand { ...@@ -78,7 +79,8 @@ class AttachCommand extends FlutterCommand {
_signals = signals, _signals = signals,
_platform = platform, _platform = platform,
_processInfo = processInfo, _processInfo = processInfo,
_fileSystem = fileSystem { _fileSystem = fileSystem,
_nativeAssetsBuilder = nativeAssetsBuilder {
addBuildModeFlags(verboseHelp: verboseHelp, defaultToRelease: false, excludeRelease: true); addBuildModeFlags(verboseHelp: verboseHelp, defaultToRelease: false, excludeRelease: true);
usesTargetOption(); usesTargetOption();
usesPortOptions(verboseHelp: verboseHelp); usesPortOptions(verboseHelp: verboseHelp);
...@@ -150,6 +152,7 @@ class AttachCommand extends FlutterCommand { ...@@ -150,6 +152,7 @@ class AttachCommand extends FlutterCommand {
final Platform _platform; final Platform _platform;
final ProcessInfo _processInfo; final ProcessInfo _processInfo;
final FileSystem _fileSystem; final FileSystem _fileSystem;
final HotRunnerNativeAssetsBuilder? _nativeAssetsBuilder;
@override @override
final String name = 'attach'; final String name = 'attach';
...@@ -416,6 +419,7 @@ known, it can be explicitly provided to attach via the command-line, e.g. ...@@ -416,6 +419,7 @@ known, it can be explicitly provided to attach via the command-line, e.g.
device: device, device: device,
flutterProject: flutterProject, flutterProject: flutterProject,
usesIpv6: usesIpv6, usesIpv6: usesIpv6,
nativeAssetsBuilder: _nativeAssetsBuilder,
); );
late AppInstance app; late AppInstance app;
try { try {
...@@ -449,6 +453,7 @@ known, it can be explicitly provided to attach via the command-line, e.g. ...@@ -449,6 +453,7 @@ known, it can be explicitly provided to attach via the command-line, e.g.
device: device, device: device,
flutterProject: flutterProject, flutterProject: flutterProject,
usesIpv6: usesIpv6, usesIpv6: usesIpv6,
nativeAssetsBuilder: _nativeAssetsBuilder,
); );
final Completer<void> onAppStart = Completer<void>.sync(); final Completer<void> onAppStart = Completer<void>.sync();
TerminalHandler? terminalHandler; TerminalHandler? terminalHandler;
...@@ -505,6 +510,7 @@ known, it can be explicitly provided to attach via the command-line, e.g. ...@@ -505,6 +510,7 @@ known, it can be explicitly provided to attach via the command-line, e.g.
required Device device, required Device device,
required FlutterProject flutterProject, required FlutterProject flutterProject,
required bool usesIpv6, required bool usesIpv6,
required HotRunnerNativeAssetsBuilder? nativeAssetsBuilder,
}) async { }) async {
final BuildInfo buildInfo = await getBuildInfo(); final BuildInfo buildInfo = await getBuildInfo();
...@@ -539,6 +545,7 @@ known, it can be explicitly provided to attach via the command-line, e.g. ...@@ -539,6 +545,7 @@ known, it can be explicitly provided to attach via the command-line, e.g.
ipv6: usesIpv6, ipv6: usesIpv6,
flutterProject: flutterProject, flutterProject: flutterProject,
nativeAssetsYamlFile: stringArg(FlutterOptions.kNativeAssetsYamlFile), nativeAssetsYamlFile: stringArg(FlutterOptions.kNativeAssetsYamlFile),
nativeAssetsBuilder: _nativeAssetsBuilder,
analytics: analytics, analytics: analytics,
) )
: ColdRunner( : ColdRunner(
...@@ -573,6 +580,7 @@ class HotRunnerFactory { ...@@ -573,6 +580,7 @@ class HotRunnerFactory {
bool ipv6 = false, bool ipv6 = false,
FlutterProject? flutterProject, FlutterProject? flutterProject,
String? nativeAssetsYamlFile, String? nativeAssetsYamlFile,
required HotRunnerNativeAssetsBuilder? nativeAssetsBuilder,
required Analytics analytics, required Analytics analytics,
}) => HotRunner( }) => HotRunner(
devices, devices,
...@@ -586,6 +594,7 @@ class HotRunnerFactory { ...@@ -586,6 +594,7 @@ class HotRunnerFactory {
stayResident: stayResident, stayResident: stayResident,
ipv6: ipv6, ipv6: ipv6,
nativeAssetsYamlFile: nativeAssetsYamlFile, nativeAssetsYamlFile: nativeAssetsYamlFile,
nativeAssetsBuilder: nativeAssetsBuilder,
analytics: analytics, analytics: analytics,
); );
} }
...@@ -654,6 +654,7 @@ class AppDomain extends Domain { ...@@ -654,6 +654,7 @@ class AppDomain extends Domain {
bool machine = true, bool machine = true,
String? userIdentifier, String? userIdentifier,
bool enableDevTools = true, bool enableDevTools = true,
required HotRunnerNativeAssetsBuilder? nativeAssetsBuilder,
}) async { }) async {
if (!await device.supportsRuntimeMode(options.buildInfo.mode)) { if (!await device.supportsRuntimeMode(options.buildInfo.mode)) {
throw Exception( throw Exception(
...@@ -705,6 +706,7 @@ class AppDomain extends Domain { ...@@ -705,6 +706,7 @@ class AppDomain extends Domain {
hostIsIde: true, hostIsIde: true,
machine: machine, machine: machine,
analytics: globals.analytics, analytics: globals.analytics,
nativeAssetsBuilder: nativeAssetsBuilder,
); );
} else { } else {
runner = ColdRunner( runner = ColdRunner(
......
...@@ -334,7 +334,11 @@ abstract class RunCommandBase extends FlutterCommand with DeviceBasedDevelopment ...@@ -334,7 +334,11 @@ abstract class RunCommandBase extends FlutterCommand with DeviceBasedDevelopment
} }
class RunCommand extends RunCommandBase { class RunCommand extends RunCommandBase {
RunCommand({ bool verboseHelp = false }) : super(verboseHelp: verboseHelp) { RunCommand({
bool verboseHelp = false,
HotRunnerNativeAssetsBuilder? nativeAssetsBuilder,
}) : _nativeAssetsBuilder = nativeAssetsBuilder,
super(verboseHelp: verboseHelp) {
requiresPubspecYaml(); requiresPubspecYaml();
usesFilesystemOptions(hide: !verboseHelp); usesFilesystemOptions(hide: !verboseHelp);
usesExtraDartFlagOptions(verboseHelp: verboseHelp); usesExtraDartFlagOptions(verboseHelp: verboseHelp);
...@@ -418,6 +422,8 @@ class RunCommand extends RunCommandBase { ...@@ -418,6 +422,8 @@ class RunCommand extends RunCommandBase {
); );
} }
final HotRunnerNativeAssetsBuilder? _nativeAssetsBuilder;
@override @override
final String name = 'run'; final String name = 'run';
...@@ -660,6 +666,7 @@ class RunCommand extends RunCommandBase { ...@@ -660,6 +666,7 @@ class RunCommand extends RunCommandBase {
ipv6: ipv6 ?? false, ipv6: ipv6 ?? false,
analytics: globals.analytics, analytics: globals.analytics,
nativeAssetsYamlFile: stringArg(FlutterOptions.kNativeAssetsYamlFile), nativeAssetsYamlFile: stringArg(FlutterOptions.kNativeAssetsYamlFile),
nativeAssetsBuilder: _nativeAssetsBuilder,
); );
} else if (webMode) { } else if (webMode) {
return webRunnerFactory!.createWebRunner( return webRunnerFactory!.createWebRunner(
...@@ -733,6 +740,7 @@ class RunCommand extends RunCommandBase { ...@@ -733,6 +740,7 @@ class RunCommand extends RunCommandBase {
ipv6: ipv6 ?? false, ipv6: ipv6 ?? false,
userIdentifier: userIdentifier, userIdentifier: userIdentifier,
enableDevTools: boolArg(FlutterCommand.kEnableDevTools), enableDevTools: boolArg(FlutterCommand.kEnableDevTools),
nativeAssetsBuilder: _nativeAssetsBuilder,
); );
} on Exception catch (error) { } on Exception catch (error) {
throwToolExit(error.toString()); throwToolExit(error.toString());
......
...@@ -13,6 +13,7 @@ import '../bundle_builder.dart'; ...@@ -13,6 +13,7 @@ import '../bundle_builder.dart';
import '../devfs.dart'; import '../devfs.dart';
import '../device.dart'; import '../device.dart';
import '../globals.dart' as globals; import '../globals.dart' as globals;
import '../native_assets.dart';
import '../project.dart'; import '../project.dart';
import '../runner/flutter_command.dart'; import '../runner/flutter_command.dart';
import '../test/coverage_collector.dart'; import '../test/coverage_collector.dart';
...@@ -64,6 +65,7 @@ class TestCommand extends FlutterCommand with DeviceBasedDevelopmentArtifacts { ...@@ -64,6 +65,7 @@ class TestCommand extends FlutterCommand with DeviceBasedDevelopmentArtifacts {
this.testWrapper = const TestWrapper(), this.testWrapper = const TestWrapper(),
this.testRunner = const FlutterTestRunner(), this.testRunner = const FlutterTestRunner(),
this.verbose = false, this.verbose = false,
this.nativeAssetsBuilder,
}) { }) {
requiresPubspecYaml(); requiresPubspecYaml();
usesPubOption(); usesPubOption();
...@@ -238,6 +240,8 @@ class TestCommand extends FlutterCommand with DeviceBasedDevelopmentArtifacts { ...@@ -238,6 +240,8 @@ class TestCommand extends FlutterCommand with DeviceBasedDevelopmentArtifacts {
/// Interface for running the tester process. /// Interface for running the tester process.
final FlutterTestRunner testRunner; final FlutterTestRunner testRunner;
final TestCompilerNativeAssetsBuilder? nativeAssetsBuilder;
final bool verbose; final bool verbose;
@visibleForTesting @visibleForTesting
...@@ -510,6 +514,7 @@ class TestCommand extends FlutterCommand with DeviceBasedDevelopmentArtifacts { ...@@ -510,6 +514,7 @@ class TestCommand extends FlutterCommand with DeviceBasedDevelopmentArtifacts {
integrationTestDevice: integrationTestDevice, integrationTestDevice: integrationTestDevice,
integrationTestUserIdentifier: stringArg(FlutterOptions.kDeviceUser), integrationTestUserIdentifier: stringArg(FlutterOptions.kDeviceUser),
testTimeRecorder: testTimeRecorder, testTimeRecorder: testTimeRecorder,
nativeAssetsBuilder: nativeAssetsBuilder,
); );
testTimeRecorder?.stop(TestTimePhases.TestRunner, testRunnerTimeRecorderStopwatch!); testTimeRecorder?.stop(TestTimePhases.TestRunner, testRunnerTimeRecorderStopwatch!);
......
The directory structure in here mirrors the directory structure in lib/src/.
The native assets feature lives in lib/src/isolated/ because this code is not
used in other build systems.
...@@ -4,17 +4,17 @@ ...@@ -4,17 +4,17 @@
import 'package:native_assets_builder/native_assets_builder.dart' import 'package:native_assets_builder/native_assets_builder.dart'
show BuildResult, DryRunResult; show BuildResult, DryRunResult;
import 'package:native_assets_cli/native_assets_cli_internal.dart'
hide BuildMode;
import 'package:native_assets_cli/native_assets_cli_internal.dart' import 'package:native_assets_cli/native_assets_cli_internal.dart'
as native_assets_cli; as native_assets_cli;
import 'package:native_assets_cli/native_assets_cli_internal.dart'
hide BuildMode;
import '../base/common.dart'; import '../../../android/android_sdk.dart';
import '../base/file_system.dart'; import '../../../base/common.dart';
import '../build_info.dart'; import '../../../base/file_system.dart';
import '../globals.dart' as globals; import '../../../build_info.dart';
import '../../../globals.dart' as globals;
import '../native_assets.dart'; import '../native_assets.dart';
import 'android_sdk.dart';
/// Dry run the native builds. /// Dry run the native builds.
/// ///
......
...@@ -9,9 +9,9 @@ import 'package:native_assets_cli/native_assets_cli_internal.dart' ...@@ -9,9 +9,9 @@ import 'package:native_assets_cli/native_assets_cli_internal.dart'
import 'package:native_assets_cli/native_assets_cli_internal.dart' import 'package:native_assets_cli/native_assets_cli_internal.dart'
as native_assets_cli; as native_assets_cli;
import '../base/file_system.dart'; import '../../../base/file_system.dart';
import '../build_info.dart'; import '../../../build_info.dart';
import '../globals.dart' as globals; import '../../../globals.dart' as globals;
import '../macos/native_assets_host.dart'; import '../macos/native_assets_host.dart';
import '../native_assets.dart'; import '../native_assets.dart';
......
...@@ -5,11 +5,11 @@ ...@@ -5,11 +5,11 @@
import 'package:native_assets_cli/native_assets_cli_internal.dart' import 'package:native_assets_cli/native_assets_cli_internal.dart'
hide BuildMode; hide BuildMode;
import '../base/common.dart'; import '../../../base/common.dart';
import '../base/file_system.dart'; import '../../../base/file_system.dart';
import '../base/io.dart'; import '../../../base/io.dart';
import '../build_info.dart'; import '../../../build_info.dart';
import '../globals.dart' as globals; import '../../../globals.dart' as globals;
import '../native_assets.dart'; import '../native_assets.dart';
/// Dry run the native builds. /// Dry run the native builds.
......
...@@ -9,9 +9,9 @@ import 'package:native_assets_cli/native_assets_cli_internal.dart' ...@@ -9,9 +9,9 @@ import 'package:native_assets_cli/native_assets_cli_internal.dart'
import 'package:native_assets_cli/native_assets_cli_internal.dart' import 'package:native_assets_cli/native_assets_cli_internal.dart'
as native_assets_cli; as native_assets_cli;
import '../base/file_system.dart'; import '../../../base/file_system.dart';
import '../build_info.dart'; import '../../../build_info.dart';
import '../globals.dart' as globals; import '../../../globals.dart' as globals;
import '../native_assets.dart'; import '../native_assets.dart';
import 'native_assets_host.dart'; import 'native_assets_host.dart';
......
...@@ -7,12 +7,12 @@ ...@@ -7,12 +7,12 @@
import 'package:native_assets_cli/native_assets_cli_internal.dart' import 'package:native_assets_cli/native_assets_cli_internal.dart'
hide BuildMode; hide BuildMode;
import '../base/common.dart'; import '../../../base/common.dart';
import '../base/file_system.dart'; import '../../../base/file_system.dart';
import '../base/io.dart'; import '../../../base/io.dart';
import '../build_info.dart'; import '../../../build_info.dart';
import '../convert.dart'; import '../../../convert.dart';
import '../globals.dart' as globals; import '../../../globals.dart' as globals;
/// Create an `Info.plist` in [target] for a framework with a single dylib. /// Create an `Info.plist` in [target] for a framework with a single dylib.
/// ///
......
// Copyright 2014 The Flutter Authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
// Logic for native assets shared between all host OSes.
import '../../../base/os.dart';
import '../../../base/platform.dart';
import '../../../build_info.dart';
import '../../../globals.dart' as globals;
import '../../../native_assets.dart';
import '../../../project.dart';
import '../linux/native_assets.dart';
import '../macos/native_assets.dart';
import '../native_assets.dart';
import '../windows/native_assets.dart';
class TestCompilerNativeAssetsBuilderImpl
implements TestCompilerNativeAssetsBuilder {
const TestCompilerNativeAssetsBuilderImpl();
@override
Future<Uri?> build(BuildInfo buildInfo) =>
testCompilerBuildNativeAssets(buildInfo);
}
Future<Uri?> testCompilerBuildNativeAssets(BuildInfo buildInfo) async {
Uri? nativeAssetsYaml;
if (!buildInfo.buildNativeAssets) {
nativeAssetsYaml = null;
} else {
final Uri projectUri = FlutterProject.current().directory.uri;
final NativeAssetsBuildRunner buildRunner = NativeAssetsBuildRunnerImpl(
projectUri,
buildInfo.packageConfig,
globals.fs,
globals.logger,
);
if (globals.platform.isMacOS) {
(nativeAssetsYaml, _) = await buildNativeAssetsMacOS(
buildMode: buildInfo.mode,
projectUri: projectUri,
flutterTester: true,
fileSystem: globals.fs,
buildRunner: buildRunner,
);
} else if (globals.platform.isLinux) {
(nativeAssetsYaml, _) = await buildNativeAssetsLinux(
buildMode: buildInfo.mode,
projectUri: projectUri,
flutterTester: true,
fileSystem: globals.fs,
buildRunner: buildRunner,
);
} else if (globals.platform.isWindows) {
final TargetPlatform targetPlatform;
if (globals.os.hostPlatform == HostPlatform.windows_x64) {
targetPlatform = TargetPlatform.windows_x64;
} else {
targetPlatform = TargetPlatform.windows_arm64;
}
(nativeAssetsYaml, _) = await buildNativeAssetsWindows(
buildMode: buildInfo.mode,
targetPlatform: targetPlatform,
projectUri: projectUri,
flutterTester: true,
fileSystem: globals.fs,
buildRunner: buildRunner,
);
} else {
await ensureNoNativeAssetsOrOsIsSupported(
projectUri,
const LocalPlatform().operatingSystem,
globals.fs,
buildRunner,
);
}
}
return nativeAssetsYaml;
}
...@@ -5,11 +5,11 @@ ...@@ -5,11 +5,11 @@
import 'package:native_assets_cli/native_assets_cli_internal.dart' import 'package:native_assets_cli/native_assets_cli_internal.dart'
hide BuildMode; hide BuildMode;
import '../base/file_system.dart'; import '../../../base/file_system.dart';
import '../build_info.dart'; import '../../../build_info.dart';
import '../globals.dart' as globals; import '../../../globals.dart' as globals;
import '../../../windows/visual_studio.dart';
import '../native_assets.dart'; import '../native_assets.dart';
import 'visual_studio.dart';
/// Dry run the native builds. /// Dry run the native builds.
/// ///
...@@ -66,7 +66,6 @@ Future<(Uri? nativeAssetsYaml, List<Uri> dependencies)> ...@@ -66,7 +66,6 @@ Future<(Uri? nativeAssetsYaml, List<Uri> dependencies)>
); );
} }
Future<CCompilerConfig> cCompilerConfigWindows() async { Future<CCompilerConfig> cCompilerConfigWindows() async {
final VisualStudio visualStudio = VisualStudio( final VisualStudio visualStudio = VisualStudio(
fileSystem: globals.fs, fileSystem: globals.fs,
......
...@@ -22,7 +22,6 @@ import 'dart/package_map.dart'; ...@@ -22,7 +22,6 @@ import 'dart/package_map.dart';
import 'devfs.dart'; import 'devfs.dart';
import 'device.dart'; import 'device.dart';
import 'globals.dart' as globals; import 'globals.dart' as globals;
import 'native_assets.dart';
import 'project.dart'; import 'project.dart';
import 'reporting/reporting.dart'; import 'reporting/reporting.dart';
import 'resident_runner.dart'; import 'resident_runner.dart';
...@@ -93,13 +92,13 @@ class HotRunner extends ResidentRunner { ...@@ -93,13 +92,13 @@ class HotRunner extends ResidentRunner {
StopwatchFactory stopwatchFactory = const StopwatchFactory(), StopwatchFactory stopwatchFactory = const StopwatchFactory(),
ReloadSourcesHelper reloadSourcesHelper = defaultReloadSourcesHelper, ReloadSourcesHelper reloadSourcesHelper = defaultReloadSourcesHelper,
ReassembleHelper reassembleHelper = _defaultReassembleHelper, ReassembleHelper reassembleHelper = _defaultReassembleHelper,
NativeAssetsBuildRunner? buildRunner, HotRunnerNativeAssetsBuilder? nativeAssetsBuilder,
String? nativeAssetsYamlFile, String? nativeAssetsYamlFile,
required Analytics analytics, required Analytics analytics,
}) : _stopwatchFactory = stopwatchFactory, }) : _stopwatchFactory = stopwatchFactory,
_reloadSourcesHelper = reloadSourcesHelper, _reloadSourcesHelper = reloadSourcesHelper,
_reassembleHelper = reassembleHelper, _reassembleHelper = reassembleHelper,
_buildRunner = buildRunner, _nativeAssetsBuilder = nativeAssetsBuilder,
_nativeAssetsYamlFile = nativeAssetsYamlFile, _nativeAssetsYamlFile = nativeAssetsYamlFile,
_analytics = analytics, _analytics = analytics,
super( super(
...@@ -139,7 +138,7 @@ class HotRunner extends ResidentRunner { ...@@ -139,7 +138,7 @@ class HotRunner extends ResidentRunner {
String? _sdkName; String? _sdkName;
bool? _emulator; bool? _emulator;
NativeAssetsBuildRunner? _buildRunner; final HotRunnerNativeAssetsBuilder? _nativeAssetsBuilder;
final String? _nativeAssetsYamlFile; final String? _nativeAssetsYamlFile;
String? flavor; String? flavor;
...@@ -374,17 +373,12 @@ class HotRunner extends ResidentRunner { ...@@ -374,17 +373,12 @@ class HotRunner extends ResidentRunner {
nativeAssetsYaml = globals.fs.path.toUri(_nativeAssetsYamlFile); nativeAssetsYaml = globals.fs.path.toUri(_nativeAssetsYamlFile);
} else { } else {
final Uri projectUri = Uri.directory(projectRootPath); final Uri projectUri = Uri.directory(projectRootPath);
_buildRunner ??= NativeAssetsBuildRunnerImpl( nativeAssetsYaml = await _nativeAssetsBuilder?.dryRun(
projectUri,
debuggingOptions.buildInfo.packageConfig,
fileSystem,
globals.logger,
);
nativeAssetsYaml = await dryRunNativeAssets(
projectUri: projectUri, projectUri: projectUri,
fileSystem: fileSystem, fileSystem: fileSystem,
buildRunner: _buildRunner!,
flutterDevices: flutterDevices, flutterDevices: flutterDevices,
logger: logger,
packageConfig: debuggingOptions.buildInfo.packageConfig,
); );
} }
...@@ -1727,3 +1721,15 @@ class ReasonForCancelling { ...@@ -1727,3 +1721,15 @@ class ReasonForCancelling {
return '$message.\nTry performing a hot restart instead.'; return '$message.\nTry performing a hot restart instead.';
} }
} }
/// An interface to enable overriding native assets build logic in other
/// build systems.
abstract class HotRunnerNativeAssetsBuilder {
Future<Uri?> dryRun({
required Uri projectUri,
required FileSystem fileSystem,
required List<FlutterDevice> flutterDevices,
required PackageConfig packageConfig,
required Logger logger,
});
}
...@@ -21,6 +21,7 @@ import '../convert.dart'; ...@@ -21,6 +21,7 @@ import '../convert.dart';
import '../dart/language_version.dart'; import '../dart/language_version.dart';
import '../device.dart'; import '../device.dart';
import '../globals.dart' as globals; import '../globals.dart' as globals;
import '../native_assets.dart';
import '../project.dart'; import '../project.dart';
import '../test/test_wrapper.dart'; import '../test/test_wrapper.dart';
...@@ -69,6 +70,7 @@ FlutterPlatform installHook({ ...@@ -69,6 +70,7 @@ FlutterPlatform installHook({
String? integrationTestUserIdentifier, String? integrationTestUserIdentifier,
TestTimeRecorder? testTimeRecorder, TestTimeRecorder? testTimeRecorder,
UriConverter? uriConverter, UriConverter? uriConverter,
TestCompilerNativeAssetsBuilder? nativeAssetsBuilder,
}) { }) {
assert(enableVmService || enableObservatory || (!debuggingOptions.startPaused && debuggingOptions.hostVmServicePort == null)); assert(enableVmService || enableObservatory || (!debuggingOptions.startPaused && debuggingOptions.hostVmServicePort == null));
...@@ -99,6 +101,7 @@ FlutterPlatform installHook({ ...@@ -99,6 +101,7 @@ FlutterPlatform installHook({
integrationTestUserIdentifier: integrationTestUserIdentifier, integrationTestUserIdentifier: integrationTestUserIdentifier,
testTimeRecorder: testTimeRecorder, testTimeRecorder: testTimeRecorder,
uriConverter: uriConverter, uriConverter: uriConverter,
nativeAssetsBuilder: nativeAssetsBuilder,
); );
platformPluginRegistration(platform); platformPluginRegistration(platform);
return platform; return platform;
...@@ -291,6 +294,7 @@ class FlutterPlatform extends PlatformPlugin { ...@@ -291,6 +294,7 @@ class FlutterPlatform extends PlatformPlugin {
this.integrationTestUserIdentifier, this.integrationTestUserIdentifier,
this.testTimeRecorder, this.testTimeRecorder,
this.uriConverter, this.uriConverter,
this.nativeAssetsBuilder,
}); });
final String shellPath; final String shellPath;
...@@ -307,6 +311,7 @@ class FlutterPlatform extends PlatformPlugin { ...@@ -307,6 +311,7 @@ class FlutterPlatform extends PlatformPlugin {
final FlutterProject? flutterProject; final FlutterProject? flutterProject;
final String? icudtlPath; final String? icudtlPath;
final TestTimeRecorder? testTimeRecorder; final TestTimeRecorder? testTimeRecorder;
final TestCompilerNativeAssetsBuilder? nativeAssetsBuilder;
// This can be used by internal projects that require custom logic for converting package: URIs to local paths. // This can be used by internal projects that require custom logic for converting package: URIs to local paths.
final UriConverter? uriConverter; final UriConverter? uriConverter;
...@@ -465,7 +470,13 @@ class FlutterPlatform extends PlatformPlugin { ...@@ -465,7 +470,13 @@ class FlutterPlatform extends PlatformPlugin {
// running this with a debugger attached. Initialize the resident // running this with a debugger attached. Initialize the resident
// compiler in this case. // compiler in this case.
if (debuggingOptions.startPaused) { if (debuggingOptions.startPaused) {
compiler ??= TestCompiler(debuggingOptions.buildInfo, flutterProject, precompiledDillPath: precompiledDillPath, testTimeRecorder: testTimeRecorder); compiler ??= TestCompiler(
debuggingOptions.buildInfo,
flutterProject,
precompiledDillPath: precompiledDillPath,
testTimeRecorder: testTimeRecorder,
nativeAssetsBuilder: nativeAssetsBuilder,
);
final Uri uri = globals.fs.file(path).uri; final Uri uri = globals.fs.file(path).uri;
// Trigger a compilation to initialize the resident compiler. // Trigger a compilation to initialize the resident compiler.
unawaited(compiler!.compile(uri)); unawaited(compiler!.compile(uri));
...@@ -487,7 +498,12 @@ class FlutterPlatform extends PlatformPlugin { ...@@ -487,7 +498,12 @@ class FlutterPlatform extends PlatformPlugin {
// Integration test device takes care of the compilation. // Integration test device takes care of the compilation.
if (integrationTestDevice == null) { if (integrationTestDevice == null) {
// Lazily instantiate compiler so it is built only if it is actually used. // Lazily instantiate compiler so it is built only if it is actually used.
compiler ??= TestCompiler(debuggingOptions.buildInfo, flutterProject, testTimeRecorder: testTimeRecorder); compiler ??= TestCompiler(
debuggingOptions.buildInfo,
flutterProject,
testTimeRecorder: testTimeRecorder,
nativeAssetsBuilder: nativeAssetsBuilder,
);
mainDart = await compiler!.compile(globals.fs.file(mainDart).uri); mainDart = await compiler!.compile(globals.fs.file(mainDart).uri);
if (mainDart == null) { if (mainDart == null) {
......
...@@ -7,6 +7,7 @@ import '../base/file_system.dart'; ...@@ -7,6 +7,7 @@ import '../base/file_system.dart';
import '../base/io.dart'; import '../base/io.dart';
import '../device.dart'; import '../device.dart';
import '../globals.dart' as globals; import '../globals.dart' as globals;
import '../native_assets.dart';
import '../project.dart'; import '../project.dart';
import '../web/chrome.dart'; import '../web/chrome.dart';
import '../web/memory_fs.dart'; import '../web/memory_fs.dart';
...@@ -53,6 +54,7 @@ abstract class FlutterTestRunner { ...@@ -53,6 +54,7 @@ abstract class FlutterTestRunner {
Device? integrationTestDevice, Device? integrationTestDevice,
String? integrationTestUserIdentifier, String? integrationTestUserIdentifier,
TestTimeRecorder? testTimeRecorder, TestTimeRecorder? testTimeRecorder,
TestCompilerNativeAssetsBuilder? nativeAssetsBuilder,
}); });
} }
...@@ -91,6 +93,7 @@ class _FlutterTestRunnerImpl implements FlutterTestRunner { ...@@ -91,6 +93,7 @@ class _FlutterTestRunnerImpl implements FlutterTestRunner {
Device? integrationTestDevice, Device? integrationTestDevice,
String? integrationTestUserIdentifier, String? integrationTestUserIdentifier,
TestTimeRecorder? testTimeRecorder, TestTimeRecorder? testTimeRecorder,
TestCompilerNativeAssetsBuilder? nativeAssetsBuilder,
}) async { }) async {
// Configure package:test to use the Flutter engine for child processes. // Configure package:test to use the Flutter engine for child processes.
final String shellPath = globals.artifacts!.getArtifactPath(Artifact.flutterTester); final String shellPath = globals.artifacts!.getArtifactPath(Artifact.flutterTester);
...@@ -210,6 +213,7 @@ class _FlutterTestRunnerImpl implements FlutterTestRunner { ...@@ -210,6 +213,7 @@ class _FlutterTestRunnerImpl implements FlutterTestRunner {
integrationTestDevice: integrationTestDevice, integrationTestDevice: integrationTestDevice,
integrationTestUserIdentifier: integrationTestUserIdentifier, integrationTestUserIdentifier: integrationTestUserIdentifier,
testTimeRecorder: testTimeRecorder, testTimeRecorder: testTimeRecorder,
nativeAssetsBuilder: nativeAssetsBuilder,
); );
try { try {
......
...@@ -10,18 +10,13 @@ import 'package:meta/meta.dart'; ...@@ -10,18 +10,13 @@ import 'package:meta/meta.dart';
import '../artifacts.dart'; import '../artifacts.dart';
import '../base/file_system.dart'; import '../base/file_system.dart';
import '../base/os.dart' show HostPlatform;
import '../base/platform.dart';
import '../build_info.dart'; import '../build_info.dart';
import '../bundle.dart'; import '../bundle.dart';
import '../compile.dart'; import '../compile.dart';
import '../flutter_plugins.dart'; import '../flutter_plugins.dart';
import '../globals.dart' as globals; import '../globals.dart' as globals;
import '../linux/native_assets.dart';
import '../macos/native_assets.dart';
import '../native_assets.dart'; import '../native_assets.dart';
import '../project.dart'; import '../project.dart';
import '../windows/native_assets.dart';
import 'test_time_recorder.dart'; import 'test_time_recorder.dart';
/// A request to the [TestCompiler] for recompilation. /// A request to the [TestCompiler] for recompilation.
...@@ -51,9 +46,11 @@ class TestCompiler { ...@@ -51,9 +46,11 @@ class TestCompiler {
/// If [testTimeRecorder] is passed, times will be recorded in it. /// If [testTimeRecorder] is passed, times will be recorded in it.
TestCompiler( TestCompiler(
this.buildInfo, this.buildInfo,
this.flutterProject, this.flutterProject, {
{ String? precompiledDillPath, this.testTimeRecorder } String? precompiledDillPath,
) : testFilePath = precompiledDillPath ?? globals.fs.path.join( this.testTimeRecorder,
TestCompilerNativeAssetsBuilder? nativeAssetsBuilder,
}) : testFilePath = precompiledDillPath ?? globals.fs.path.join(
flutterProject!.directory.path, flutterProject!.directory.path,
getBuildDirectory(), getBuildDirectory(),
'test_cache', 'test_cache',
...@@ -62,7 +59,8 @@ class TestCompiler { ...@@ -62,7 +59,8 @@ class TestCompiler {
dartDefines: buildInfo.dartDefines, dartDefines: buildInfo.dartDefines,
extraFrontEndOptions: buildInfo.extraFrontEndOptions, extraFrontEndOptions: buildInfo.extraFrontEndOptions,
)), )),
shouldCopyDillFile = precompiledDillPath == null { shouldCopyDillFile = precompiledDillPath == null,
_nativeAssetsBuilder = nativeAssetsBuilder {
// Compiler maintains and updates single incremental dill file. // Compiler maintains and updates single incremental dill file.
// Incremental compilation requests done for each test copy that file away // Incremental compilation requests done for each test copy that file away
// for independent execution. // for independent execution.
...@@ -83,6 +81,7 @@ class TestCompiler { ...@@ -83,6 +81,7 @@ class TestCompiler {
final String testFilePath; final String testFilePath;
final bool shouldCopyDillFile; final bool shouldCopyDillFile;
final TestTimeRecorder? testTimeRecorder; final TestTimeRecorder? testTimeRecorder;
final TestCompilerNativeAssetsBuilder? _nativeAssetsBuilder;
ResidentCompiler? compiler; ResidentCompiler? compiler;
...@@ -170,57 +169,7 @@ class TestCompiler { ...@@ -170,57 +169,7 @@ class TestCompiler {
invalidatedRegistrantFiles.add(flutterProject!.dartPluginRegistrant.absolute.uri); invalidatedRegistrantFiles.add(flutterProject!.dartPluginRegistrant.absolute.uri);
} }
Uri? nativeAssetsYaml; final Uri? nativeAssetsYaml = await _nativeAssetsBuilder?.build(buildInfo);
if (!buildInfo.buildNativeAssets) {
nativeAssetsYaml = null;
} else {
final Uri projectUri = FlutterProject.current().directory.uri;
final NativeAssetsBuildRunner buildRunner = NativeAssetsBuildRunnerImpl(
projectUri,
buildInfo.packageConfig,
globals.fs,
globals.logger,
);
if (globals.platform.isMacOS) {
(nativeAssetsYaml, _) = await buildNativeAssetsMacOS(
buildMode: buildInfo.mode,
projectUri: projectUri,
flutterTester: true,
fileSystem: globals.fs,
buildRunner: buildRunner,
);
} else if (globals.platform.isLinux) {
(nativeAssetsYaml, _) = await buildNativeAssetsLinux(
buildMode: buildInfo.mode,
projectUri: projectUri,
flutterTester: true,
fileSystem: globals.fs,
buildRunner: buildRunner,
);
} else if (globals.platform.isWindows) {
final TargetPlatform targetPlatform;
if (globals.os.hostPlatform == HostPlatform.windows_x64) {
targetPlatform = TargetPlatform.windows_x64;
} else {
targetPlatform = TargetPlatform.windows_arm64;
}
(nativeAssetsYaml, _) = await buildNativeAssetsWindows(
buildMode: buildInfo.mode,
targetPlatform: targetPlatform,
projectUri: projectUri,
flutterTester: true,
fileSystem: globals.fs,
buildRunner: buildRunner,
);
} else {
await ensureNoNativeAssetsOrOsIsSupported(
projectUri,
const LocalPlatform().operatingSystem,
globals.fs,
buildRunner,
);
}
}
final CompilerOutput? compilerOutput = await compiler!.recompile( final CompilerOutput? compilerOutput = await compiler!.recompile(
request.mainUri, request.mainUri,
......
...@@ -1203,6 +1203,7 @@ class FakeHotRunnerFactory extends Fake implements HotRunnerFactory { ...@@ -1203,6 +1203,7 @@ class FakeHotRunnerFactory extends Fake implements HotRunnerFactory {
FlutterProject? flutterProject, FlutterProject? flutterProject,
Analytics? analytics, Analytics? analytics,
String? nativeAssetsYamlFile, String? nativeAssetsYamlFile,
HotRunnerNativeAssetsBuilder? nativeAssetsBuilder,
}) { }) {
if (_artifactTester != null) { if (_artifactTester != null) {
for (final FlutterDevice device in devices) { for (final FlutterDevice device in devices) {
......
...@@ -29,6 +29,7 @@ import 'package:flutter_tools/src/macos/macos_ipad_device.dart'; ...@@ -29,6 +29,7 @@ import 'package:flutter_tools/src/macos/macos_ipad_device.dart';
import 'package:flutter_tools/src/project.dart'; import 'package:flutter_tools/src/project.dart';
import 'package:flutter_tools/src/reporting/reporting.dart'; import 'package:flutter_tools/src/reporting/reporting.dart';
import 'package:flutter_tools/src/resident_runner.dart'; import 'package:flutter_tools/src/resident_runner.dart';
import 'package:flutter_tools/src/run_hot.dart';
import 'package:flutter_tools/src/runner/flutter_command.dart'; import 'package:flutter_tools/src/runner/flutter_command.dart';
import 'package:flutter_tools/src/vmservice.dart'; import 'package:flutter_tools/src/vmservice.dart';
import 'package:test/fake.dart'; import 'package:test/fake.dart';
...@@ -1578,6 +1579,7 @@ class CapturingAppDomain extends AppDomain { ...@@ -1578,6 +1579,7 @@ class CapturingAppDomain extends AppDomain {
String? userIdentifier, String? userIdentifier,
bool enableDevTools = true, bool enableDevTools = true,
String? flavor, String? flavor,
HotRunnerNativeAssetsBuilder? nativeAssetsBuilder,
}) async { }) async {
this.userIdentifier = userIdentifier; this.userIdentifier = userIdentifier;
this.enableDevTools = enableDevTools; this.enableDevTools = enableDevTools;
......
...@@ -14,6 +14,7 @@ import 'package:flutter_tools/src/cache.dart'; ...@@ -14,6 +14,7 @@ import 'package:flutter_tools/src/cache.dart';
import 'package:flutter_tools/src/commands/test.dart'; import 'package:flutter_tools/src/commands/test.dart';
import 'package:flutter_tools/src/device.dart'; import 'package:flutter_tools/src/device.dart';
import 'package:flutter_tools/src/globals.dart' as globals; import 'package:flutter_tools/src/globals.dart' as globals;
import 'package:flutter_tools/src/native_assets.dart';
import 'package:flutter_tools/src/project.dart'; import 'package:flutter_tools/src/project.dart';
import 'package:flutter_tools/src/runner/flutter_command.dart'; import 'package:flutter_tools/src/runner/flutter_command.dart';
import 'package:flutter_tools/src/test/coverage_collector.dart'; import 'package:flutter_tools/src/test/coverage_collector.dart';
...@@ -1124,6 +1125,7 @@ class FakeFlutterTestRunner implements FlutterTestRunner { ...@@ -1124,6 +1125,7 @@ class FakeFlutterTestRunner implements FlutterTestRunner {
Device? integrationTestDevice, Device? integrationTestDevice,
String? integrationTestUserIdentifier, String? integrationTestUserIdentifier,
TestTimeRecorder? testTimeRecorder, TestTimeRecorder? testTimeRecorder,
TestCompilerNativeAssetsBuilder? nativeAssetsBuilder,
}) async { }) async {
lastEnableVmServiceValue = enableVmService; lastEnableVmServiceValue = enableVmService;
lastDebuggingOptionsValue = debuggingOptions; lastDebuggingOptionsValue = debuggingOptions;
......
// Copyright 2014 The Flutter Authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
import 'package:flutter_tools/src/application_package.dart';
import 'package:flutter_tools/src/asset.dart';
import 'package:flutter_tools/src/build_info.dart';
import 'package:flutter_tools/src/build_system/tools/shader_compiler.dart';
import 'package:flutter_tools/src/compile.dart';
import 'package:flutter_tools/src/devfs.dart';
import 'package:flutter_tools/src/device.dart';
import 'package:flutter_tools/src/project.dart';
import 'package:flutter_tools/src/resident_runner.dart';
import 'package:flutter_tools/src/run_hot.dart';
import 'package:flutter_tools/src/vmservice.dart';
import 'package:package_config/package_config.dart';
import 'package:test/fake.dart';
import 'package:vm_service/vm_service.dart' as vm_service;
class FakeDevFs extends Fake implements DevFS {
@override
Future<void> destroy() async { }
@override
List<Uri> sources = <Uri>[];
@override
DateTime? lastCompiled;
@override
PackageConfig? lastPackageConfig;
@override
Set<String> assetPathsToEvict = <String>{};
@override
Set<String> shaderPathsToEvict= <String>{};
@override
Set<String> scenePathsToEvict= <String>{};
@override
Uri? baseUri;
}
class FakeDevice extends Fake implements Device {
FakeDevice({
TargetPlatform targetPlatform = TargetPlatform.tester,
}) : _targetPlatform = targetPlatform;
final TargetPlatform _targetPlatform;
bool disposed = false;
@override
bool isSupported() => true;
@override
bool supportsHotReload = true;
@override
bool supportsHotRestart = true;
@override
bool supportsFlutterExit = true;
@override
Future<TargetPlatform> get targetPlatform async => _targetPlatform;
@override
Future<String> get sdkNameAndVersion async => 'Tester';
@override
Future<bool> get isLocalEmulator async => false;
@override
String get name => 'Fake Device';
@override
Future<bool> stopApp(
ApplicationPackage? app, {
String? userIdentifier,
}) async {
return true;
}
@override
Future<void> dispose() async {
disposed = true;
}
}
class FakeFlutterDevice extends Fake implements FlutterDevice {
FakeFlutterDevice(this.device);
bool stoppedEchoingDeviceLog = false;
late Future<UpdateFSReport> Function() updateDevFSReportCallback;
@override
final FakeDevice device;
@override
Future<void> stopEchoingDeviceLog() async {
stoppedEchoingDeviceLog = true;
}
@override
DevFS? devFS = FakeDevFs();
@override
FlutterVmService get vmService => FakeFlutterVmService();
@override
ResidentCompiler? generator;
@override
Future<UpdateFSReport> updateDevFS({
Uri? mainUri,
String? target,
AssetBundle? bundle,
DateTime? firstBuildTime,
bool bundleFirstUpload = false,
bool bundleDirty = false,
bool fullRestart = false,
String? projectRootPath,
String? pathToReload,
required String dillOutputPath,
required List<Uri> invalidatedFiles,
required PackageConfig packageConfig,
}) => updateDevFSReportCallback();
@override
TargetPlatform? get targetPlatform => device._targetPlatform;
}
class TestFlutterDevice extends FlutterDevice {
TestFlutterDevice({
required Device device,
required this.exception,
required ResidentCompiler generator,
}) : super(device, buildInfo: BuildInfo.debug, generator: generator, developmentShaderCompiler: const FakeShaderCompiler());
/// The exception to throw when the connect method is called.
final Exception exception;
@override
Future<void> connect({
ReloadSources? reloadSources,
Restart? restart,
CompileExpression? compileExpression,
GetSkSLMethod? getSkSLMethod,
FlutterProject? flutterProject,
PrintStructuredErrorLogMethod? printStructuredErrorLogMethod,
bool disableServiceAuthCodes = false,
bool enableDds = true,
bool cacheStartupProfile = false,
bool? ipv6 = false,
int? hostVmServicePort,
int? ddsPort,
bool allowExistingDdsInstance = false,
}) async {
throw exception;
}
}
class TestHotRunnerConfig extends HotRunnerConfig {
TestHotRunnerConfig({this.successfulHotRestartSetup, this.successfulHotReloadSetup});
bool? successfulHotRestartSetup;
bool? successfulHotReloadSetup;
bool shutdownHookCalled = false;
bool updateDevFSCompleteCalled = false;
@override
Future<bool?> setupHotRestart() async {
assert(successfulHotRestartSetup != null, 'setupHotRestart is not expected to be called in this test.');
return successfulHotRestartSetup;
}
@override
Future<bool?> setupHotReload() async {
assert(successfulHotReloadSetup != null, 'setupHotReload is not expected to be called in this test.');
return successfulHotReloadSetup;
}
@override
void updateDevFSComplete() {
updateDevFSCompleteCalled = true;
}
@override
Future<void> runPreShutdownOperations() async {
shutdownHookCalled = true;
}
}
class FakeResidentCompiler extends Fake implements ResidentCompiler {
@override
void accept() {}
}
class FakeFlutterVmService extends Fake implements FlutterVmService {
@override
vm_service.VmService get service => FakeVmService();
@override
Future<List<FlutterView>> getFlutterViews({bool returnEarly = false, Duration delay = const Duration(milliseconds: 50)}) async {
return <FlutterView>[];
}
}
class FakeVmService extends Fake implements vm_service.VmService {
@override
Future<vm_service.VM> getVM() async => FakeVm();
}
class FakeVm extends Fake implements vm_service.VM {
@override
List<vm_service.IsolateRef> get isolates => <vm_service.IsolateRef>[];
}
class FakeShaderCompiler implements DevelopmentShaderCompiler {
const FakeShaderCompiler();
@override
void configureCompiler(TargetPlatform? platform) { }
@override
Future<DevFSContent> recompileShader(DevFSContent inputShader) {
throw UnimplementedError();
}
}
...@@ -5,7 +5,6 @@ ...@@ -5,7 +5,6 @@
import 'package:file/file.dart'; import 'package:file/file.dart';
import 'package:file/memory.dart'; import 'package:file/memory.dart';
import 'package:file_testing/file_testing.dart'; import 'package:file_testing/file_testing.dart';
import 'package:flutter_tools/src/android/native_assets.dart';
import 'package:flutter_tools/src/artifacts.dart'; import 'package:flutter_tools/src/artifacts.dart';
import 'package:flutter_tools/src/base/common.dart'; import 'package:flutter_tools/src/base/common.dart';
import 'package:flutter_tools/src/base/file_system.dart'; import 'package:flutter_tools/src/base/file_system.dart';
...@@ -15,15 +14,16 @@ import 'package:flutter_tools/src/build_info.dart'; ...@@ -15,15 +14,16 @@ import 'package:flutter_tools/src/build_info.dart';
import 'package:flutter_tools/src/build_system/build_system.dart'; import 'package:flutter_tools/src/build_system/build_system.dart';
import 'package:flutter_tools/src/features.dart'; import 'package:flutter_tools/src/features.dart';
import 'package:flutter_tools/src/globals.dart' as globals; import 'package:flutter_tools/src/globals.dart' as globals;
import 'package:flutter_tools/src/isolated/native_assets/android/native_assets.dart';
import 'package:native_assets_cli/native_assets_cli_internal.dart' import 'package:native_assets_cli/native_assets_cli_internal.dart'
as native_assets_cli; as native_assets_cli;
import 'package:native_assets_cli/native_assets_cli_internal.dart' import 'package:native_assets_cli/native_assets_cli_internal.dart'
hide BuildMode, Target; hide BuildMode, Target;
import 'package:package_config/package_config_types.dart'; import 'package:package_config/package_config_types.dart';
import '../../src/common.dart'; import '../../../src/common.dart';
import '../../src/context.dart'; import '../../../src/context.dart';
import '../../src/fakes.dart'; import '../../../src/fakes.dart';
import '../fake_native_assets_build_runner.dart'; import '../fake_native_assets_build_runner.dart';
void main() { void main() {
......
...@@ -12,14 +12,14 @@ import 'package:flutter_tools/src/build_system/build_system.dart'; ...@@ -12,14 +12,14 @@ import 'package:flutter_tools/src/build_system/build_system.dart';
import 'package:flutter_tools/src/build_system/exceptions.dart'; import 'package:flutter_tools/src/build_system/exceptions.dart';
import 'package:flutter_tools/src/build_system/targets/native_assets.dart'; import 'package:flutter_tools/src/build_system/targets/native_assets.dart';
import 'package:flutter_tools/src/features.dart'; import 'package:flutter_tools/src/features.dart';
import 'package:flutter_tools/src/native_assets.dart'; import 'package:flutter_tools/src/isolated/native_assets/native_assets.dart';
import 'package:native_assets_cli/native_assets_cli_internal.dart' import 'package:native_assets_cli/native_assets_cli_internal.dart'
as native_assets_cli; as native_assets_cli;
import 'package:package_config/package_config.dart' show Package; import 'package:package_config/package_config.dart' show Package;
import '../../../src/common.dart'; import '../../../../src/common.dart';
import '../../../src/context.dart'; import '../../../../src/context.dart';
import '../../../src/fakes.dart'; import '../../../../src/fakes.dart';
import '../../fake_native_assets_build_runner.dart'; import '../../fake_native_assets_build_runner.dart';
void main() { void main() {
......
...@@ -2,7 +2,11 @@ ...@@ -2,7 +2,11 @@
// Use of this source code is governed by a BSD-style license that can be // Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file. // found in the LICENSE file.
import 'package:flutter_tools/src/native_assets.dart'; import 'package:file/src/interface/file_system.dart';
import 'package:flutter_tools/src/base/logger.dart';
import 'package:flutter_tools/src/isolated/native_assets/native_assets.dart';
import 'package:flutter_tools/src/resident_runner.dart';
import 'package:flutter_tools/src/run_hot.dart';
import 'package:native_assets_builder/native_assets_builder.dart' import 'package:native_assets_builder/native_assets_builder.dart'
as native_assets_builder; as native_assets_builder;
import 'package:native_assets_cli/native_assets_cli_internal.dart'; import 'package:native_assets_cli/native_assets_cli_internal.dart';
...@@ -95,3 +99,25 @@ final class FakeNativeAssetsBuilderResult ...@@ -95,3 +99,25 @@ final class FakeNativeAssetsBuilderResult
@override @override
final bool success; final bool success;
} }
class FakeHotRunnerNativeAssetsBuilder implements HotRunnerNativeAssetsBuilder {
FakeHotRunnerNativeAssetsBuilder(this.buildRunner);
final NativeAssetsBuildRunner buildRunner;
@override
Future<Uri?> dryRun({
required Uri projectUri,
required FileSystem fileSystem,
required List<FlutterDevice> flutterDevices,
required PackageConfig packageConfig,
required Logger logger,
}) {
return dryRunNativeAssets(
projectUri: projectUri,
fileSystem: fileSystem,
buildRunner: buildRunner,
flutterDevices: flutterDevices,
);
}
}
// Copyright 2014 The Flutter Authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
import 'package:file/memory.dart';
import 'package:flutter_tools/src/artifacts.dart';
import 'package:flutter_tools/src/base/file_system.dart';
import 'package:flutter_tools/src/base/platform.dart';
import 'package:flutter_tools/src/build_info.dart';
import 'package:flutter_tools/src/devfs.dart';
import 'package:flutter_tools/src/device.dart';
import 'package:flutter_tools/src/features.dart';
import 'package:flutter_tools/src/resident_devtools_handler.dart';
import 'package:flutter_tools/src/resident_runner.dart';
import 'package:flutter_tools/src/run_hot.dart';
import 'package:native_assets_cli/native_assets_cli_internal.dart'
hide BuildMode, Target;
import 'package:native_assets_cli/native_assets_cli_internal.dart'
as native_assets_cli;
import 'package:package_config/package_config.dart';
import 'package:unified_analytics/unified_analytics.dart';
import '../../src/common.dart';
import '../../src/context.dart';
import '../../src/fakes.dart';
import '../hot_shared.dart';
import 'fake_native_assets_build_runner.dart';
void main() {
group('native assets', () {
late TestHotRunnerConfig testingConfig;
late FileSystem fileSystem;
late FakeAnalytics fakeAnalytics;
setUp(() {
fileSystem = MemoryFileSystem.test();
testingConfig = TestHotRunnerConfig(
successfulHotRestartSetup: true,
);
fakeAnalytics = getInitializedFakeAnalyticsInstance(
fs: fileSystem,
fakeFlutterVersion: FakeFlutterVersion(),
);
});
testUsingContext('native assets restart', () async {
final FakeDevice device = FakeDevice();
final FakeFlutterDevice fakeFlutterDevice = FakeFlutterDevice(device);
final List<FlutterDevice> devices = <FlutterDevice>[
fakeFlutterDevice,
];
fakeFlutterDevice.updateDevFSReportCallback = () async => UpdateFSReport(
success: true,
invalidatedSourcesCount: 6,
syncedBytes: 8,
scannedSourcesCount: 16,
compileDuration: const Duration(seconds: 16),
transferDuration: const Duration(seconds: 32),
);
(fakeFlutterDevice.devFS! as FakeDevFs).baseUri = Uri.parse('file:///base_uri');
final FakeNativeAssetsBuildRunner buildRunner = FakeNativeAssetsBuildRunner(
packagesWithNativeAssetsResult: <Package>[
Package('bar', fileSystem.currentDirectory.uri),
],
dryRunResult: FakeNativeAssetsBuilderResult(
assets: <Asset>[
Asset(
id: 'package:bar/bar.dart',
linkMode: LinkMode.dynamic,
target: native_assets_cli.Target.macOSArm64,
path: AssetAbsolutePath(Uri.file('bar.dylib')),
),
],
),
);
final HotRunner hotRunner = HotRunner(
devices,
debuggingOptions: DebuggingOptions.disabled(BuildInfo.debug),
target: 'main.dart',
devtoolsHandler: createNoOpHandler,
nativeAssetsBuilder: FakeHotRunnerNativeAssetsBuilder(buildRunner),
analytics: fakeAnalytics,
);
final OperationResult result = await hotRunner.restart(fullRestart: true);
expect(result.isOk, true);
// Hot restart does not require rerunning anything for native assets.
// The previous native assets mapping should be used.
expect(buildRunner.buildInvocations, 0);
expect(buildRunner.dryRunInvocations, 0);
expect(buildRunner.hasPackageConfigInvocations, 0);
expect(buildRunner.packagesWithNativeAssetsInvocations, 0);
}, overrides: <Type, Generator>{
HotRunnerConfig: () => testingConfig,
Artifacts: () => Artifacts.test(),
FileSystem: () => fileSystem,
Platform: () => FakePlatform(),
ProcessManager: () => FakeProcessManager.empty(),
FeatureFlags: () => TestFeatureFlags(isNativeAssetsEnabled: true, isMacOSEnabled: true),
});
testUsingContext('native assets run unsupported', () async {
final FakeDevice device = FakeDevice(targetPlatform: TargetPlatform.fuchsia_arm64);
final FakeFlutterDevice fakeFlutterDevice = FakeFlutterDevice(device);
final List<FlutterDevice> devices = <FlutterDevice>[
fakeFlutterDevice,
];
fakeFlutterDevice.updateDevFSReportCallback = () async => UpdateFSReport(
success: true,
invalidatedSourcesCount: 6,
syncedBytes: 8,
scannedSourcesCount: 16,
compileDuration: const Duration(seconds: 16),
transferDuration: const Duration(seconds: 32),
);
(fakeFlutterDevice.devFS! as FakeDevFs).baseUri = Uri.parse('file:///base_uri');
final FakeNativeAssetsBuildRunner buildRunner = FakeNativeAssetsBuildRunner(
packagesWithNativeAssetsResult: <Package>[
Package('bar', fileSystem.currentDirectory.uri),
],
dryRunResult: FakeNativeAssetsBuilderResult(
assets: <Asset>[
Asset(
id: 'package:bar/bar.dart',
linkMode: LinkMode.dynamic,
target: native_assets_cli.Target.macOSArm64,
path: AssetAbsolutePath(Uri.file('bar.dylib')),
),
],
),
);
final HotRunner hotRunner = HotRunner(
devices,
debuggingOptions: DebuggingOptions.disabled(BuildInfo.debug),
target: 'main.dart',
devtoolsHandler: createNoOpHandler,
nativeAssetsBuilder: FakeHotRunnerNativeAssetsBuilder(buildRunner),
analytics: fakeAnalytics,
);
expect(
() => hotRunner.run(),
throwsToolExit( message:
'Package(s) bar require the native assets feature. '
'This feature has not yet been implemented for `TargetPlatform.fuchsia_arm64`. '
'For more info see https://github.com/flutter/flutter/issues/129757.',
)
);
}, overrides: <Type, Generator>{
HotRunnerConfig: () => testingConfig,
Artifacts: () => Artifacts.test(),
FileSystem: () => fileSystem,
Platform: () => FakePlatform(),
ProcessManager: () => FakeProcessManager.empty(),
FeatureFlags: () => TestFeatureFlags(isNativeAssetsEnabled: true, isMacOSEnabled: true),
});
});
}
...@@ -13,16 +13,16 @@ import 'package:flutter_tools/src/build_info.dart'; ...@@ -13,16 +13,16 @@ import 'package:flutter_tools/src/build_info.dart';
import 'package:flutter_tools/src/build_system/build_system.dart'; import 'package:flutter_tools/src/build_system/build_system.dart';
import 'package:flutter_tools/src/features.dart'; import 'package:flutter_tools/src/features.dart';
import 'package:flutter_tools/src/globals.dart' as globals; import 'package:flutter_tools/src/globals.dart' as globals;
import 'package:flutter_tools/src/ios/native_assets.dart'; import 'package:flutter_tools/src/isolated/native_assets/ios/native_assets.dart';
import 'package:native_assets_cli/native_assets_cli_internal.dart' import 'package:native_assets_cli/native_assets_cli_internal.dart'
hide BuildMode, Target; hide BuildMode, Target;
import 'package:native_assets_cli/native_assets_cli_internal.dart' import 'package:native_assets_cli/native_assets_cli_internal.dart'
as native_assets_cli; as native_assets_cli;
import 'package:package_config/package_config_types.dart'; import 'package:package_config/package_config_types.dart';
import '../../src/common.dart'; import '../../../src/common.dart';
import '../../src/context.dart'; import '../../../src/context.dart';
import '../../src/fakes.dart'; import '../../../src/fakes.dart';
import '../fake_native_assets_build_runner.dart'; import '../fake_native_assets_build_runner.dart';
void main() { void main() {
......
...@@ -15,17 +15,17 @@ import 'package:flutter_tools/src/build_system/build_system.dart'; ...@@ -15,17 +15,17 @@ import 'package:flutter_tools/src/build_system/build_system.dart';
import 'package:flutter_tools/src/dart/package_map.dart'; import 'package:flutter_tools/src/dart/package_map.dart';
import 'package:flutter_tools/src/features.dart'; import 'package:flutter_tools/src/features.dart';
import 'package:flutter_tools/src/globals.dart' as globals; import 'package:flutter_tools/src/globals.dart' as globals;
import 'package:flutter_tools/src/linux/native_assets.dart'; import 'package:flutter_tools/src/isolated/native_assets/linux/native_assets.dart';
import 'package:flutter_tools/src/native_assets.dart'; import 'package:flutter_tools/src/isolated/native_assets/native_assets.dart';
import 'package:native_assets_cli/native_assets_cli_internal.dart' import 'package:native_assets_cli/native_assets_cli_internal.dart'
hide BuildMode, Target; hide BuildMode, Target;
import 'package:native_assets_cli/native_assets_cli_internal.dart' import 'package:native_assets_cli/native_assets_cli_internal.dart'
as native_assets_cli; as native_assets_cli;
import 'package:package_config/package_config_types.dart'; import 'package:package_config/package_config_types.dart';
import '../../src/common.dart'; import '../../../src/common.dart';
import '../../src/context.dart'; import '../../../src/context.dart';
import '../../src/fakes.dart'; import '../../../src/fakes.dart';
import '../fake_native_assets_build_runner.dart'; import '../fake_native_assets_build_runner.dart';
void main() { void main() {
......
...@@ -2,9 +2,9 @@ ...@@ -2,9 +2,9 @@
// Use of this source code is governed by a BSD-style license that can be // Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file. // found in the LICENSE file.
import 'package:flutter_tools/src/macos/native_assets_host.dart'; import 'package:flutter_tools/src/isolated/native_assets/macos/native_assets_host.dart';
import '../../src/common.dart'; import '../../../src/common.dart';
void main() { void main() {
test('framework name', () { test('framework name', () {
......
...@@ -13,17 +13,17 @@ import 'package:flutter_tools/src/build_system/build_system.dart'; ...@@ -13,17 +13,17 @@ import 'package:flutter_tools/src/build_system/build_system.dart';
import 'package:flutter_tools/src/dart/package_map.dart'; import 'package:flutter_tools/src/dart/package_map.dart';
import 'package:flutter_tools/src/features.dart'; import 'package:flutter_tools/src/features.dart';
import 'package:flutter_tools/src/globals.dart' as globals; import 'package:flutter_tools/src/globals.dart' as globals;
import 'package:flutter_tools/src/macos/native_assets.dart'; import 'package:flutter_tools/src/isolated/native_assets/macos/native_assets.dart';
import 'package:flutter_tools/src/native_assets.dart'; import 'package:flutter_tools/src/isolated/native_assets/native_assets.dart';
import 'package:native_assets_cli/native_assets_cli_internal.dart' import 'package:native_assets_cli/native_assets_cli_internal.dart'
hide BuildMode, Target; hide BuildMode, Target;
import 'package:native_assets_cli/native_assets_cli_internal.dart' import 'package:native_assets_cli/native_assets_cli_internal.dart'
as native_assets_cli; as native_assets_cli;
import 'package:package_config/package_config_types.dart'; import 'package:package_config/package_config_types.dart';
import '../../src/common.dart'; import '../../../src/common.dart';
import '../../src/context.dart'; import '../../../src/context.dart';
import '../../src/fakes.dart'; import '../../../src/fakes.dart';
import '../fake_native_assets_build_runner.dart'; import '../fake_native_assets_build_runner.dart';
void main() { void main() {
......
// Copyright 2014 The Flutter Authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
import 'package:flutter_tools/src/build_info.dart';
import 'package:flutter_tools/src/device.dart';
import 'package:flutter_tools/src/features.dart';
import 'package:flutter_tools/src/globals.dart' as globals;
import 'package:flutter_tools/src/resident_devtools_handler.dart';
import 'package:flutter_tools/src/resident_runner.dart';
import 'package:flutter_tools/src/run_hot.dart';
import 'package:unified_analytics/unified_analytics.dart';
import '../../src/common.dart';
import '../../src/context.dart';
import '../../src/fake_vm_services.dart';
import '../../src/fakes.dart';
import '../../src/testbed.dart';
import '../resident_runner_helpers.dart';
import 'fake_native_assets_build_runner.dart';
void main() {
late Testbed testbed;
late FakeFlutterDevice flutterDevice;
late FakeDevFS devFS;
late ResidentRunner residentRunner;
late FakeDevice device;
late FakeAnalytics fakeAnalytics;
FakeVmServiceHost? fakeVmServiceHost;
setUp(() {
testbed = Testbed(setup: () {
fakeAnalytics = getInitializedFakeAnalyticsInstance(
fs: globals.fs,
fakeFlutterVersion: FakeFlutterVersion(),
);
globals.fs.file('.packages')
.writeAsStringSync('\n');
globals.fs.file(globals.fs.path.join('build', 'app.dill'))
..createSync(recursive: true)
..writeAsStringSync('ABC');
residentRunner = HotRunner(
<FlutterDevice>[
flutterDevice,
],
stayResident: false,
debuggingOptions: DebuggingOptions.enabled(BuildInfo.debug),
target: 'main.dart',
devtoolsHandler: createNoOpHandler,
analytics: fakeAnalytics,
);
});
device = FakeDevice();
devFS = FakeDevFS();
flutterDevice = FakeFlutterDevice()
..testUri = testUri
..vmServiceHost = (() => fakeVmServiceHost)
..device = device
..fakeDevFS = devFS;
});
testUsingContext(
'use the nativeAssetsYamlFile when provided',
() => testbed.run(() async {
final FakeDevice device = FakeDevice(
targetPlatform: TargetPlatform.darwin,
sdkNameAndVersion: 'Macos',
);
final FakeResidentCompiler residentCompiler = FakeResidentCompiler();
final FakeFlutterDevice flutterDevice = FakeFlutterDevice()
..testUri = testUri
..vmServiceHost = (() => fakeVmServiceHost)
..device = device
..fakeDevFS = devFS
..targetPlatform = TargetPlatform.darwin
..generator = residentCompiler;
fakeVmServiceHost = FakeVmServiceHost(requests: <VmServiceExpectation>[
listViews,
listViews,
]);
globals.fs
.file(globals.fs.path.join('lib', 'main.dart'))
.createSync(recursive: true);
final FakeNativeAssetsBuildRunner buildRunner = FakeNativeAssetsBuildRunner();
residentRunner = HotRunner(
<FlutterDevice>[
flutterDevice,
],
stayResident: false,
debuggingOptions: DebuggingOptions.enabled(const BuildInfo(
BuildMode.debug,
'',
treeShakeIcons: false,
trackWidgetCreation: true,
)),
target: 'main.dart',
devtoolsHandler: createNoOpHandler,
nativeAssetsBuilder: FakeHotRunnerNativeAssetsBuilder(buildRunner),
analytics: fakeAnalytics,
nativeAssetsYamlFile: 'foo.yaml',
);
final int? result = await residentRunner.run();
expect(result, 0);
expect(buildRunner.buildInvocations, 0);
expect(buildRunner.dryRunInvocations, 0);
expect(buildRunner.hasPackageConfigInvocations, 0);
expect(buildRunner.packagesWithNativeAssetsInvocations, 0);
expect(residentCompiler.recompileCalled, true);
expect(residentCompiler.receivedNativeAssetsYaml, globals.fs.path.toUri('foo.yaml'));
}),
overrides: <Type, Generator>{
ProcessManager: () => FakeProcessManager.any(),
FeatureFlags: () => TestFeatureFlags(isNativeAssetsEnabled: true, isMacOSEnabled: true),
});
}
...@@ -14,17 +14,17 @@ import 'package:flutter_tools/src/build_system/build_system.dart'; ...@@ -14,17 +14,17 @@ import 'package:flutter_tools/src/build_system/build_system.dart';
import 'package:flutter_tools/src/dart/package_map.dart'; import 'package:flutter_tools/src/dart/package_map.dart';
import 'package:flutter_tools/src/features.dart'; import 'package:flutter_tools/src/features.dart';
import 'package:flutter_tools/src/globals.dart' as globals; import 'package:flutter_tools/src/globals.dart' as globals;
import 'package:flutter_tools/src/native_assets.dart'; import 'package:flutter_tools/src/isolated/native_assets/native_assets.dart';
import 'package:flutter_tools/src/windows/native_assets.dart'; import 'package:flutter_tools/src/isolated/native_assets/windows/native_assets.dart';
import 'package:native_assets_cli/native_assets_cli_internal.dart' import 'package:native_assets_cli/native_assets_cli_internal.dart'
hide BuildMode, Target; hide BuildMode, Target;
import 'package:native_assets_cli/native_assets_cli_internal.dart' import 'package:native_assets_cli/native_assets_cli_internal.dart'
as native_assets_cli; as native_assets_cli;
import 'package:package_config/package_config_types.dart'; import 'package:package_config/package_config_types.dart';
import '../../src/common.dart'; import '../../../src/common.dart';
import '../../src/context.dart'; import '../../../src/context.dart';
import '../../src/fakes.dart'; import '../../../src/fakes.dart';
import '../fake_native_assets_build_runner.dart'; import '../fake_native_assets_build_runner.dart';
void main() { void main() {
......
...@@ -21,9 +21,9 @@ import 'package:flutter_tools/src/base/logger.dart'; ...@@ -21,9 +21,9 @@ import 'package:flutter_tools/src/base/logger.dart';
import 'package:flutter_tools/src/base/os.dart'; import 'package:flutter_tools/src/base/os.dart';
import 'package:native_assets_cli/native_assets_cli_internal.dart'; import 'package:native_assets_cli/native_assets_cli_internal.dart';
import '../src/common.dart'; import '../../src/common.dart';
import 'test_utils.dart' show ProcessResultMatcher, fileSystem, platform; import '../test_utils.dart' show ProcessResultMatcher, fileSystem, platform;
import 'transition_test_utils.dart'; import '../transition_test_utils.dart';
final String hostOs = platform.operatingSystem; final String hostOs = platform.operatingSystem;
......
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