commit 09937019f1638d61a69bab7053699c8b44428428
author Brian Norris <briannorris@chromium.org> Fri Mar 31 15:16:55 2023 -0700
committer Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> Tue Apr 04 20:11:27 2023 +0000
tree 2f877b845819bf190a2c05674641032815cc7180
parent 78ff85837118a6ba7980a76edb05cccbf06b8f41

api: controller: Plumb chroot.out_path in tests

Fix some typing while we're at it, since protobufs want strings, but
chroot_lib wants Path.

BUG=b:265885353
TEST=./run_tests

Change-Id: I26cfe5141af8f94c5db18e9f5bbd061b6c111895
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/chromite/+/4391049
Tested-by: Brian Norris <briannorris@chromium.org>
Reviewed-by: Sergey Frolov <sfrolov@google.com>
Commit-Queue: Brian Norris <briannorris@chromium.org>

api/controller/controller_util.py

diff --git a/api/controller/controller_util.py b/api/controller/controller_util.py
index 0cf3a59..0476676 100644
--- a/api/controller/controller_util.py
+++ b/api/controller/controller_util.py
@@ -7,6 +7,7 @@
 import glob
 import logging
 import os
+from pathlib import Path
 from typing import Iterable, Optional, TYPE_CHECKING, Union
 
 from chromite.api.gen.chromite.api import sysroot_pb2
@@ -51,7 +52,7 @@
     path = chroot_message.path or constants.DEFAULT_CHROOT_PATH
     cache_dir = chroot_message.cache_dir
     chrome_root = chroot_message.chrome_dir
-    out_path = chroot_message.out_path or constants.DEFAULT_OUT_PATH
+    out_path = Path(chroot_message.out_path) or constants.DEFAULT_OUT_PATH
 
     use_flags = [u.flag for u in chroot_message.env.use_flags]
     features = [f.feature for f in chroot_message.env.features]