Skip to content
This repository has been archived by the owner on Dec 4, 2018. It is now read-only.

Add tox for running unit tests #844

Merged
merged 1 commit into from
Jan 4, 2018
Merged

Add tox for running unit tests #844

merged 1 commit into from
Jan 4, 2018

Conversation

chosak
Copy link
Member

@chosak chosak commented Dec 20, 2017

This change adds support for tox (http://tox.readthedocs.io/) for running Python unit tests in a parameterized way.

Python unit tests can be run from the command line using tox. For now there is only a single environment (Python 2.7 and Django 1.8), but this change will make it easier to add testing against other Python/Django versions in future.

Some minor changes/fixes were made to a few unit tests to make this work properly, specifically some tests were assuming an order for dictionary keys, which is not dependable.

This change also modifies how TravisCI runs the unit tests so that it invokes tox instead of running the test command directly.

It also removes the invocation of browser tests from Travis (using Sauce Labs) as these have been failing for quite some time.

This change adds support for tox (http://tox.readthedocs.io/) for
running Python unit tests in a parameterized way.

Python unit tests can be run from the command line using `tox`. For now
there is only a single environment (Python 2.7 and Django 1.8), but this
change will make it easier to add testing against other Python/Django
versions in future.

Some minor changes/fixes were made to a few unit tests to make this work
properly, specifically some tests were assuming an order for dictionary
keys, which is not dependable.

This change also modifies how TravisCI runs the unit tests so that it
invokes tox instead of running the test command directly.

It also removes the invocation of browser tests from Travis (using Sauce
Labs) as these have been failing for quite some time.
@chosak chosak merged commit 1ca46ca into master Jan 4, 2018
@chosak chosak deleted the tox-testing branch January 4, 2018 14:13
chosak added a commit that referenced this pull request Jan 4, 2018
PR #844 (1ca46ca) broke packaging of
this project by omitting subdirectories of the regulations app. This
change reverts that and ensures that all code is packaged properly.
@chosak chosak mentioned this pull request Jan 4, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants