add tox support for charm-helpers unit tests

Bug #1257763 reported by Edward Hope-Morley
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Charm Helpers
In Progress
Wishlist
Edward Hope-Morley

Bug Description

currently, charm-helpers unit tests use nosetests which does not use a virtualenv and requires packages like python-nosetests to be installed on the host system in order to run the tests. We could avoid this by adding support for tox.

Related branches

Revision history for this message
Edward Hope-Morley (hopem) wrote :

hmm having looked into this it may not be so simple since there's a bunch of dependencies not found in pypi.

Revision history for this message
Matthew Wedgwood (mew) wrote :

Hi, Edward. My typical work-around in this sort of situation is to either a) create my own virtualenv and activate it before running tests or b) install the requirements using pip's --user option, avoiding the need for superuser privileges and isolating the scope of the module to a single user account.

If neither of those will fit your needs, I'd be interested to know your situation.

Changed in charm-helpers:
importance: Undecided → Wishlist
status: New → Won't Fix
Revision history for this message
Edward Hope-Morley (hopem) wrote :

Gonna have another stab at this. There are now quite a few deps for charmhelpers unit tests and we need (a) an aeasy to do this safely and predictably and (b) an easy way to control what versions we install by using a test-requirements.txt file that matches versions with those in the current release (Trusty).

Changed in charm-helpers:
assignee: nobody → Edward Hope-Morley (hopem)
Revision history for this message
Edward Hope-Morley (hopem) wrote :

Hmm i'm unable to set the state to In Progress. @Mathew can you unblock this please?

Changed in charm-helpers:
status: Won't Fix → In Progress
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.