commit b6847e205426cefbd567336d4b848a4bae61dacf
author Alex Klein <saklein@chromium.org> Mon Nov 07 10:44:48 2022 -0700
committer Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> Tue Nov 08 00:45:23 2022 +0000
tree 84593b2ec04309e916f4b50684fa9486257cb23f
parent 7b146c9c8ece0dac6eacf4c070b105f9f3effee6

api: Misc artifacts cleanup.

Dropped low value Args: sections. Added missing type hints.
Added missing return types. Fixed warning vs die convention
for one function.

BUG=None
TEST=run_tests

Change-Id: I268544be6b0beba892f4c8f5c11b578b333cb19c
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/chromite/+/4009362
Tested-by: Alex Klein <saklein@chromium.org>
Commit-Queue: Alex Klein <saklein@chromium.org>
Auto-Submit: Alex Klein <saklein@chromium.org>
Reviewed-by: Sergey Frolov <sfrolov@google.com>
Commit-Queue: Sergey Frolov <sfrolov@google.com>

api/controller/artifacts_unittest.py

diff --git a/api/controller/artifacts_unittest.py b/api/controller/artifacts_unittest.py
index de4f19c..fb651d2 100644
--- a/api/controller/artifacts_unittest.py
+++ b/api/controller/artifacts_unittest.py
@@ -787,10 +787,10 @@
             sysroot=self.does_not_exist,
         )
         response = self.response
-        with self.assertRaises(cros_build_lib.DieSystemExit):
-            artifacts.BundleSimpleChromeArtifacts(
-                request, response, self.api_config
-            )
+        artifacts.BundleSimpleChromeArtifacts(
+            request, response, self.api_config
+        )
+        self.assertFalse(self.response.artifacts)
 
     def testNoOutputDir(self):
         """Test no output dir fails."""