fix: clean up PostgreSQL data between tests and rename rollback test function

Author: umbertogriffo

tests/api/endpoints/test_heroes.py

@@ -101,7 +101,7 @@ def test_delete_hero(session: Session, client_with_db: TestClient):
     assert hero_in_db is None
 
 
-def test_update_hero_rollback_on_error(
+def test_update_hero_rollback_on_commit_error(
     mocker, client_with_db: TestClient, session: Session
 ):
     """Test that updating a hero rolls back on commit error."""

tests/conftest.py

@@ -62,6 +62,10 @@ def session_fixture(request, monkeypatch) -> Session:
     with Session(engine) as session:
         yield session
 
+    # For Postgres, clean up all data between tests
+    if db_type == "postgres":
+        command.downgrade(config, "base")
+
     # Clean up
     engine.dispose()
     if path: