fix(sqflite): recursion in openDatabase when using SentrySqfliteDatabaseFactory

CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## Unreleased
+
+### Fixes
+
+- Recursion in `openDatabase` when using `SentrySqfliteDatabaseFactory` ([#3231](https://github.com/getsentry/sentry-dart/pull/3231))
+
 ## 9.7.0-beta.2
 
 ### Features

packages/sqflite/lib/src/sentry_sqflite_database_factory.dart

@@ -1,6 +1,7 @@
 import 'package:meta/meta.dart';
 import 'package:sentry/sentry.dart';
 import 'package:sqflite/sqflite.dart';
+import 'package:sqflite/sqflite.dart' as sqflite show databaseFactory;
 // ignore: implementation_imports
 import 'package:sqflite_common/src/factory_mixin.dart';
 // ignore: implementation_imports
@@ -35,11 +36,11 @@ class SentrySqfliteDatabaseFactory with SqfliteDatabaseFactoryMixin {
   SentrySqfliteDatabaseFactory({
     DatabaseFactory? databaseFactory,
     @internal Hub? hub,
-  })  : _databaseFactory = databaseFactory,
+  })  : _databaseFactory = databaseFactory ?? sqflite.databaseFactory,
         _hub = hub ?? HubAdapter();
 
   final Hub _hub;
-  final DatabaseFactory? _databaseFactory;
+  final DatabaseFactory _databaseFactory;
 
   @override
   Future<T> invokeMethod<T>(String method, [Object? arguments]) =>
@@ -50,7 +51,7 @@ class SentrySqfliteDatabaseFactory with SqfliteDatabaseFactoryMixin {
     String path, {
     OpenDatabaseOptions? options,
   }) async {
-    final databaseFactory = _databaseFactory ?? this;
+    final databaseFactory = _databaseFactory;
 
     // ignore: invalid_use_of_internal_member
     if (!_hub.options.isTracingEnabled()) {

packages/sqflite/test/sentry_sqflite_database_factory_dart_test.dart

@@ -88,6 +88,32 @@ void main() {
     });
   });
 
+  group('openDatabase without delegate', () {
+    late Fixture fixture;
+
+    setUp(() {
+      fixture = Fixture();
+
+      when(fixture.hub.options).thenReturn(fixture.options);
+      when(fixture.hub.scope).thenReturn(fixture.scope);
+      when(fixture.hub.getSpan()).thenReturn(fixture.tracer);
+
+      // using ffi for testing on vm
+      sqfliteFfiInit();
+      databaseFactory = databaseFactoryFfi;
+    });
+
+    test('does not recurse when calling instance openDatabase', () async {
+      final wrapper = SentrySqfliteDatabaseFactory(hub: fixture.hub);
+
+      final db = await wrapper.openDatabase(inMemoryDatabasePath);
+
+      expect(db is SentryDatabase, true);
+
+      await db.close();
+    });
+  });
+
   tearDown(() {
     databaseFactory = sqfliteDatabaseFactoryDefault;
   });