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

Improve testing #5

Open
bcongdon opened this issue Oct 4, 2016 · 1 comment
Open

Improve testing #5

bcongdon opened this issue Oct 4, 2016 · 1 comment
Labels
Milestone

Comments

@bcongdon
Copy link
Owner

bcongdon commented Oct 4, 2016

Next big push is to get the actual tests working.
#3 reduced the dependencies between the app and trackers, so it should be possible to unit test the trackers without having to load / startup the app. (If it isn't, then that's something to do too)

Not sure how much testing is necessary for an external api driven project like this, but something better than running the updateall cli command would be good. 👍

@bcongdon
Copy link
Owner Author

bcongdon commented Nov 1, 2016

I think it might be acceptable to assume that the data coming from the API's is good, and focus on the database side of the transaction.

Should use mocked db and test data in the test cases.

Not worth the time to test the API libraries for 'correctness'

@bcongdon bcongdon added this to the 0.1.0 milestone Nov 1, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant