search

inveniosoftware / invenio-records-permissions

Permissions for Invenio's records REST API.
https://invenio-records-permissions.readthedocs.io
MIT License
0 stars 25 forks source link

Remove need for running ElasticSearch in tests #3

Closed fenekku closed 4 years ago

fenekku commented 4 years ago

Tests should be fast and "external software" free i.e. self-contained. Right now, we need a running ElasticSearch container which is not great.

We can place integration tests in the invenio-app-rdm to test with a running instance, perhaps?

ppanero commented 4 years ago

Just to point out: This would affect travis not needed to wait for ES :)

fenekku commented 4 years ago

Unfortunately, it turns out this module explicitly relies on ElasticSearch (generators' query) so we have to use it in tests (and especially multi-versions in Travis). I am closing this for now.