search

peritus / bumpversion

Version-bump your software with a single command
https://pypi.python.org/pypi/bumpversion
MIT License
1.5k stars 147 forks source link

bumpversion must be installed to run tests #94

Closed lgiordani closed 8 years ago

lgiordani commented 9 years ago

Some of the tests in tests.py require bumpversion to be installed (e.g. test_usage_string_fork()). Is possibile to avoid this? main() is already imported and some tests make use of it (e.g. test_usage_string()). It is not totally wrong, but creates a strange circular reference in the package.

peritus commented 9 years ago

I think tox already takes care of that the bumpversion command that is called is in fact the one that is in the directory, not a globally installed one.

I'll have to reproduce this with a clean vagrant box.

peritus commented 8 years ago

Hey so, yes, I introduced these tests after releasing a broken bumpversion where everything (and all the tests) worked, except __main__. So these tests also exist to make sure setup.py works as expected.

When hacking on bumpversion, a simple bin/pip install -e . inside a virtualenv should do the trick.

Please re-open if you disagree.