Make gclient_scm.py use cache_dir

tests/gclient_utils_test.py

Index: tests/gclient_utils_test.py
diff --git a/tests/gclient_utils_test.py b/tests/gclient_utils_test.py
index b0b74363130c6a9a721704f09ee787d1968bc8c3..11325c441ea0ccda43a57cc64a0fb84ba1036967 100755
--- a/tests/gclient_utils_test.py
+++ b/tests/gclient_utils_test.py
@@ -24,30 +24,6 @@ class GclientUtilBase(SuperMoxTestBase):
     self.mox.StubOutWithMock(subprocess2, 'communicate')
 
 
-class GclientUtilsUnittest(GclientUtilBase):

[agable, 2014/02/28 02:06:32]

You could fix the expectation in the test instead of removing the test entirely.

But honestly it's a horrible test so I'm okay with this.

[Ryan Tseng, 2014/02/28 02:22:50]

I got a verbal OK from Robbie already :)

-  """General gclient_utils.py tests."""
-  def testMembersChanged(self):
-    members = [
-        'Annotated', 'AutoFlush', 'CheckCallAndFilter', 'CommandToStr',
-        'CheckCallAndFilterAndHeader', 'Error', 'ExecutionQueue', 'FileRead',
-        'FileWrite', 'FindFileUpwards', 'FindGclientRoot',
-        'GetGClientRootAndEntries', 'GetEditor', 'GetExeSuffix',
-        'GetMacWinOrLinux', 'GitFilter', 'IsDateRevision', 'MakeDateRevision',
-        'MakeFileAutoFlush', 'MakeFileAnnotated', 'PathDifference',
-        'ParseCodereviewSettingsContent', 'NumLocalCpus', 'PrintableObject',
-        'RETRY_INITIAL_SLEEP', 'RETRY_MAX', 'RunEditor', 'GCLIENT_CHILDREN',
-        'GCLIENT_CHILDREN_LOCK', 'GClientChildren', 'SplitUrlRevision',
-        'SyntaxErrorToError', 'UpgradeToHttps', 'Wrapper', 'WorkItem',
-        'codecs', 'lockedmethod', 'logging', 'os', 'pipes', 'Queue', 're',
-        'rmtree', 'safe_makedirs', 'safe_rename', 'stat', 'subprocess',
-        'subprocess2', 'sys', 'tempfile', 'threading', 'time', 'urlparse',
-
-    ]
-    # If this test fails, you should add the relevant test.
-    self.compareMembers(gclient_utils, members)
-
-
-
 class CheckCallAndFilterTestCase(GclientUtilBase):
   class ProcessIdMock(object):
     def __init__(self, test_string):