Actually test against the postgress db in CI

.github/workflows/ci.yml

@@ -55,7 +55,7 @@ jobs:
       - name: Prepare postgres env
         if: matrix.db == 'postgres'
         run: |
-          sed < env-template -e 's/DATABASE_URL=.*$/DATABASE_URL=postgresql:\/\/pep:pep@localhost\/memberdb/' > .env
+          sed < env-template -e 's|DATABASE_URL=.*$|DATABASE_URL=postgresql://pep:pep@localhost/memberdb|' > .env
 
       - name: "Perform migrations"
         run: poetry run flask db upgrade
@@ -66,7 +66,9 @@ jobs:
           grep 'No changes in schema detected.' migrate.log
 
       - name: tests
-        run: poetry run python -m pytest --cov member_database --cov-report=xml
+        run: |
+          export USE_CONFIG_DB=1
+          poetry run python -m pytest --cov member_database --cov-report=xml
 
       - uses: codecov/codecov-action@v1
         if: matrix.db == 'postgres'

tests/conftest.py

@@ -1,4 +1,5 @@
 import pytest
+import os
 
 from member_database import (
     init_authentication_database,
@@ -18,7 +19,14 @@ def app(db_path):
 
     from member_database import create_app
 
-    TestingConfig.SQLALCHEMY_DATABASE_URI = f"sqlite:///{db_path}"
+    # in the CI, we rely on the normal configuration db
+    # to test vs sqlite and postgres
+    # locally we create a fresh temporary sqlite db database
+    if not os.getenv("USE_CONFIG_DB", "0") == "1":
+        print("Using tmp sqlite db")
+        TestingConfig.SQLALCHEMY_DATABASE_URI = f"sqlite:///{db_path}"
+    else:
+        print("Using configured database")
     app = create_app(TestingConfig)
     return app