Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Don't assume pytest executable is called py.test in unit tests #65

Merged
merged 1 commit into from
Feb 3, 2018

Conversation

lpsinger
Copy link
Contributor

@lpsinger lpsinger commented Feb 3, 2018

Many distributions append suffixes to installed Python script names to differentiate between Python 2 and 3 versions. For example, Debian installs the Python 3 pytest binary as py.test-3.

In the unit tests, invoke the pytest binary using sys.exectuable to make sure that the subprocesses run with the same version of Python as the unit tests themselves.

Many distributions append suffixes to installed Python script
names to differentiate between Python 2 and 3 versions. For
example, Debian installs the Python 3 pytest binary as `py.test-3`
(see https://packages.debian.org/stretch/all/python3-pytest/filelist).

In the unit tests, invoke the pytest binary using `sys.exectuable`
to make sure that the subprocesses run with the same version of
Python as the unit tests themselves.
@astrofrog astrofrog merged commit b9890ea into matplotlib:master Feb 3, 2018
@astrofrog
Copy link
Collaborator

Thanks!

@lpsinger lpsinger deleted the test-executable-name branch February 3, 2018 22:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants