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

Use "/usr/bin/env python" instead of "/usr/bin/python" for test_tools.py #9

Merged
merged 1 commit into from May 11, 2013

Conversation

xuhdev
Copy link
Contributor

@xuhdev xuhdev commented May 10, 2013

No description provided.

@xuhdev
Copy link
Contributor Author

xuhdev commented May 10, 2013

and I also have no idea how to run the test. python test_tools.py seems not right.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 5900475 on xuhdev:fix-test-tools into a0eeb5e on treyhunner:master.

treyhunner added a commit that referenced this pull request May 11, 2013
Use "/usr/bin/env python" instead of "/usr/bin/python" for test_tools.py
@treyhunner treyhunner merged commit a0cec0d into treyhunner:master May 11, 2013
@treyhunner
Copy link
Owner

I added an explanation for running the tests to the README file.

Running ./test_tools.py should work now, but python setup.py test is the recommended way to run the tests in a single version of Python.

@xuhdev
Copy link
Contributor Author

xuhdev commented May 11, 2013

I got this:

py2.7 runtests: commands[0]
ERROR: InvocationError: could not find executable 'coverage'
py2.7 runtests: commands[1]
ERROR: InvocationError: could not find executable 'coverage'
py2.7 runtests: commands[2]
ERROR: InvocationError: could not find executable 'coverage'

@treyhunner
Copy link
Owner

Is that from running tox with the most recent version of the code? That works for me, so I'm not sure what the problem is.

@xuhdev
Copy link
Contributor Author

xuhdev commented May 11, 2013

I just pulled the latest version and now it works. Thanks!

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

3 participants