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

Implementing Nose2 tests #44

Merged
merged 11 commits into from
Jul 11, 2016
Merged

Implementing Nose2 tests #44

merged 11 commits into from
Jul 11, 2016

Conversation

fsteggink
Copy link
Collaborator

I've begun with the implementation of Nose2 tests. Nose2 is the successor of Nose, which is no longer maintained. The tests are far from complete. My goal is eventually to implement tests for all inputs, filters, and outputs which are used by NLExtract. Also the framework should have sufficient coverage. Currently the framework has only coverage for the "happy flow", but not for exceptions.

In order to run the tests, you need to install nose2 and mock. If you would like to get a coverage report, you also need to install cov-core. See tests/README.txt for the details.

@fsteggink
Copy link
Collaborator Author

Note that this is just a beginning. I'll continue development on this branch, until I'm satisfied.

@justb4 justb4 merged commit b93fc3a into geopython:master Jul 11, 2016
@justb4
Copy link
Member

justb4 commented Jul 11, 2016

The setup.py and Docker builds/installs are ok after this PR. Can we do something with Docker https://hub.docker.com/r/justb4/stetl/ for running tests? E.g. some infrastructure like PostGIS is required, but these are also available as Docker images that I have used, e.g. https://hub.docker.com/r/kartoza/postgis/, but ok possibly the next phase.

@fsteggink fsteggink mentioned this pull request Aug 5, 2016
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.

2 participants