VimPlug/jedi
1
0
Fork 1
mirror of https://github.com/davidhalter/jedi.git synced 2025-12-07 06:24:27 +08:00
Code Issues Actions Packages Projects Releases Wiki Activity

Improve venv_and_pths test

Browse Source 
Python is not necessarily installed in /usr/bin. Execute Python to find the
real prefix.
This commit is contained in:
micbou
2018-03-23 15:45:01 +01:00
committed by Dave Halter
parent c6635ccc55
commit 65a8ec6abc
1 changed files with 26 additions and 20 deletions
Download Patch File Download Diff File Expand all files Collapse all files

46
test/test_evaluate/test_sys_path.py
View File

@@ -2,6 +2,7 @@ import os
from glob import glob from glob import glob
import sys import sys
import shutil import shutil
import subprocess
import pytest import pytest
from jedi.evaluate import sys_path from jedi.evaluate import sys_path
@@ -36,31 +37,36 @@ def test_venv_and_pths(tmpdir, environment):
dirname = pjoin(tmpdir.dirname, 'venv') dirname = pjoin(tmpdir.dirname, 'venv')
# Ignore if it fails. It usually fails if it's not able to properly install # We cannot use the Python from tox because tox creates virtualenvs and they
# pip. However we don't need that for this test. # have different site.py files that work differently than the default ones.
executable_path = '/usr/bin/' + os.path.basename(environment._executable) # Instead, we find the real Python executable by printing the value of
if not os.path.exists(executable_path): # sys.base_prefix or sys.real_prefix if we are in a virtualenv.
# Need to not use the path in the virtualenv. Since tox creates output = subprocess.check_output([
# virtualenvs we cannot reuse them, because they have different site.py environment._executable, "-c",
# files that work differently than the default ones. "import sys; "
# Since nobody creates venv's from within virtualenvs (doesn't make "print(sys.real_prefix if hasattr(sys, 'real_prefix') else "
# sense and people are hopefully starting to avoid virtualenv now - "sys.base_prefix)"
# because it's more complicated than venv), it's the correct approach ])
# to just use the systems Python directly. prefix = output.rstrip().decode('utf8')
# This doesn't work for windows and others, but I currently don't care. if os.name == 'nt':
# Feel free to improve. executable_path = os.path.join(prefix, 'python')
pytest.skip() else:
executable_name = os.path.basename(environment._executable)
executable_path = os.path.join(prefix, 'bin', executable_name)
os.system(executable_path + ' -m venv ' + dirname) subprocess.call([executable_path, '-m', 'venv', dirname])
# We cannot find the virtualenv in some cases, because the virtualenv was bin_name = 'Scripts' if os.name == 'nt' else 'bin'
# not created correctly. virtualenv = Environment(dirname, pjoin(dirname, bin_name, 'python'))
virtualenv = Environment(dirname, pjoin(dirname, 'bin', 'python'))
CUR_DIR = os.path.dirname(__file__) CUR_DIR = os.path.dirname(__file__)
site_pkg_path = glob(pjoin(virtualenv._base_path, 'lib', 'python*', 'site-packages'))[0] site_pkg_path = pjoin(virtualenv._base_path, 'lib')
if os.name == 'nt':
site_pkg_path = pjoin(site_pkg_path, 'site-packages')
else:
site_pkg_path = glob(pjoin(site_pkg_path, 'python*', 'site-packages'))[0]
shutil.rmtree(site_pkg_path) shutil.rmtree(site_pkg_path)
shutil.copytree(pjoin(CUR_DIR, 'sample_venvs/pth_directory'), site_pkg_path) shutil.copytree(pjoin(CUR_DIR, 'sample_venvs', 'pth_directory'), site_pkg_path)
venv_paths = virtualenv.get_sys_path() venv_paths = virtualenv.get_sys_path()