From 8b6277e63868c2029f1e2327879b7899be44fbe2 Mon Sep 17 00:00:00 2001
From: godofredoc <godofredoc@google.com>
Date: Tue, 28 Nov 2023 15:58:55 -0800
Subject: [PATCH] Move analysis test to shard tests. (#139161)

This is part of the effort to deprecated adhoc tests.

Bug: https://github.com/flutter/flutter/issues/139153
---
 .ci.yaml           | 14 ++++++--------
 TESTOWNERS         |  3 +--
 dev/bots/test.dart | 14 ++++++++++++++
 3 files changed, 21 insertions(+), 10 deletions(-)

diff --git a/.ci.yaml b/.ci.yaml
index 1e1c0c812baa..f3ce32dde2bf 100644
--- a/.ci.yaml
+++ b/.ci.yaml
@@ -242,13 +242,12 @@ platform_properties:
 
 targets:
   - name: Linux analyze
-    recipe: flutter/flutter
+    recipe: flutter/flutter_drone
     timeout: 60
     properties:
+      shard: analyze
       tags: >
-        ["framework","hostonly","linux"]
-      validation: analyze
-      validation_name: Analyze
+        ["framework","hostonly","shard","linux"]
 
   - name: Linux analyzer_benchmark
     recipe: devicelab/devicelab_drone
@@ -2948,14 +2947,13 @@ targets:
   - name: Staging_build_linux analyze
     presubmit: false
     bringup: true
-    recipe: flutter/flutter
+    recipe: flutter/flutter_drone
     timeout: 60
     properties:
+      shard: analyze
       ignore_flakiness: "true"
       tags: >
-        ["framework","hostonly","linux"]
-      validation: analyze
-      validation_name: Analyze
+        ["framework","hostonly","shard","linux"]
 
   - name: Mac_benchmark animated_complex_opacity_perf_macos__e2e_summary
     presubmit: false
diff --git a/TESTOWNERS b/TESTOWNERS
index deb1ebf3993e..2ee8b537d86f 100644
--- a/TESTOWNERS
+++ b/TESTOWNERS
@@ -295,8 +295,6 @@
 /dev/devicelab/bin/tasks/windows_startup_test.dart @loic-sharma @flutter/desktop
 
 ## Host only framework tests
-# Linux analyze
-/dev/bots/analyze.dart @HansMuller @flutter/framework
 # Linux docs
 # Linux docs_test
 # Linux docs_publish
@@ -321,6 +319,7 @@
 # TODO(keyonghan): add files/paths for below framework host only testss.
 # https://github.com/flutter/flutter/issues/82068
 #
+# analyze @HansMuller @flutter/framework
 # build_tests @eliasyishak @flutter/tool
 # ci_yaml flutter roller @caseyhillers @flutter/infra
 # coverage @godofredoc @flutter/infra
diff --git a/dev/bots/test.dart b/dev/bots/test.dart
index 06f473e29e41..4ae7f94d26e3 100644
--- a/dev/bots/test.dart
+++ b/dev/bots/test.dart
@@ -264,6 +264,7 @@ Future<void> main(List<String> args) async {
       'skp_generator': _runSkpGeneratorTests,
       'realm_checker': _runRealmCheckerTest,
       'customer_testing': _runCustomerTesting,
+      'analyze': _runAnalyze,
       kTestHarnessShardName: _runTestHarnessTests, // Used for testing this script; also run as part of SHARD=framework_tests, SUBSHARD=misc.
     });
   } catch (error, stackTrace) {
@@ -1588,6 +1589,19 @@ Future<void> _runCustomerTesting() async {
   );
 }
 
+// Runs analysis tests.
+Future<void> _runAnalyze() async {
+  printProgress('${green}Running analysis testing$reset');
+  await runCommand(
+    'dart',
+    <String>[
+      '--enable-asserts',
+      path.join(flutterRoot, 'dev', 'bots', 'analyze.dart'),
+    ],
+    workingDirectory: flutterRoot,
+  );
+}
+
 /// Runs the skp_generator from the flutter/tests repo.
 ///
 /// See also the customer_tests shard.