add an iOSPlatformChecker param to SentryFlutter.init

flutter/lib/src/sentry_flutter.dart

@@ -18,9 +18,15 @@ mixin SentryFlutter {
     OptionsConfiguration optionsConfiguration,
     Function callback, {
     PackageLoader packageLoader = _loadPackageInfo,
+    iOSPlatformChecker iOSPlatformChecker = _platformChecker,
   }) async {
     await Sentry.init((options) async {
-      await _initDefaultValues(options, callback, packageLoader);
+      await _initDefaultValues(
+        options,
+        callback,
+        packageLoader,
+        iOSPlatformChecker,
+      );
 
       await optionsConfiguration(options);
     });
@@ -30,6 +36,7 @@ mixin SentryFlutter {
     SentryOptions options,
     Function callback,
     PackageLoader packageLoader,
+    iOSPlatformChecker iOSPlatformChecker,
   ) async {
     // it is necessary to initialize Flutter method channels so that
     // our plugin can call into the native code.
@@ -58,7 +65,7 @@ mixin SentryFlutter {
 
     // first step is to install the native integration and set default values,
     // so we are able to capture future errors.
-    _addDefaultIntegrations(options, callback);
+    _addDefaultIntegrations(options, callback, iOSPlatformChecker);
 
     await _setReleaseAndDist(options, packageLoader);
 
@@ -103,6 +110,7 @@ mixin SentryFlutter {
   static void _addDefaultIntegrations(
     SentryOptions options,
     Function callback,
+    iOSPlatformChecker isIOS,
   ) {
     // the ordering here matters, as we'd like to first start the native integration
     // that allow us to send events to the network and then the Flutter integrations.
@@ -121,8 +129,7 @@ mixin SentryFlutter {
       options.addIntegration(isolateErrorIntegration);
     }
 
-    // TODO: make it testable/mockable
-    if (Platform.isIOS) {
+    if (isIOS()) {
       options.addIntegration(loadContextsIntegration(options, _channel));
     }
     // finally the runZonedGuarded, catch any errors in Dart code running
@@ -145,7 +152,11 @@ mixin SentryFlutter {
 
 typedef PackageLoader = Future<PackageInfo> Function();
 
+typedef iOSPlatformChecker = bool Function();
+
 /// Package info loader.
 Future<PackageInfo> _loadPackageInfo() async {
   return await PackageInfo.fromPlatform();
 }
+
+bool _platformChecker() => Platform.isIOS;

flutter/test/load_contexts_integrations_test.dart

@@ -1,3 +1,4 @@
+//@TestOn('ios')
 import 'package:flutter/services.dart';
 import 'package:flutter_test/flutter_test.dart';
 import 'package:mockito/mockito.dart';

flutter/test/sentry_flutter_test.dart

@@ -22,9 +22,20 @@ void main() {
 
   test('Flutter init for mobile will run default configurations', () async {
     await SentryFlutter.init(
-      configurationTester,
+      getConfigurationTester(false),
       callback,
       packageLoader: loadTestPackage,
+      iOSPlatformChecker: () => false,
+    );
+  });
+
+  test('Flutter init for mobile will run default configurations on ios',
+      () async {
+    await SentryFlutter.init(
+      getConfigurationTester(true),
+      callback,
+      packageLoader: loadTestPackage,
+      iOSPlatformChecker: () => true,
     );
   });
 }

flutter/test/sentry_flutter_util.dart

@@ -6,36 +6,35 @@ import 'package:sentry/sentry.dart';
 import 'package:sentry_flutter/sentry_flutter.dart';
 import 'package:sentry_flutter/src/file_system_transport.dart';
 import 'package:sentry_flutter/src/version.dart';
+
 import 'mocks.dart';
 
-FutureOr<void> configurationTester(
-  SentryOptions options, {
-  bool isWeb = false,
-}) async {
-  options.dsn = fakeDsn;
+FutureOr<void> Function(SentryOptions) getConfigurationTester(bool onIOS) =>
+    (SentryOptions options) async {
+      options.dsn = fakeDsn;
 
-  expect(kDebugMode, options.debug);
-  expect('debug', options.environment);
+      expect(kDebugMode, options.debug);
+      expect('debug', options.environment);
 
-  expect(true, options.transport is FileSystemTransport);
+      expect(true, options.transport is FileSystemTransport);
 
-  expect(
-      options.integrations
-          .where((element) => element == flutterErrorIntegration),
-      isNotEmpty);
+      expect(
+          options.integrations
+              .where((element) => element == flutterErrorIntegration),
+          isNotEmpty);
 
-  expect(
-      options.integrations
-          .where((element) => element == isolateErrorIntegration),
-      isNotEmpty);
+      expect(
+          options.integrations
+              .where((element) => element == isolateErrorIntegration),
+          isNotEmpty);
 
-  expect(4, options.integrations.length);
+      expect(onIOS ? 5 : 4, options.integrations.length);
 
-  expect(sdkName, options.sdk.name);
-  expect(sdkVersion, options.sdk.version);
-  expect('pub:sentry_flutter', options.sdk.packages.last.name);
-  expect(sdkVersion, options.sdk.packages.last.version);
+      expect(sdkName, options.sdk.name);
+      expect(sdkVersion, options.sdk.version);
+      expect('pub:sentry_flutter', options.sdk.packages.last.name);
+      expect(sdkVersion, options.sdk.packages.last.version);
 
-  expect('packageName@version+buildNumber', options.release);
-  expect('buildNumber', options.dist);
-}
+      expect('packageName@version+buildNumber', options.release);
+      expect('buildNumber', options.dist);
+    };