commit 6ca42ddeb63e05df9147fec2fe944a96aaa29e2e
author Garry Wang <xianuowang@chromium.org> Mon Feb 28 21:54:19 2022 -0800
committer Commit Bot <commit-bot@chromium.org> Tue Mar 01 20:16:10 2022 +0000
tree f0e30a4d52cd2684a4a3e03abf3d059c4b7e3bb2
parent 7511346204b4695a44d6b7018ca51eaeaa8d6683

suite_scheduler: Support schedule multi-DUTs test with Android boards/models

BUG=b:214074008
TEST=bin/run_tests

Change-Id: I2083312e50461e457b7eebf3f4ad21371d7f7ddd
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/infra/suite_scheduler/+/3496724
Tested-by: Garry Wang <xianuowang@chromium.org>
Reviewed-by: Jared Loucks <jaredloucks@google.com>
Commit-Queue: Garry Wang <xianuowang@chromium.org>

task_unittest.py

diff --git a/task_unittest.py b/task_unittest.py
index f702573..382bfd7 100644
--- a/task_unittest.py
+++ b/task_unittest.py
@@ -39,6 +39,9 @@
   def get_cros_model_map(self):
     return {'coral': ['robo', 'nasher', 'lava'], 'hatch': ['akemi', 'kled']}
 
+  def get_android_model_map(self):
+    return {'pixel3': ['pixel3xl'], 'pixel4': ['pixel4a']}
+
 
 class FakeBigqueryRestClient(object):
 
@@ -838,3 +841,82 @@
     self.task._schedule_multi_duts_cros_builds(
         self.BUILD_DICT, self.RECENT_BUILD_DICT, self._fake_configs)
     self.assertEqual(self._mock_push.call_count, 4)
+
+
+class TaskMultiDUTsWithAndroidByBoardTestCase(TaskBaseMultiDUTsTestCase):
+
+  def setUp(self):
+    super(TaskMultiDUTsWithAndroidByBoardTestCase, self).setUp(
+        multi_dut_boards='coral;pixel3, hatch;pixel4')
+
+    mock_push = mock.patch('task.Task._push_suite')
+    self._mock_push = mock_push.start()
+    self.addCleanup(mock_push.stop)
+
+  def testGetMultiDUTsBuildTargetsDictWithAndroidByBoard(self):
+    """Test the correct build targets dict can be generated with
+    Android devices when specified by board.
+    """
+    self.task.any_model = True
+    d = self.task._get_multi_duts_build_targets_dict(self._fake_configs)
+    expected_dict = {
+      'coral': [
+        [
+          task.BuildTarget('coral', None, None),
+          task.BuildTarget('pixel3', None, None),
+        ],
+      ],
+      'hatch': [
+        [
+          task.BuildTarget('hatch', None, None),
+          task.BuildTarget('pixel4', None, None),
+        ],
+      ]
+    }
+    self.assertEqual(d, expected_dict)
+
+  def testScheduleMultiDUTsWithAndroidByBoard(self):
+    """Test schedule multi DUTs cros build with Android by board."""
+    self.task.any_model = True
+    self.task._schedule_multi_duts_cros_builds(
+        self.BUILD_DICT, self.RECENT_BUILD_DICT, self._fake_configs)
+    self.assertEqual(self._mock_push.call_count, 4)
+
+
+class TaskMultiDUTsWithAndroidByModelTestCase(TaskBaseMultiDUTsTestCase):
+
+  def setUp(self):
+    super(TaskMultiDUTsWithAndroidByModelTestCase, self).setUp(
+        multi_dut_models='robo;pixel3xl, kled;pixel4a')
+
+    mock_push = mock.patch('task.Task._push_suite')
+    self._mock_push = mock_push.start()
+    self.addCleanup(mock_push.stop)
+
+  def testGetMultiDUTsBuildTargetsDictWithAndroidByModel(self):
+    """Test the correct build targets dict can be generated with
+    Android devices when specified by model.
+    """
+    d = self.task._get_multi_duts_build_targets_dict(self._fake_configs)
+    expected_dict = {
+      'coral': [
+        [
+          task.BuildTarget('coral', 'robo', None),
+          task.BuildTarget('pixel3', 'pixel3xl', None),
+        ],
+      ],
+      'hatch': [
+        [
+          task.BuildTarget('hatch', 'kled', None),
+          task.BuildTarget('pixel4', 'pixel4a', None),
+        ],
+      ]
+    }
+    self.assertEqual(d, expected_dict)
+
+  def testScheduleMultiDUTsWithAndroidByModel(self):
+    """Test schedule multi DUTs cros build with Android by model."""
+    self.task.any_model = True
+    self.task._schedule_multi_duts_cros_builds(
+        self.BUILD_DICT, self.RECENT_BUILD_DICT, self._fake_configs)
+    self.assertEqual(self._mock_push.call_count, 4)