Hoist import git to module level in test module

Because it's going to be necessary to express things in terms of it
in parametrization markings, in order for mypy to show the expected
errors for names that are available dynamically but deliberately
static type errors.

Author: EliahKagan

test/deprecation/test_attributes.py

@@ -4,10 +4,10 @@
 
 import pytest
 
+import git
 
-def test_cannot_get_undefined() -> None:
-    import git
 
+def test_cannot_get_undefined() -> None:
     with pytest.raises(AttributeError):
         git.foo
 
@@ -19,21 +19,16 @@ def test_cannot_import_undefined() -> None:
 
 def test_util_alias_access_resolves() -> None:
     """These resolve for now, though they're private we do not guarantee this."""
-    import git
-
     assert git.util is git.index.util
 
 
 def test_util_alias_import_resolves() -> None:
     from git import util
-    import git
 
     util is git.index.util
 
 
 def test_util_alias_access_warns() -> None:
-    import git
-
     with pytest.deprecated_call() as ctx:
         git.util
 
@@ -72,8 +67,6 @@ def test_util_alias_import_warns() -> None:
 @_parametrize_by_private_alias
 def test_private_module_alias_access_resolves(name: str, fullname: str) -> None:
     """These resolve for now, though they're private we do not guarantee this."""
-    import git
-
     assert getattr(git, name) is importlib.import_module(fullname)
 
 
@@ -85,8 +78,6 @@ def test_private_module_alias_import_resolves(name: str, fullname: str) -> None:
 
 @_parametrize_by_private_alias
 def test_private_module_alias_access_warns(name: str, fullname: str) -> None:
-    import git
-
     with pytest.deprecated_call() as ctx:
         getattr(git, name)