Remove more logic out of trychange.py into scm.py.

tests/trychange_unittest.py

 #!/usr/bin/python
 # Copyright (c) 2009 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 """Unit tests for trychange.py."""
 
 import optparse
 
 # Local imports
 import trychange
 from super_mox import mox, SuperMoxTestBase
 
 
 class TryChangeTestsBase(SuperMoxTestBase):
   """Setups and tear downs the mocks but doesn't test anything as-is."""
   def setUp(self):
     SuperMoxTestBase.setUp(self)
     self.mox.StubOutWithMock(trychange.gclient_utils, 'CheckCall')
     self.mox.StubOutWithMock(trychange.scm.GIT, 'Capture')
     self.mox.StubOutWithMock(trychange.scm.GIT, 'GenerateDiff')
+    self.mox.StubOutWithMock(trychange.scm.GIT, 'GetCheckoutRoot')
+    self.mox.StubOutWithMock(trychange.scm.GIT, 'GetPatchName')
     self.mox.StubOutWithMock(trychange.scm.GIT, 'GetEmail')
     self.mox.StubOutWithMock(trychange.scm.SVN, 'DiffItem')
     self.mox.StubOutWithMock(trychange.scm.SVN, 'GenerateDiff')
     self.mox.StubOutWithMock(trychange.scm.SVN, 'GetCheckoutRoot')
     self.mox.StubOutWithMock(trychange.scm.SVN, 'GetEmail')
     self.fake_root = self.Dir()
     self.expected_files = ['foo.txt', 'bar.txt']
     self.options = optparse.Values()
     self.options.files = self.expected_files
     self.options.diff = None
@@ skipping 40 matching lines @@
 class GITUnittest(TryChangeTestsBase):
   """trychange.GIT tests."""
   def testMembersChanged(self):
     members = [
       'GetBots', 'GetFileNames', 'GetLocalRoot',
     ]
     # If this test fails, you should add the relevant test.
     self.compareMembers(trychange.GIT, members)
 
   def testBasic(self):
-    trychange.gclient_utils.CheckCall(
-        ['git', 'rev-parse', '--show-cdup']).AndReturn(self.fake_root)
-    trychange.os.path.abspath(self.fake_root).AndReturn(self.fake_root)
+    trychange.os.getcwd().AndReturn(self.fake_root)
+    trychange.scm.GIT.GetCheckoutRoot(self.fake_root).AndReturn(self.fake_root)
     trychange.scm.GIT.GenerateDiff(self.fake_root).AndReturn('a diff')
-    trychange.gclient_utils.CheckCall(
-        ['git', 'symbolic-ref', 'HEAD']).AndReturn('refs/heads/random branch')
-    trychange.scm.GIT.GetEmail('.').AndReturn('georges@example.com')
+    trychange.scm.GIT.GetPatchName(self.fake_root).AndReturn('bleh-1233')
+    trychange.scm.GIT.GetEmail(self.fake_root).AndReturn('georges@example.com')
     self.mox.ReplayAll()
     git = trychange.GIT(self.options)
     self.assertEqual(git.GetFileNames(), self.expected_files)
     self.assertEqual(git.GetLocalRoot(), self.fake_root)
 
 
 if __name__ == '__main__':
   import unittest
   unittest.main()