commit 19c0674373ccf6a626896fe7a193e1d61beca37c
author Alex Klein <saklein@chromium.org> Mon Aug 08 18:10:27 2022 -0600
committer Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> Wed Jul 05 20:59:22 2023 +0000
tree badfb17aff73307ff72484a9e30d9a0c29fba799
parent 63a7c4e7f87a926359aaef1cab2938ee81d120f2

api: controller/dependency: revdep update.

Use the new affected packages argument instead of the reverse
dependencies. The affected packages excludes virtual packages
that have no concrete ancestors, but will also traverse virtual
revdeps, marking them and their concrete ancestors as affected
to ensure the packages that actually depend on the changed package
get rebuilt.

BUG=b:263919459
TEST=run_tests, CQ

Change-Id: Idd0c3e1c1494fd65775c0aae08c436b5e5cdcddd
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/chromite/+/4553959
Reviewed-by: Navil Perez <navil@google.com>
Tested-by: Alex Klein <saklein@chromium.org>
Commit-Queue: Alex Klein <saklein@chromium.org>

api/controller/dependency_unittest.py

diff --git a/api/controller/dependency_unittest.py b/api/controller/dependency_unittest.py
index eadae1e..83aeec8 100644
--- a/api/controller/dependency_unittest.py
+++ b/api/controller/dependency_unittest.py
@@ -208,7 +208,7 @@
             self.sysroot,
             src_paths=[],
             packages=[],
-            include_rev_dependencies=False,
+            include_affected_pkgs=False,
         )
 
     def testListResponse(self):
@@ -244,7 +244,7 @@
             self.sysroot,
             src_paths=[path],
             packages=[input_package_info],
-            include_rev_dependencies=True,
+            include_affected_pkgs=True,
         )
         self.assertCountEqual(
             [return_package_info_proto], self.response.package_deps