Commit 41deeb76 authored by Jason R. Coombs's avatar Jason R. Coombs

Bypass pytest-virtualenv due to bugs in 'run' command. Instead, use...

Bypass pytest-virtualenv due to bugs in 'run' command. Instead, use jaraco.envs.VirtualEnv and build a bespoke fixture with a run command.
parent 6281bd09
...@@ -72,6 +72,7 @@ tests = ...@@ -72,6 +72,7 @@ tests =
paver; python_version>="3.6" paver; python_version>="3.6"
futures; python_version=="2.7" futures; python_version=="2.7"
pip>=19.1 # For proper file:// URLs support. pip>=19.1 # For proper file:// URLs support.
jaraco.envs
docs = docs =
# Keep these in sync with docs/requirements.txt # Keep these in sync with docs/requirements.txt
......
...@@ -10,3 +10,4 @@ pytest-cov>=2.5.1 ...@@ -10,3 +10,4 @@ pytest-cov>=2.5.1
paver; python_version>="3.6" paver; python_version>="3.6"
futures; python_version=="2.7" futures; python_version=="2.7"
pip>=19.1 # For proper file:// URLs support. pip>=19.1 # For proper file:// URLs support.
jaraco.envs
import os import os
import sys import sys
import functools import functools
import subprocess
import pytest import pytest
import jaraco.envs
import path
class VirtualEnv(jaraco.envs.VirtualEnv):
name = '.env'
def run(self, cmd, *args, **kwargs):
cmd = [self.exe(cmd[0])] + cmd[1:]
return subprocess.check_output(cmd, *args, cwd=self.root, **kwargs)
@pytest.fixture
def venv(tmpdir):
env = VirtualEnv()
env.root = path.Path(tmpdir)
env.req = os.getcwd()
return env.create()
def popen_text(call): def popen_text(call):
...@@ -13,36 +32,29 @@ def popen_text(call): ...@@ -13,36 +32,29 @@ def popen_text(call):
if sys.version_info < (3, 7) else functools.partial(call, text=True) if sys.version_info < (3, 7) else functools.partial(call, text=True)
@pytest.fixture def find_distutils(venv, imports='distutils', **kwargs):
def env(virtualenv):
virtualenv.run(['pip', 'uninstall', '-y', 'setuptools'])
virtualenv.run(['pip', 'install', os.getcwd()])
return virtualenv
def find_distutils(env, imports='distutils'):
py_cmd = 'import {imports}; print(distutils.__file__)'.format(**locals()) py_cmd = 'import {imports}; print(distutils.__file__)'.format(**locals())
cmd = ['python', '-c', py_cmd] cmd = ['python', '-c', py_cmd]
return popen_text(env.run)(cmd, capture=True) return popen_text(venv.run)(cmd, **kwargs)
def test_distutils_stdlib(env): def test_distutils_stdlib(venv):
""" """
Ensure stdlib distutils is used when appropriate. Ensure stdlib distutils is used when appropriate.
""" """
assert '.env' not in find_distutils(env).split(os.sep) assert venv.name not in find_distutils(venv, env=dict()).split(os.sep)
def test_distutils_local_with_setuptools(env): def test_distutils_local_with_setuptools(venv):
""" """
Ensure local distutils is used when appropriate. Ensure local distutils is used when appropriate.
""" """
env.env.update(SETUPTOOLS_USE_DISTUTILS='local') env = dict(SETUPTOOLS_USE_DISTUTILS='local')
loc = find_distutils(env, imports='setuptools, distutils') loc = find_distutils(venv, imports='setuptools, distutils', env=env)
assert '.env' in loc.split(os.sep) assert venv.name in loc.split(os.sep)
@pytest.mark.xfail(reason="#2259") @pytest.mark.xfail(reason="#2259")
def test_distutils_local(env): def test_distutils_local(venv):
env.env.update(SETUPTOOLS_USE_DISTUTILS='local') env = dict(SETUPTOOLS_USE_DISTUTILS='local')
assert '.env' in find_distutils(env).split(os.sep) assert venv.name in find_distutils(venv, env=env).split(os.sep)
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment