Removed unnecessary (non-gitpython) tests and fixed flake8

Author: Byron

git/__init__.py

@@ -3,6 +3,7 @@
 #
 # This module is part of GitPython and is released under
 # the BSD License: http://www.opensource.org/licenses/bsd-license.php
+# flake8: noqa
 
 import os
 import sys

git/exc.py

@@ -5,7 +5,7 @@
 # the BSD License: http://www.opensource.org/licenses/bsd-license.php
 """ Module containing all exceptions thrown througout the git package, """
 
-from gitdb.exc import *
+from gitdb.exc import *     # NOQA
 
 
 class InvalidGitRepositoryError(Exception):

git/index/__init__.py

@@ -1,5 +1,5 @@
 """Initialize the index package"""
-
+# flake8: noqa
 from __future__ import absolute_import
 
 from .base import *

git/objects/__init__.py

@@ -1,6 +1,7 @@
 """
 Import all submodules main classes into the package space
 """
+# flake8: noqa
 from __future__ import absolute_import
 import inspect
 from .base import *

git/objects/blob.py

@@ -17,9 +17,9 @@ class Blob(base.IndexObject):
     type = "blob"
 
     # valid blob modes
-    executable_mode = 0100755
-    file_mode = 0100644
-    link_mode = 0120000
+    executable_mode = 0o100755
+    file_mode = 0o100644
+    link_mode = 0o120000
 
     __slots__ = tuple()
 

git/objects/tree.py

@@ -119,10 +119,10 @@ class Tree(IndexObject, diff.Diffable, util.Traversable, util.Serializable):
     __slots__ = "_cache"
 
     # actual integer ids for comparison
-    commit_id = 016     # equals stat.S_IFDIR | stat.S_IFLNK - a directory link
-    blob_id = 010
-    symlink_id = 012
-    tree_id = 004
+    commit_id = 0o16     # equals stat.S_IFDIR | stat.S_IFLNK - a directory link
+    blob_id = 0o10
+    symlink_id = 0o12
+    tree_id = 0o04
 
     _map_id_to_type = {
         commit_id: Submodule,

git/refs/__init__.py

@@ -1,4 +1,4 @@
-
+# flake8: noqa
 from __future__ import absolute_import
 # import all modules in order, fix the names they require
 from .symbolic import *
@@ -7,16 +7,4 @@
 from .tag import *
 from .remote import *
 
-# name fixes
-from . import head
-head.RemoteReference = RemoteReference
-del(head)
-
-
-from . import symbolic
-for item in (HEAD, Head, RemoteReference, TagReference, Reference, SymbolicReference):
-    setattr(symbolic, item.__name__, item)
-del(symbolic)
-
-
 from .log import *

git/refs/head.py

@@ -81,7 +81,7 @@ def reset(self, commit='HEAD', index=True, working_tree=False,
 
         try:
             self.repo.git.reset(mode, commit, add_arg, paths, **kwargs)
-        except GitCommandError, e:
+        except GitCommandError as e:
             # git nowadays may use 1 as status to indicate there are still unstaged
             # modifications after the reset
             if e.status != 1:
@@ -134,6 +134,7 @@ def set_tracking_branch(self, remote_reference):
         :param remote_reference: The remote reference to track or None to untrack
             any references
         :return: self"""
+        from .remote import RemoteReference
         if remote_reference is not None and not isinstance(remote_reference, RemoteReference):
             raise ValueError("Incorrect parameter type: %r" % remote_reference)
         # END handle type
@@ -156,6 +157,7 @@ def tracking_branch(self):
         """
         :return: The remote_reference we are tracking, or None if we are
             not a tracking branch"""
+        from .remote import RemoteReference
         reader = self.config_reader()
         if reader.has_option(self.k_config_remote) and reader.has_option(self.k_config_remote_ref):
             ref = Head(self.repo, Head.to_full_path(reader.get_value(self.k_config_remote_ref)))

git/refs/symbolic.py

@@ -625,6 +625,7 @@ def from_path(cls, repo, path):
 
         # Names like HEAD are inserted after the refs module is imported - we have an import dependency
         # cycle and don't want to import these names in-function
+        from . import HEAD, Head, RemoteReference, TagReference, Reference
         for ref_type in (HEAD, Head, RemoteReference, TagReference, Reference, SymbolicReference):
             try:
                 instance = ref_type(repo, path)

git/repo/__init__.py

@@ -1,3 +1,4 @@
 """Initialize the Repo package"""
+# flake8: noqa
 from __future__ import absolute_import
 from .base import *