commit d8cd4c69fc4d7f2a773fb064c269ab4eaab69aea
author Alex Klein <saklein@chromium.org> Mon Sep 14 13:37:47 2020 -0600
committer Commit Bot <commit-bot@chromium.org> Wed Sep 16 02:20:38 2020 +0000
tree 4417a4126e5804ba10316ac657eea9480aab3d17
parent 65fccf7b5f0490b105e5aa72208aaeb01a930207

dependency service: Cache dependency data.

Cache the dependency data to avoid repeated depgraph generation
on successive calls.

BUG=chromium:1086714
TEST=run_pytest

Change-Id: Id4747cab1f639ce76a92c805289c00edfdf9a05f
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/chromite/+/2411176
Tested-by: Alex Klein <saklein@chromium.org>
Reviewed-by: Chris McDonald <cjmcdonald@chromium.org>
Reviewed-by: Mike Frysinger <vapier@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 455bd9b..9dfd670 100644
--- a/api/controller/dependency_unittest.py
+++ b/api/controller/dependency_unittest.py
@@ -136,7 +136,7 @@
                                        self.api_config)
     self.assertEqual(self.response.dep_graph.build_target.name, 'deathstar')
     pkg_to_cpv.assert_called_once_with(package)
-    get_dep.assert_called_once_with('/build/target', 'target', [pkg_mock])
+    get_dep.assert_called_once_with('/build/target', 'target', (pkg_mock,))
 
   def testValidateOnly(self):
     """Sanity check that a validate only call does not execute any logic."""