[Predator] Add local cache for get command output.

appengine/findit/common/test/repo_util_test.py

Index: appengine/findit/common/test/repo_util_test.py
diff --git a/appengine/findit/common/test/repo_util_test.py b/appengine/findit/common/test/repo_util_test.py
index 48c1e22d2db96aaf16ff15749d5640166be7bf4f..146ead662794e295ef6c2b00be49fd7c8b794701 100644
--- a/appengine/findit/common/test/repo_util_test.py
+++ b/appengine/findit/common/test/repo_util_test.py
@@ -10,6 +10,7 @@ import textwrap
 import urllib2
 
 from common import repo_util
+from common.cache_decorator import LocalCacher
 
 
 class RepoUtilTest(testing.AppengineTestCase):
@@ -141,7 +142,7 @@ class RepoUtilTest(testing.AppengineTestCase):
         self.command = command
 
       def communicate(self, *_):
-        return self.command, 'dummy'
+        return self.command, 'error'
 
       @property
       def returncode(self):
@@ -152,8 +153,12 @@ class RepoUtilTest(testing.AppengineTestCase):
 
     def _MockPopen(command, **_):
       return _MockProcess(command)
-
     self.mock(subprocess, 'Popen', _MockPopen)
+
+    def _MockLocalCacherGet(*_):
+      return None
+    self.mock(LocalCacher, 'Get', _MockLocalCacherGet)

[wrengr, 2016/10/27 16:36:47]

Why not just use a lambda. It's shorter and cleaner

[Sharu Jiang, 2016/10/27 21:59:05]

Done.

+
     output = repo_util.GetCommandOutput('command')
     self.assertEqual(output, 'command')