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.
flutter · May 22, 2023 · 758ea6c · 758ea6c
1 parent 7a12d82
commit 758ea6c
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/packages/flutter_tools/lib/src/artifacts.dart b/packages/flutter_tools/lib/src/artifacts.dart
@@ -1108,7 +1108,7 @@ class CachedLocalWebSdkArtifacts implements Artifacts {
           );
         case Artifact.wasmOptBinary:
           return _fileSystem.path.join(
-            _getDartSdkPath(), 'bin', 'snapshots',
+            _getDartSdkPath(), 'bin', 'utils',
             _artifactToFileName(artifact, _platform, mode),
           );
         case Artifact.genSnapshot:

diff --git a/packages/flutter_tools/test/general.shard/artifacts_test.dart b/packages/flutter_tools/test/general.shard/artifacts_test.dart
@@ -418,6 +418,13 @@ void main() {
         fileSystem.path.join('/flutter', 'prebuilts', 'linux-x64', 'dart-sdk',
             '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', () {