controller/test: Refactor SplitCPV to parse.
BUG=chromium:1077379
TEST=./run_pytest
Cq-Depend: chromium:2454501
Change-Id: If38d475ac81abbcab23b1b23bb0bf789d580eb7b
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/chromite/+/2454194
Tested-by: Alex Klein <saklein@chromium.org>
Commit-Queue: Alex Klein <saklein@chromium.org>
Reviewed-by: Michael Mortensen <mmortensen@google.com>
diff --git a/api/controller/test.py b/api/controller/test.py
index 7db8a59..4ef4f58 100644
--- a/api/controller/test.py
+++ b/api/controller/test.py
@@ -68,10 +68,10 @@
def _BuildTargetUnitTestFailedResponse(_input_proto, output_proto, _config):
"""Add failed packages to a failed response."""
packages = ['foo/bar', 'cat/pkg']
- failed_cpvs = [package_info.SplitCPV(p, strict=False) for p in packages]
- for cpv in failed_cpvs:
- package_info_msg = output_proto.failed_packages.add()
- controller_util.CPVToPackageInfo(cpv, package_info_msg)
+ for pkg in packages:
+ pkg_info = package_info.parse(pkg)
+ pkg_info_msg = output_proto.failed_packages.add()
+ controller_util.serialize_package_info(pkg_info, pkg_info_msg)
@faux.success(_BuildTargetUnitTestResponse)