commit e652ba19fd3cbe7939e8a9b3dc23f34b10ff50b9
author Mike Frysinger <vapier@chromium.org> Sun Sep 08 00:57:43 2019 -0400
committer Commit Bot <commit-bot@chromium.org> Mon Sep 09 22:53:36 2019 +0000
tree 80fbae2cc046dd7ca7235f8b8b1daa9b0bd0a37b
parent 51262fb3cca7fee04dcdbfed81e827640a690e21

key_value_store: pull LoadKeyValueFile into a module

This isn't used too often, so lets pull it out into a standalone
module.  It also has no real deps on other bits, so move it to
the utils namespaces.

BUG=chromium:997354
TEST=`./run_tests` passes

Change-Id: Ieee6b3476600cca98aaad7597cda57b6a79f3d55
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/chromite/+/1790373
Reviewed-by: Alex Klein <saklein@chromium.org>
Commit-Queue: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>

api/controller/test_unittest.py

diff --git a/api/controller/test_unittest.py b/api/controller/test_unittest.py
index 4a4caaf..b5422f5 100644
--- a/api/controller/test_unittest.py
+++ b/api/controller/test_unittest.py
@@ -23,6 +23,7 @@
 from chromite.lib import portage_util
 from chromite.scripts import cros_set_lsb_release
 from chromite.service import test as test_service
+from chromite.utils import key_value_store
 
 
 class BuildTargetUnitTestTest(cros_test_lib.MockTempDirTestCase,
@@ -271,7 +272,7 @@
     response = self._Output()
 
     self.PatchObject(
-        cros_build_lib, 'LoadKeyValueFile',
+        key_value_store, 'LoadFile',
         return_value={cros_set_lsb_release.LSB_KEY_BUILDER_PATH: self.builder})
 
     test_controller.MoblabVmTest(request, response, self.api_config)
@@ -295,7 +296,7 @@
     request = self._Input()
     response = self._Output()
 
-    self.PatchObject(cros_build_lib, 'LoadKeyValueFile', return_value={})
+    self.PatchObject(key_value_store, 'LoadFile', return_value={})
 
     with self.assertRaises(cros_build_lib.DieSystemExit):
       test_controller.MoblabVmTest(request, response, self.api_config)