Page MenuHomeSoftware Heritage

conftest: use pytest-postgresql for django related tests
ClosedPublic

Authored by douardda on Dec 19 2019, 9:59 AM.

Details

Summary

Replace the (non-working) django init hook by overloaded django_db_setup
and make this django_db_setup use pytest-postgresql so we do not depend on
external test environment (running postgresql server, tox or hand managed
pifpaf).

Also make tests run with a proper scheduler so the scheduler interaction
code is executed.

Note that this does not test for correctness in these interactions yet.

also move tests/__init__.py content in tests/conftest.py and adapt test code
accordingly.

This also ensures retries_left is set otherwise tests may fail when using
the local sheduler.

Diff Detail

Repository
rDDEP Push deposit
Branch
master
Lint
Lint Skipped
Unit
Unit Tests Skipped
Build Status
Buildable 9937
Build 14684: tox-on-jenkinsJenkins
Build 14683: arc lint + arc unit

Event Timeline

ardumont added a subscriber: ardumont.

awesome!

swh/deposit/tests/conftest.py
92

.package.

not that it matters i think.

93

because why not?

i don't know, maybe to ease readability...
but in the end, task types usually are self explanatory.

so maybe this can be dropped now that we know.

This revision is now accepted and ready to land.Dec 19 2019, 12:18 PM