[flutter_tools] Add bot configuration to run web_tool_tests for linux, mac, and windows (#70412)

* Move web integration tool tests to web.shard

Web integration tool tests depend on DDC changes in SDK. This change
moves them to a separate shard and subshard so CI bot configurations
can run them separately.

In particular, with will allow running those tests on dart CI flutter
HHH web bot instead of a non-web one, allowing early detection and easy
classification of issues caused by SDK changes as VM- or Web related.

* Enabled verbose mode for flaky web_tool_tests

* fixed local engine crash with --enable-asserts

* Disable failing tests, fix local engine with asserts

* Removed mac web_tool_tests bot

* Addressed CR comments

* Add disabled mac web_tool_tests and disable win web_tool_tests

dev/prod_builders.json

@@ -114,6 +114,12 @@
          "task_name": "linux_tool_tests",
          "flaky": false
       },
+      {
+         "name": "Linux web_tool_tests",
+         "repo": "flutter",
+         "task_name": "linux_web_tool_tests",
+         "flaky": true
+      },
       {
          "name": "Linux web_benchmarks_html",
          "repo": "flutter",
@@ -329,6 +335,12 @@
          "repo": "flutter",
          "task_name": "win_tool_tests",
          "flaky": false
+      },
+      {
+         "name": "Windows web_tool_tests",
+         "repo": "flutter",
+         "task_name": "win_web_tool_tests",
+         "flaky": true
       }
    ]
 }

dev/try_builders.json

@@ -132,6 +132,13 @@
          "enabled":true,
          "run_if":["dev/", "packages/flutter_tools/", "bin/"]
       },
+      {
+         "name":"Linux web_tool_tests",
+         "repo":"flutter",
+         "task_name":"linux_web_tool_tests",
+         "enabled":true,
+         "run_if":["dev/", "packages/flutter_tools/", "bin/"]
+      },
       {
          "name":"Linux web_benchmarks_html",
          "repo":"flutter",
@@ -290,6 +297,13 @@
          "enabled":true,
          "run_if":["dev/**", "packages/flutter_tools/**", "bin/**"]
       },
+      {
+         "name":"Mac web_tool_tests",
+         "repo":"flutter",
+         "task_name":"mac_web_tool_tests",
+         "enabled":false,
+         "run_if":["dev/**", "packages/flutter_tools/**", "bin/**"]
+      },
       {
          "name": "Windows build_aar_module_test",
          "repo": "flutter",
@@ -370,6 +384,12 @@
         "repo": "flutter",
         "task_name": "win_tool_tests",
         "enabled":true
+      },
+      {
+         "name": "Windows web_tool_tests",
+         "repo": "flutter",
+         "task_name": "win_web_tool_tests",
+         "enabled":false
       }
    ]
 }

packages/flutter_tools/lib/src/artifacts.dart

@@ -536,7 +536,8 @@ class LocalEngineArtifacts implements Artifacts {
   @override
   String getArtifactPath(Artifact artifact, { TargetPlatform platform, BuildMode mode }) {
     platform ??= _currentHostPlatform(_platform);
-    final String artifactFileName = _artifactToFileName(artifact, platform, mode);
+    final bool isDirectoryArtifact = artifact == Artifact.flutterWebSdk || artifact == Artifact.flutterPatchedSdkPath;
+    final String artifactFileName = isDirectoryArtifact ? null : _artifactToFileName(artifact, platform, mode);
     switch (artifact) {
       case Artifact.genSnapshot:
         return _genSnapshotPath();