tests: remove useless conditions in test files

Author: azmeuk

tests/backends/sql/postgresql.py

@@ -3,19 +3,11 @@
 from canaille.app.configuration import settings_factory
 from canaille.backends.sql.backend import SQLBackend
 
-try:
-    import pytest_postgresql  # noqa: F401
-
-    PYTEST_POSTGRESQL_AVAILABLE = True
-except ImportError:
-    PYTEST_POSTGRESQL_AVAILABLE = False
-
 
 @pytest.fixture
 def postgresql_configuration(configuration, request):
     """Configure Canaille to use PostgreSQL test database."""
-    if not PYTEST_POSTGRESQL_AVAILABLE:
-        pytest.skip("pytest-postgresql not available")
+    pytest.importorskip("pytest_postgresql")
 
     postgresql = request.getfixturevalue("postgresql")
     info = postgresql.info

tests/backends/sql/test_alembic.py

@@ -18,8 +18,7 @@ def test_migrations_with_data(app, backend, user, foo_group):
 
     # IMPORTANT: Close any open transaction before running migrations
     # Reading data above may have started an implicit transaction that holds locks
-    if backend.db_session:
-        backend.db_session.commit()
+    backend.db_session.commit()
 
     # Downgrade to first revision (preserves tables and data)
     steps_to_first = len(revisions) - 1