Refactor infra git libs and testing.

infra/libs/git2/__init__.py

Index: infra/libs/git2/__init__.py
diff --git a/infra/libs/git2/__init__.py b/infra/libs/git2/__init__.py
new file mode 100644
index 0000000000000000000000000000000000000000..845432ab640e6532bca7a36b7e113f0a631ad901
--- /dev/null
+++ b/infra/libs/git2/__init__.py
@@ -0,0 +1,12 @@
+# Copyright 2014 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.
+
+from infra.libs.git2.util import INVALID

[Vadim Sh., 2014/06/27 18:26:22]

why git2? :) Do we have git 1 in infra/infra? Alternatives: 'git', 'gitlib'..
dunno.

[agable, 2014/06/27 18:53:11]

Agreed -- I think now is the time to absorb git into these, and fix lkgr-finder
to use these instead.

[iannucci, 2014/06/28 08:33:35]

I'd rather do in a followup CL. AFAIK, neither lkgr nor git have any tests, so
I'd rather refactor them separately (preferably after, you know, adding tests
:p).

+from infra.libs.git2.util import CalledProcessError
+
+from infra.libs.git2.git2 import Commit
+from infra.libs.git2.git2 import Ref
+from infra.libs.git2.git2 import Repo

[agable, 2014/06/27 18:53:11]

Inconsistent between listing these all on their own lines vs. all on one line as
you do in data_structures/ and decorators/.

[iannucci, 2014/06/28 08:33:35]

I'm torn, but looks like google python style says 1/line, so I'll do that:
http://google-styleguide.googlecode.com/svn/trunk/pyguide.html?showone=Import...

+
+from infra.libs.git2 import data