Fix git branch parsing.

tests/git_common_test.py

 #!/usr/bin/env python
 # Copyright 2013 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 git_common.py"""
 
 import binascii
 import collections
 import os
@@ skipping 212 matching lines @@
 
   def testCurrentBranch(self):
     def cur_branch_out_of_git():
       os.chdir('..')
       return self.gc.current_branch()
     self.assertIsNone(self.repo.run(cur_branch_out_of_git))
 
     self.repo.git('checkout', 'branch_D')
     self.assertEqual(self.repo.run(self.gc.current_branch), 'branch_D')
 
-  @unittest.skip('broken by git 2.4')
   def testBranches(self):
     # This check fails with git 2.4 (see crbug.com/487172)
     self.assertEqual(self.repo.run(set, self.gc.branches()),
                      {'master', 'branch_D', 'root_A'})
 
   def testDormant(self):
     self.assertFalse(self.repo.run(self.gc.is_dormant, 'master'))
     self.repo.git('config', 'branch.master.dormant', 'true')
     self.assertTrue(self.repo.run(self.gc.is_dormant, 'master'))
 
@@ skipping 197 matching lines @@
   COMMIT_L = {'file': {'data': 'L'}}
 
   def setUp(self):
     super(GitMutableStructuredTest, self).setUp()
     self.repo.git('branch', '--set-upstream-to', 'root_X', 'branch_Z')
     self.repo.git('branch', '--set-upstream-to', 'branch_G', 'branch_K')
     self.repo.git('branch', '--set-upstream-to', 'branch_K', 'branch_L')
     self.repo.git('branch', '--set-upstream-to', 'root_A', 'branch_G')
     self.repo.git('branch', '--set-upstream-to', 'root_X', 'root_A')
 
-  @unittest.skip('broken by git 2.4')
   def testTooManyBranches(self):
     for i in xrange(30):
       self.repo.git('branch', 'a'*i)
 
     _, rslt = self.repo.capture_stdio(list, self.gc.branches())
     self.assertIn('too many branches (39/20)', rslt)
 
     self.repo.git('config', 'depot-tools.branch-limit', 'cat')
 
     _, rslt = self.repo.capture_stdio(list, self.gc.branches())
@@ skipping 66 matching lines @@
     # base and recompute it.
     self.repo.run(self.gc.run, 'branch', '-u', 'root_A')
     self.assertEqual(
       self.repo['A'],
       self.repo.run(self.gc.get_or_create_merge_base, 'branch_K')
     )
 
     self.assertIsNone(
       self.repo.run(self.gc.get_or_create_merge_base, 'branch_DOG'))
 
-  @unittest.skip('broken by git 2.4')
   def testGetBranchTree(self):
     skipped, tree = self.repo.run(self.gc.get_branch_tree)
     # This check fails with git 2.4 (see crbug.com/487172)
     self.assertEqual(skipped, {'master', 'root_X', 'branch_DOG', 'root_CAT'})
     self.assertEqual(tree, {
       'branch_G': 'root_A',
       'root_A': 'root_X',
       'branch_K': 'branch_G',
       'branch_L': 'branch_K',
       'branch_Z': 'root_X'
@@ skipping 175 matching lines @@
       self.assertIsNone(self.gc.thaw())
       self.assertIsNotNone(self.gc.thaw())  # One thaw should thaw everything
 
       self.assertEquals(self.repo.git('status', '--porcelain').stdout, STATUS_1)
 
     self.repo.run(inner)
 
 
 if __name__ == '__main__':
   sys.exit(coverage_utils.covered_main(
-    os.path.join(DEPOT_TOOLS_ROOT, 'git_common.py'),
-    required_percentage=88.0
-  ))
+    os.path.join(DEPOT_TOOLS_ROOT, 'git_common.py')))