commit 239459af80e66ce079737a38f842f28f64fc76cc
author Lizzy Presland <zland@google.com> Thu May 05 22:03:19 2022 +0000
committer Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> Mon May 09 21:41:49 2022 +0000
tree 110cfe53a0c97ef96850c34c7584594dd5f5950e
parent 6c9d5985253ec4efd62a6f39446940ac8d2dd9b2

api: Stop populating repeated PackageInfo field

In preparation for deprecating and reserving the old repeated
PackageInfo field which only contained the names of the failed packages,
we want to stop populating and referencing the field. Recipes no longer
references this field for retrieving and displaying the names of failed
packages, so it is safe to stop populating the field in response protos.

BUG=b:204816060
TEST=./run_tests

Change-Id: Id73ee535d07f662fbf8582f342258579ed4284c2
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/chromite/+/3631025
Auto-Submit: Lizzy Presland <zland@google.com>
Reviewed-by: Ram Chandrasekar <rchandrasekar@google.com>
Tested-by: Lizzy Presland <zland@google.com>
Commit-Queue: Lizzy Presland <zland@google.com>

api/controller/test_unittest.py

diff --git a/api/controller/test_unittest.py b/api/controller/test_unittest.py
index f76bd09..37174c1 100644
--- a/api/controller/test_unittest.py
+++ b/api/controller/test_unittest.py
@@ -200,11 +200,11 @@
                                              self.mock_error_config)
     patch.assert_not_called()
     self.assertEqual(controller.RETURN_CODE_UNSUCCESSFUL_RESPONSE_AVAILABLE, rc)
-    self.assertTrue(response.failed_packages)
-    self.assertEqual(response.failed_packages[0].category, 'foo')
-    self.assertEqual(response.failed_packages[0].package_name, 'bar')
-    self.assertEqual(response.failed_packages[1].category, 'cat')
-    self.assertEqual(response.failed_packages[1].package_name, 'pkg')
+    self.assertTrue(response.failed_package_data)
+    self.assertEqual(response.failed_package_data[0].name.category, 'foo')
+    self.assertEqual(response.failed_package_data[0].name.package_name, 'bar')
+    self.assertEqual(response.failed_package_data[1].name.category, 'cat')
+    self.assertEqual(response.failed_package_data[1].name.package_name, 'pkg')
 
   def testInvalidPackageFails(self):
     """Test missing result path fails."""
@@ -244,13 +244,7 @@
                                              self.api_config)
 
     self.assertEqual(controller.RETURN_CODE_UNSUCCESSFUL_RESPONSE_AVAILABLE, rc)
-    self.assertTrue(output_msg.failed_packages)
     self.assertTrue(output_msg.failed_package_data)
-    # TODO(b/206514844): remove when field is deleted
-    failed = []
-    for pi in output_msg.failed_packages:
-      failed.append((pi.category, pi.package_name))
-    self.assertCountEqual(expected, failed)
 
     failed_with_logs = []
     for data in output_msg.failed_package_data:
@@ -278,7 +272,7 @@
                                              self.api_config)
 
     self.assertEqual(controller.RETURN_CODE_COMPLETED_UNSUCCESSFULLY, rc)
-    self.assertFalse(output_msg.failed_packages)
+    self.assertFalse(output_msg.failed_package_data)
 
   def testBuildTargetUnitTest(self):
     """Test BuildTargetUnitTest successful call."""
@@ -292,7 +286,7 @@
     response = self._GetOutput()
     test_controller.BuildTargetUnitTest(input_msg, response,
                                         self.api_config)
-    self.assertFalse(response.failed_packages)
+    self.assertFalse(response.failed_package_data)
 
 
 class DockerConstraintsTest(cros_test_lib.MockTestCase):