Add support for use of requirements files #71

Merged
merged 1 commit into from Nov 9, 2016

Conversation

Projects
None yet
3 participants
Contributor

javacruft commented Nov 9, 2016

Some charms use tox to setup the virtualenv for bundletester to
run inside, including dependencies for the actual amulet tests
that get executed.

Add support to allow the requirements.txt/test-requirements.txt
files used in these configurations to also be used by bundletester
in a tests.yaml file, using a new requirements list:

requirements:
- test-requirements.txt
- requirements.txt

Each file is verified to actually exist prior to use when installing
dependencies for the tests.

Closes #65

Add support for use of requirements files
Some charms use tox to setup the virtualenv for bundletester to
run inside, including dependencies for the actual amulet tests
that get executed.

Add support to allow the requirements.txt/test-requirements.txt
files used in these configurations to also be used by bundletester
in a tests.yaml file, using a new requirements list:

   requirements:
     - test-requirements.txt
     - requirements.txt

Each file is verified to actually exist prior to use when installing
dependencies for the tests.
Owner

marcoceppi commented Nov 9, 2016

LGTM, 👍 waiting for another review to merge

@@ -117,6 +117,9 @@ A sample `tests.yaml` file::
- juju-deployer
- amulet
- requests
+ requirements:
+ - test-requirements.txt
+ - requirements.txt
@tvansteenburgh

tvansteenburgh Nov 9, 2016

Member

Please document the requirements key in the readme section below.

Member

tvansteenburgh commented Nov 9, 2016

LGTM after my one comment is addressed. Thanks @javacruft!

@tvansteenburgh tvansteenburgh merged commit e2d70cc into juju-solutions:master Nov 9, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment