fix: update database settings retrieval in fixtures and skip test for non-Postgres environments

Author: rboixaderg

guillotina_volto/tests/fixtures.py

@@ -50,22 +50,13 @@ def base_settings_configurator(settings):
 @pytest_asyncio.fixture(scope="function")
 async def app_client(event_loop, db, request):
     globalregistry.reset()
-    app = make_app(settings=get_db_settings_volto(request.node), loop=event_loop)
+    app = make_app(settings=get_db_settings(request.node), loop=event_loop)
     async with TestClient(app, timeout=90) as client:
         await _clear_dbs(app.app.root)
         yield app, client
     clear_task_vars()
 
 
-def get_db_settings_volto(node):
-    db_settings = get_db_settings(node)
-    print(db_settings)
-    db_settings["storages"]["db"] = {"dsn": {"storage": "postgresql", "password": "postgres", "scheme": "postgres"}}
-    db_settings["databases"]["db"]["dsn"]["password"] = "postgres"
-    db_settings["databases"]["db-custom"]["dsn"]["password"] = "postgres"
-    return db_settings
-
-
 class GuillotinaVoltoDBAsgiRequester(GuillotinaDBAsgiRequester):
     def __init__(self, client):
         self.client = client

guillotina_volto/tests/test_site.py

@@ -1,4 +1,5 @@
 import pytest
+from guillotina.tests.test_catalog import NOT_POSTGRES
 
 
 pytestmark = pytest.mark.asyncio
@@ -14,6 +15,7 @@ async def test_component(cms_requester):
     assert len(response["@components"]["actions"]["object"]) > 0
 
 
+@pytest.mark.skipif(NOT_POSTGRES, reason="Only PG")
 async def test_get_database(cms_requester):
     response, status = await cms_requester("GET", "/db/guillotina/@database")
     assert status == 200