Unverified Commit 758ea6c0 authored by Jackson Gardner's avatar Jackson Gardner Committed by GitHub

Fix wasm-opt location when using local_web_sdk (#127355)

We were looking up the wrong path for `wasm-opt` previously when using the `local-web-sdk` flag.
parent 7a12d825
...@@ -1108,7 +1108,7 @@ class CachedLocalWebSdkArtifacts implements Artifacts { ...@@ -1108,7 +1108,7 @@ class CachedLocalWebSdkArtifacts implements Artifacts {
); );
case Artifact.wasmOptBinary: case Artifact.wasmOptBinary:
return _fileSystem.path.join( return _fileSystem.path.join(
_getDartSdkPath(), 'bin', 'snapshots', _getDartSdkPath(), 'bin', 'utils',
_artifactToFileName(artifact, _platform, mode), _artifactToFileName(artifact, _platform, mode),
); );
case Artifact.genSnapshot: case Artifact.genSnapshot:
......
...@@ -418,6 +418,13 @@ void main() { ...@@ -418,6 +418,13 @@ void main() {
fileSystem.path.join('/flutter', 'prebuilts', 'linux-x64', 'dart-sdk', fileSystem.path.join('/flutter', 'prebuilts', 'linux-x64', 'dart-sdk',
'bin', 'snapshots', 'dart2js.dart.snapshot'), 'bin', 'snapshots', 'dart2js.dart.snapshot'),
); );
expect(
artifacts.getArtifactPath(
Artifact.wasmOptBinary,
platform: TargetPlatform.web_javascript),
fileSystem.path.join('/flutter', 'prebuilts', 'linux-x64', 'dart-sdk',
'bin', 'utils', 'wasm-opt'),
);
}); });
testWithoutContext('getEngineType', () { testWithoutContext('getEngineType', () {
......
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