commit ea0c89e9c3464c2be8f7e33b3b4368a4ea9c924c
author Alex Klein <saklein@chromium.org> Thu Sep 09 15:17:35 2021 -0600
committer Commit Bot <commit-bot@chromium.org> Wed Sep 15 22:55:22 2021 +0000
tree 7cfb157b886e48e1d40c6fb50a27d61f0b4b6978
parent 28e59a645aecb5a8800de0c0cb3fa0a2adac393d

package_info.SplitCPV: Refactor usages to parse.

Refactor another cluster of SplitCPV and the corresponding CPV usages.

BUG=b:187792048
TEST=run_tests

Change-Id: I08bec55f983489ef52241d0d2d8e492d4d6c578a
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/chromite/+/3152618
Tested-by: Alex Klein <saklein@chromium.org>
Reviewed-by: Chris McDonald <cjmcdonald@chromium.org>
Reviewed-by: Sergey Frolov <sfrolov@google.com>
Commit-Queue: Alex Klein <saklein@chromium.org>

api/controller/test.py

diff --git a/api/controller/test.py b/api/controller/test.py
index 6040cf5..f5168d9 100644
--- a/api/controller/test.py
+++ b/api/controller/test.py
@@ -135,10 +135,10 @@
   if not result.success:
     # Failed to run tests or some tests failed.
     # Record all failed packages.
-    for cpv in result.failed_cpvs:
+    for pkg_info in result.failed_pkgs:
       package_info_msg = output_proto.failed_packages.add()
-      controller_util.CPVToPackageInfo(cpv, package_info_msg)
-    if result.failed_cpvs:
+      controller_util.serialize_package_info(pkg_info, package_info_msg)
+    if result.failed_pkgs:
       return controller.RETURN_CODE_UNSUCCESSFUL_RESPONSE_AVAILABLE
     else:
       return controller.RETURN_CODE_COMPLETED_UNSUCCESSFULLY