fix: abortTransaction does not call provided execute function when parent span is null (#3062)

* Call execute when parent span is null

* Add tests to check calling execute with parent span or not

* Rewrite tests that tests the actual user scenario

* Append this PR to CHANGELOG.md

* Update CHANGELOG.md

---------

Co-authored-by: Giancarlo Buenaflor <[email protected]>
Co-authored-by: Giancarlo Buenaflor <[email protected]>

Author: ichiko

CHANGELOG.md

@@ -7,6 +7,7 @@
 - Span ids not re-generating for headers created from scope ([#3051](https://github.com/getsentry/sentry-dart/pull/3051))
 - `ScreenshotIntegration` not being added for web ([#3055](https://github.com/getsentry/sentry-dart/pull/3055))
 - `PropagationContext` not being set when `Scope` is cloned resulting in different trace ids when using `withScope` ([#3069](https://github.com/getsentry/sentry-dart/pull/3069))
+- Drift transaction rollback not executed when parent span is null ([#3062](https://github.com/getsentry/sentry-dart/pull/3062))
 
 ### Enhancements
 

drift/lib/src/sentry_span_helper.dart

@@ -151,7 +151,7 @@ class SentrySpanHelper {
         'Active Sentry transaction does not exist, could not finish span for Drift operation: Abort Transaction',
         logger: loggerName,
       );
-      return Future<T>.value();
+      return execute();
     }
 
     try {

drift/test/sentry_drift_test.dart

@@ -603,6 +603,52 @@ void main() {
       );
     });
 
+    test(
+        'transaction is rolled back within Sentry transaction, added aborted span',
+        () async {
+      final sut = fixture.getSut();
+      final db = AppDatabase(NativeDatabase.memory().interceptWith(sut));
+
+      final tx = _startTransaction();
+
+      // pre-condition: table empty
+      expect(await db.select(db.todoItems).get(), isEmpty);
+
+      // run a transaction that is forced to fail -> should be rolled back
+      await expectLater(
+        () => db.transaction(() async {
+          await _insertRow(db, withError: true);
+        }),
+        throwsA(isA<Exception>()),
+      );
+
+      final abortedSpans =
+          tx.children.where((child) => child.status == SpanStatus.aborted());
+      expect(abortedSpans.length, 1);
+
+      // if rollback happened the row must be absent
+      expect(await db.select(db.todoItems).get(), isEmpty);
+    });
+
+    test('transaction is rolled back without Sentry transaction', () async {
+      final sut = fixture.getSut();
+      final db = AppDatabase(NativeDatabase.memory().interceptWith(sut));
+
+      // pre-condition: table empty
+      expect(await db.select(db.todoItems).get(), isEmpty);
+
+      // run a transaction that is forced to fail -> should be rolled back
+      await expectLater(
+        () => db.transaction(() async {
+          await _insertRow(db, withError: true);
+        }),
+        throwsA(isA<Exception>()),
+      );
+
+      // if rollback happened the row must be absent
+      expect(await db.select(db.todoItems).get(), isEmpty);
+    });
+
     test('batch does not add span for failed operations', () async {
       final sut = fixture.getSut();
       final db = AppDatabase(NativeDatabase.memory().interceptWith(sut));