commit d03a416ac1e1253ace95e8fa33c6dacf51433c00
author Dhanya Ganesh <dhanyaganesh@chromium.org> Wed Aug 25 00:21:50 2021 +0000
committer Commit Bot <commit-bot@chromium.org> Fri Aug 27 21:28:38 2021 +0000
tree 5ff5fa9b602df7d1ff4a223f7b24b17f551a91d4
parent fc5a51a57e68848b3f102ae09e76f010e13a8dc2

api: Populate UprevPackagesResponse.packages field

Logic and unittest to populate the new field.

BUG=b:195415844
TEST=run_tests

Change-Id: Ib36f8936b2ec31870932d601dd9479ee6e7fec05
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/chromite/+/3115229
Reviewed-by: Alex Klein <saklein@chromium.org>
Commit-Queue: Dhanya Ganesh <dhanyaganesh@chromium.org>
Tested-by: Dhanya Ganesh <dhanyaganesh@chromium.org>

api/controller/packages_unittest.py

diff --git a/api/controller/packages_unittest.py b/api/controller/packages_unittest.py
index 1c7ab80..83dbd39 100644
--- a/api/controller/packages_unittest.py
+++ b/api/controller/packages_unittest.py
@@ -78,11 +78,12 @@
     targets = ['foo', 'bar']
     output_dir = '/tmp/uprev_output_dir'
     changed = ['/ebuild-1.0-r1.ebuild', '/ebuild-1.0-r2.ebuild']
+    revved_packages = ['cat1/pkg1-1.11', 'cat2/pkg2-1.12']
     expected_type = constants.BOTH_OVERLAYS
     request = self._GetRequest(targets=targets, overlay_type=self._BOTH,
                                output_dir=output_dir)
     uprev_patch = self.PatchObject(packages_service, 'uprev_build_targets',
-                                   return_value=changed)
+                                   return_value=(changed, revved_packages))
 
     packages_controller.Uprev(request, self.response, self.api_config)
 
@@ -98,6 +99,11 @@
       changed.remove(ebuild.path)
     self.assertFalse(changed)
 
+    for pkg in self.response.packages:
+      self.assertTrue(pkg.category.startswith('cat'))
+      self.assertTrue(pkg.package_name.startswith('pkg'))
+      self.assertTrue(pkg.version.startswith('1.1'))
+
 
 class UprevVersionedPackageTest(cros_test_lib.MockTestCase, ApiConfigMixin):
   """UprevVersionedPackage tests."""