Transaction test the rollback function

SimpleForest · SimpleForest · commit 72d20651fcc5 · 2025-12-05T15:04:07.000+01:00
We as well perform the necessary fix.
diff --git a/include/sqlpp23/core/database/transaction.h b/include/sqlpp23/core/database/transaction.h
@@ -83,7 +83,7 @@ class transaction_t {
 
   void rollback() {
     _finished = true;
-    _db.rollback_transaction(false);
+    _db.rollback_transaction();
   }
 };
 
diff --git a/tests/postgresql/usage/Transaction.cpp b/tests/postgresql/usage/Transaction.cpp
@@ -60,6 +60,10 @@ int Transaction(int, char*[]) {
 
       tx.commit();
       require_equal(__LINE__, db.is_transaction_active(), false);
+      auto tx2 = start_transaction(db, sqlpp::isolation_level::serializable);
+      require_equal(__LINE__, db.is_transaction_active(), true);
+      tx2.rollback();
+      require_equal(__LINE__, db.is_transaction_active(), false);
     }
 
     require_equal(__LINE__, db.get_default_isolation_level(),

Original file line number	Diff line number	Diff line change
`@@ -83,7 +83,7 @@ class transaction_t {`
`83`	`83`
`84`	`84`	`void rollback() {`
`85`	`85`	`_finished = true;`
`86`		`- _db.rollback_transaction(false);`
	`86`	`+ _db.rollback_transaction();`
`87`	`87`	`}`
`88`	`88`	`};`
`89`	`89`
Original file line number	Diff line number	Diff line change
`@@ -60,6 +60,10 @@ int Transaction(int, char*[]) {`
`60`	`60`
`61`	`61`	`tx.commit();`
`62`	`62`	`require_equal(__LINE__, db.is_transaction_active(), false);`
	`63`	`+ auto tx2 = start_transaction(db, sqlpp::isolation_level::serializable);`
	`64`	`+ require_equal(__LINE__, db.is_transaction_active(), true);`
	`65`	`+ tx2.rollback();`
	`66`	`+ require_equal(__LINE__, db.is_transaction_active(), false);`
`63`	`67`	`}`
`64`	`68`
`65`	`69`	`require_equal(__LINE__, db.get_default_isolation_level(),`