Skip to content
This repository has been archived by the owner on Feb 22, 2020. It is now read-only.

Unit tests for Python backend code #275

Open
isms opened this issue Jan 5, 2018 · 3 comments
Open

Unit tests for Python backend code #275

isms opened this issue Jan 5, 2018 · 3 comments

Comments

@isms
Copy link
Contributor

isms commented Jan 5, 2018

Go through and add unit tests for Django/DRF or Python functions in the prediction service. (This is correctness testing separate from ML evaluation.)

Tests should not unduly slow down the build.


Points will be awarded continuously through the end of the competition -- this issue will not close.

@WGierke
Copy link
Contributor

WGierke commented Jan 5, 2018

Would it be useful to add integrations such as CodeCov to continuously monitor whether a test really adds more coverage / a feature reduces the coverage?

@isms
Copy link
Contributor Author

isms commented Jan 6, 2018

Good thought, we have used and enjoyed codecov.io.

Maybe we can start with just a report in the test pipeline, and staff can look into setting up the integration if time permits.

@WGierke
Copy link
Contributor

WGierke commented Jan 7, 2018

Does something like #282 make sense? It generates a coverage report locally which could then also be persisted by Travis (like this one).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants