tests: review and refactor conftests

CERNDocumentServer / cds-videos

Access articles, reports and multimedia content in HEP

https://videos.cern.ch

GNU General Public License v2.0

15 stars 32 forks source link

tests: review and refactor conftests #494

Open egabancho opened 7 years ago

egabancho commented 7 years ago

The conftest.py has grown a lot and it needs some refactoring, in particular to identify all the fixtures that are quite similar to merge them when possible, and to put more realistic metadata in the example tests.

egabancho commented 7 years ago

There are other things that are wrong in the conftest.py:

app vs api_app: fixtures like records_rest_app or deposit_rest should use the api_app instead.

egabancho commented 7 years ago

We should also try to reduce the time spend in the tests keeping the coverage ... 30 minutes is too much ❗️

egabancho commented 6 years ago

Probably we want to use this instead of our own fixtures https://github.com/inveniosoftware/pytest-invenio