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

Move base tests to pytest #40

Closed
jason-ellis opened this issue Feb 9, 2018 · 1 comment
Closed

Move base tests to pytest #40

jason-ellis opened this issue Feb 9, 2018 · 1 comment

Comments

@jason-ellis
Copy link
Contributor

Tests in the base app should use pytest like the other apps do, but we need to make sure that Travis CI doesn't run them. Also, may want to keep them out of the "all" test collection by default, should only be run when flagged.

A warning is produced in create_field_test when a field does not exist (ObjectDoesNotExist). Unittest prints the warning as soon as it's encountered, while pytest has a warning summary at the end. The warning summary only shows 1 warning, even though 5 of the same are encountered. Maybe need to find a way to display all 5 or indicate in the message that this isn't the only field that was missing.

@jthomale
Copy link
Member

Done, as of merging PR #48 into master. The tests were moved into base.tests.test_databases and are run regularly via pytest.

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

No branches or pull requests

2 participants