commit bbce7e49052d4a5cc58631bcee56b954e897f83a
author Michael Mortensen <mmortensen@google.com> Fri May 01 15:03:49 2020 -0600
committer Commit Bot <commit-bot@chromium.org> Tue May 05 05:33:09 2020 +0000
tree 46a6dafa9097987f9e03808e12eb15dff407ef7c
parent 8f6c632bfd85e7385bf6fd7a201d2c89329508c7

Add GetBuilderMetadata fields for android target metadata.

These fields can be added without adding new service methods.

BUG=chromium:1071620
TEST=manual, run_tests

Change-Id: Id6c85fece6d69a3b0a9599aa6058b288badca4d3
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/chromite/+/2175526
Reviewed-by: Alex Klein <saklein@chromium.org>
Commit-Queue: Michael Mortensen <mmortensen@google.com>
Tested-by: Michael Mortensen <mmortensen@google.com>

api/controller/packages_unittest.py

diff --git a/api/controller/packages_unittest.py b/api/controller/packages_unittest.py
index 722f273..b9dc22c 100644
--- a/api/controller/packages_unittest.py
+++ b/api/controller/packages_unittest.py
@@ -516,6 +516,39 @@
       packages_controller.GetBuilderMetadata(request, self.response,
                                              self.api_config)
 
+  def testGetBuilderMetadata(self):
+    """Verify basic return values."""
+    android_version = 'android_test_version'
+    self.PatchObject(packages_service, 'determine_android_version',
+                     return_value=android_version)
+    android_branch = 'android_test_branch'
+    self.PatchObject(packages_service, 'determine_android_branch',
+                     return_value=android_branch)
+    android_target = 'android_test_target'
+    self.PatchObject(packages_service, 'determine_android_target',
+                     return_value=android_target)
+    self.PatchObject(portage_util, 'GetBoardUseFlags',
+                     return_value=['arc', 'arcvm', 'big_little', 'cheets'])
+    request = self._GetRequest(board='betty')
+    packages_controller.GetBuilderMetadata(request, self.response,
+                                           self.api_config)
+    self.assertEqual(
+        self.response.build_target_metadata[0].build_target,
+        'betty')
+    self.assertEqual(
+        self.response.build_target_metadata[0].android_container_version,
+        android_version)
+    self.assertEqual(
+        self.response.build_target_metadata[0].android_container_branch,
+        android_branch)
+    self.assertEqual(
+        self.response.build_target_metadata[0].android_container_target,
+        android_target)
+    self.assertEqual(
+        self.response.build_target_metadata[0].arc_use_set,
+        True)
+
+
 class HasChromePrebuiltTest(cros_test_lib.MockTestCase, ApiConfigMixin):
   """HasChromePrebuilt tests."""