aiida-core icon indicating copy to clipboard operation
aiida-core copied to clipboard

Tests: Allow running SQL backend-agnostic tests with SQLite

Open GeigerJ2 opened this issue 4 months ago • 1 comments

Some tests for the storage backend were historically implemented for PSQL as the only backend available. However, backend-agnostic tests can now also be run with SQLite. We should evaluate which are those tests, and move them to the SQLite backend tests, and remove any requires_psql markers (e.g., all of tests/storage/psql_dos/ is marked, but tests might not actually require PSQL and could be moved). See also here, originally brought up in #6829.

GeigerJ2 avatar Aug 19 '25 15:08 GeigerJ2

I'll note that the requires_psql marker is now applied automatically in tests/conftest.py::pytest_collection_modifyitems for all tests in tests/storage/psql_dos/. Dependending on how many tests remain there, perhaps a less confusing and more explicit solution is to mark all these tests explicitly (or at file level).

danielhollas avatar Aug 20 '25 17:08 danielhollas