Removed 'from X import *' whereever possible

Author: Byron

git/test/lib/__init__.py

@@ -5,9 +5,8 @@
 # the BSD License: http://www.opensource.org/licenses/bsd-license.php
 
 import inspect
-from mock import *
-from asserts import *
-from helper import *
+from .asserts import *
+from .helper import *
 
 __all__ = [name for name, obj in locals().items()
            if not (name.startswith('_') or inspect.ismodule(obj))]

git/test/lib/asserts.py

@@ -5,15 +5,27 @@
 # the BSD License: http://www.opensource.org/licenses/bsd-license.php
 
 import re
-from nose import tools
-from nose.tools import *
 import stat
 
 __all__ = ['assert_instance_of', 'assert_not_instance_of',
            'assert_none', 'assert_not_none',
            'assert_match', 'assert_not_match', 'assert_mode_644',
-           'assert_mode_755'] + tools.__all__
-
+           'assert_mode_755',
+           'assert_equal', 'assert_not_equal', 'assert_raises', 'patch', 'raises',
+           'assert_true', 'assert_false']
+
+from nose.tools import (
+    assert_equal,
+    assert_not_equal,
+    assert_raises,
+    raises,
+    assert_true,
+    assert_false
+)
+
+from mock import (
+    patch
+)
 
 def assert_instance_of(expected, actual, msg=None):
     """Verify that object is an instance of expected """

git/test/performance/lib.py

@@ -1,6 +1,8 @@
 """Contains library functions"""
 import os
-from git.test.lib import *
+from git.test.lib import (
+    TestBase
+)
 from gitdb.test.lib import skip_on_travis_ci
 import shutil
 import tempfile

git/test/performance/test_commit.py

@@ -4,8 +4,8 @@
 # This module is part of GitPython and is released under
 # the BSD License: http://www.opensource.org/licenses/bsd-license.php
 
-from lib import *
-from git import *
+from .lib import TestBigRepoRW
+from git import Commit
 from gitdb import IStream
 from git.test.test_commit import assert_commit_serialization
 from cStringIO import StringIO

git/test/performance/test_odb.py

@@ -3,7 +3,7 @@
 from time import time
 import sys
 
-from lib import (
+from .lib import (
     TestBigRepoR
 )
 

git/test/performance/test_streams.py

@@ -1,20 +1,23 @@
 """Performance data streaming performance"""
-
-from git.test.lib import *
-from gitdb import *
-from gitdb.util import bin_to_hex
-
 from time import time
 import os
 import sys
 import subprocess
 
+from git.test.lib import (
+    with_rw_repo
+)
+from gitdb.util import bin_to_hex
 from gitdb.test.lib import make_memory_file
 
-from lib import (
+from .lib import (
     TestBigRepoR
 )
 
+from gitdb import (
+    LooseObjectDB,
+    IStream
+)
 
 class TestObjDBPerformance(TestBigRepoR):
 

git/test/performance/test_utils.py

@@ -2,7 +2,7 @@
 from time import time
 import sys
 
-from lib import (
+from .lib import (
     TestBigRepoR
 )
 
@@ -93,14 +93,14 @@ def test_instantiation(self):
         # tuple and tuple direct
         st = time()
         for i in xrange(ni):
-            t = (1, 2, 3, 4)
+            (1, 2, 3, 4)
         # END for each item
         elapsed = time() - st
         print >> sys.stderr, "Created %i tuples (1,2,3,4) in %f s ( %f tuples / s)" % (ni, elapsed, ni / elapsed)
 
         st = time()
         for i in xrange(ni):
-            t = tuple((1, 2, 3, 4))
+            tuple((1, 2, 3, 4))
         # END for each item
         elapsed = time() - st
         print >> sys.stderr, "Created %i tuples tuple((1,2,3,4)) in %f s ( %f tuples / s)" % (ni, elapsed, ni / elapsed)

git/test/test_actor.py

@@ -4,9 +4,8 @@
 # This module is part of GitPython and is released under
 # the BSD License: http://www.opensource.org/licenses/bsd-license.php
 
-from git.test.lib import *
-from git import *
-
+from git.test.lib import assert_equal
+from git import Actor
 
 class TestActor(object):
 

git/test/test_base.py

@@ -7,8 +7,18 @@
 import git.objects.base as base
 import os
 
-from git.test.lib import *
-from git import *
+from git.test.lib import (
+    TestBase,
+    assert_raises,
+    with_rw_repo,
+    with_rw_and_rw_remote_repo
+)
+from git import (
+    Blob,
+    Tree,
+    Commit,
+    TagObject
+)
 from git.objects.util import get_object_type_by_name
 from gitdb.util import hex_to_bin
 

git/test/test_blob.py

@@ -4,8 +4,11 @@
 # This module is part of GitPython and is released under
 # the BSD License: http://www.opensource.org/licenses/bsd-license.php
 
-from git.test.lib import *
-from git import *
+from git.test.lib import (
+    TestBase,
+    assert_equal
+)
+from git import Blob
 
 
 class TestBlob(TestBase):

git/test/test_commit.py

@@ -5,8 +5,18 @@
 # This module is part of GitPython and is released under
 # the BSD License: http://www.opensource.org/licenses/bsd-license.php
 
-from git.test.lib import *
-from git import *
+from git.test.lib import (
+    TestBase,
+    assert_equal,
+    assert_not_equal,
+    with_rw_repo,
+    fixture_path,
+    StringProcessAdapter
+)
+from git import (
+    Commit,
+    Actor,
+)
 from gitdb import IStream
 from gitdb.util import hex_to_bin
 

git/test/test_config.py

@@ -4,8 +4,13 @@
 # This module is part of GitPython and is released under
 # the BSD License: http://www.opensource.org/licenses/bsd-license.php
 
-from git.test.lib import *
-from git import *
+from git.test.lib import (
+    TestCase,
+    fixture_path
+)
+from git import (
+    GitConfigParser
+)
 import StringIO
 from copy import copy
 from ConfigParser import NoSectionError

git/test/test_db.py

@@ -3,8 +3,8 @@
 #
 # This module is part of GitPython and is released under
 # the BSD License: http://www.opensource.org/licenses/bsd-license.php
-from git.test.lib import *
-from git.db import *
+from git.test.lib import TestBase
+from git.db import GitCmdObjectDB
 from gitdb.util import bin_to_hex
 from git.exc import BadObject
 import os

git/test/test_diff.py

@@ -4,9 +4,19 @@
 # This module is part of GitPython and is released under
 # the BSD License: http://www.opensource.org/licenses/bsd-license.php
 
-from git.test.lib import *
-from git import *
-
+from git.test.lib import (
+    TestBase,
+    StringProcessAdapter,
+    fixture,
+    assert_equal,
+    assert_true,
+
+)
+
+from git import (
+    Diff,
+    DiffIndex
+)
 
 class TestDiff(TestBase):
 

git/test/test_fun.py

@@ -1,4 +1,7 @@
-from git.test.lib import *
+from git.test.lib import (
+    TestBase,
+    with_rw_repo
+)
 from git.objects.fun import (
     traverse_tree_recursive,
     traverse_trees_recursive,
@@ -78,7 +81,7 @@ def mktree(self, odb, entries):
     @with_rw_repo('0.1.6')
     def test_three_way_merge(self, rwrepo):
         def mkfile(name, sha, executable=0):
-            return (sha, S_IFREG | 0644 | executable * 0111, name)
+            return (sha, S_IFREG | 0o644 | executable * 0o111, name)
 
         def mkcommit(name, sha):
             return (sha, S_IFDIR | S_IFLNK, name)

git/test/test_index.py

@@ -4,8 +4,23 @@
 # This module is part of GitPython and is released under
 # the BSD License: http://www.opensource.org/licenses/bsd-license.php
 
-from git.test.lib import *
-from git import *
+from git.test.lib import (
+    TestBase,
+    fixture_path,
+    fixture,
+    with_rw_repo
+)
+from git import (
+    IndexFile,
+    BlobFilter,
+    UnmergedEntriesError,
+    Tree,
+    Object,
+    Diff,
+    GitCommandError,
+    CheckoutError,
+    
+)
 from gitdb.util import hex_to_bin
 import os
 import sys

git/test/test_reflog.py

@@ -1,6 +1,12 @@
-from git.test.lib import *
+from git.test.lib import (
+    TestBase,
+    fixture_path
+)
 from git.objects import IndexObject
-from git.refs import *
+from git.refs import (
+    RefLogEntry,
+    RefLog
+)
 from git.util import Actor
 
 import tempfile

git/test/test_refs.py

@@ -4,9 +4,20 @@
 # This module is part of GitPython and is released under
 # the BSD License: http://www.opensource.org/licenses/bsd-license.php
 
-from mock import *
-from git.test.lib import *
-from git import *
+from git.test.lib import (
+    TestBase,
+    with_rw_repo
+)
+from git import (
+    Reference,
+    Head,
+    TagReference,
+    RemoteReference,
+    Commit,
+    SymbolicReference,
+    GitCommandError,
+    RefLog
+)
 import git.refs as refs
 from git.util import Actor
 from git.objects.tag import TagObject

git/test/test_remote.py

@@ -4,8 +4,24 @@
 # This module is part of GitPython and is released under
 # the BSD License: http://www.opensource.org/licenses/bsd-license.php
 
-from git.test.lib import *
-from git import *
+from git.test.lib import (
+    TestBase,
+    with_rw_repo,
+    with_rw_and_rw_remote_repo
+)
+from git import (
+    RemoteProgress,
+    FetchInfo,
+    Reference,
+    SymbolicReference,
+    Head,
+    Commit,
+    PushInfo,
+    RemoteReference,
+    TagReference,
+    Remote,
+    GitCommandError
+)
 from git.util import IterableList
 import tempfile
 import shutil

git/test/test_repo.py

@@ -3,15 +3,30 @@
 #
 # This module is part of GitPython and is released under
 # the BSD License: http://www.opensource.org/licenses/bsd-license.php
-from git.test.lib import (patch,
-                          TestBase,
-                          with_rw_repo,
-                          fixture,
-                          assert_false,
-                          assert_equal,
-                          assert_true,
-                          raises)
-from git import *
+from git.test.lib import (
+    patch,
+    TestBase,
+    with_rw_repo,
+    fixture,
+    assert_false,
+    assert_equal,
+    assert_true,
+    raises
+)
+from git import (
+    InvalidGitRepositoryError,
+    Repo,
+    NoSuchPathError,
+    Head,
+    Commit,
+    Tree,
+    IndexFile,
+    Git,
+    Reference,
+    GitDB,
+    Submodule,
+    GitCmdObjectDB
+)
 from git.util import join_path_native
 from git.exc import BadObject
 from gitdb.util import bin_to_hex