diff options
| author | Jason R. Coombs <jaraco@jaraco.com> | 2017-10-12 09:43:21 +0200 |
|---|---|---|
| committer | Jason R. Coombs <jaraco@jaraco.com> | 2017-10-12 09:43:21 +0200 |
| commit | fa5f1be3b6cc13db5d6df9dfbe90542aad6bd3c9 (patch) | |
| tree | d387c67656c005f31cab96af1fd979bf2299627b /setuptools | |
| parent | 4e5c1640ac88e227aa967b8bfe04ce26325f284f (diff) | |
| download | external_python_setuptools-fa5f1be3b6cc13db5d6df9dfbe90542aad6bd3c9.tar.gz external_python_setuptools-fa5f1be3b6cc13db5d6df9dfbe90542aad6bd3c9.tar.bz2 external_python_setuptools-fa5f1be3b6cc13db5d6df9dfbe90542aad6bd3c9.zip | |
Clean up imports in test_pep517 to conform to PEP 8 standards.
Diffstat (limited to 'setuptools')
| -rw-r--r-- | setuptools/tests/test_pep517.py | 28 |
1 files changed, 14 insertions, 14 deletions
diff --git a/setuptools/tests/test_pep517.py b/setuptools/tests/test_pep517.py index df4414a1..adc155cc 100644 --- a/setuptools/tests/test_pep517.py +++ b/setuptools/tests/test_pep517.py @@ -1,18 +1,15 @@ -import pytest import os +import contextlib +import tempfile -# Only test the backend on Python 3 -# because we don't want to require -# a concurrent.futures backport for testing -pytest.importorskip('concurrent.futures') +import pytest -from contextlib import contextmanager -from importlib import import_module -from tempfile import mkdtemp -from concurrent.futures import ProcessPoolExecutor from .files import build_files from .textwrap import DALS -from . import contexts + + +futures = pytest.importorskip('concurrent.futures') +importlib = pytest.importorskip('importlib') class BuildBackendBase(object): @@ -26,7 +23,7 @@ class BuildBackend(BuildBackendBase): """PEP 517 Build Backend""" def __init__(self, *args, **kwargs): super(BuildBackend, self).__init__(*args, **kwargs) - self.pool = ProcessPoolExecutor() + self.pool = futures.ProcessPoolExecutor() def __getattr__(self, name): """Handles aribrary function invocations on the build backend.""" @@ -44,10 +41,11 @@ class BuildBackendCaller(BuildBackendBase): """Handles aribrary function invocations on the build backend.""" os.chdir(self.cwd) os.environ.update(self.env) - return getattr(import_module(self.backend_name), name)(*args, **kw) + mod = importlib.import_module(self.backend_name) + return getattr(mod, name)(*args, **kw) -@contextmanager +@contextlib.contextmanager def enter_directory(dir, val=None): original_dir = os.getcwd() os.chdir(dir) @@ -57,7 +55,7 @@ def enter_directory(dir, val=None): @pytest.fixture def build_backend(): - tmpdir = mkdtemp() + tmpdir = tempfile.mkdtemp() with enter_directory(tmpdir): setup_script = DALS(""" from setuptools import setup @@ -87,6 +85,7 @@ def test_get_requires_for_build_wheel(build_backend): assert list(sorted(b.get_requires_for_build_wheel())) == \ list(sorted(['six', 'setuptools', 'wheel'])) + def test_build_wheel(build_backend): with build_backend as b: dist_dir = os.path.abspath('pip-wheel') @@ -104,6 +103,7 @@ def test_build_sdist(build_backend): assert os.path.isfile(os.path.join(dist_dir, sdist_name)) + def test_prepare_metadata_for_build_wheel(build_backend): with build_backend as b: dist_dir = os.path.abspath('pip-dist-info') |
