Add ConnectivityIntegration for web

flutter/lib/src/integrations/connectivity_integration.dart

@@ -0,0 +1,57 @@
+import 'dart:async';
+
+import 'package:connectivity_plus/connectivity_plus.dart';
+import 'package:meta/meta.dart';
+import '../../sentry_flutter.dart';
+
+class ConnectivityIntegration extends Integration<SentryFlutterOptions> {
+  Connectivity connectivity = Connectivity();
+  Hub? _hub;
+  StreamSubscription<ConnectivityResult>? _subscription;
+
+  @override
+  void call(Hub hub, SentryFlutterOptions options) {
+    _hub = hub;
+    _subscription = connectivity.onConnectivityChanged.listen(addBreadcrumb);
+
+    options.sdk.addIntegration('connectivityIntegration');
+  }
+
+  @override
+  void close() {
+    _hub = null;
+    _subscription?.cancel();
+    _subscription = null;
+  }
+
+  @internal
+  @visibleForTesting
+  void addBreadcrumb(ConnectivityResult result) {
+    _hub?.addBreadcrumb(
+      Breadcrumb(
+          category: 'device.connectivity',
+          level: SentryLevel.info,
+          type: 'connectivity',
+          data: {'connectivity': result.toSentryConnectivity()}),
+    );
+  }
+}
+
+extension on ConnectivityResult {
+  String toSentryConnectivity() {
+    switch (this) {
+      case ConnectivityResult.bluetooth:
+      case ConnectivityResult.vpn:
+      case ConnectivityResult.wifi:
+        return 'wifi';
+      case ConnectivityResult.ethernet:
+        return 'ethernet';
+      case ConnectivityResult.mobile:
+        return 'cellular';
+      case ConnectivityResult.none:
+        return 'none';
+      case ConnectivityResult.other:
+        return 'other';
+    }
+  }
+}

flutter/lib/src/sentry_flutter.dart

@@ -9,6 +9,7 @@ import '../sentry_flutter.dart';
 import 'event_processor/android_platform_exception_event_processor.dart';
 import 'event_processor/flutter_exception_event_processor.dart';
 import 'event_processor/platform_exception_event_processor.dart';
+import 'integrations/connectivity_integration.dart';
 import 'integrations/screenshot_integration.dart';
 import 'native/factory.dart';
 import 'native/native_scope_observer.dart';
@@ -171,6 +172,10 @@ mixin SentryFlutter {
       integrations.add(ScreenshotIntegration());
     }
 
+    if (!platform.isIOS && !platform.isMacOS || !platform.isAndroid) {
+      integrations.add(ConnectivityIntegration());
+    }
+
     // works with Skia, CanvasKit and HTML renderer
     integrations.add(SentryViewHierarchyIntegration());
 

flutter/pubspec.yaml

@@ -19,6 +19,7 @@ dependencies:
   package_info_plus: '>=1.0.0 <=5.0.1'
   meta: ^1.3.0
   ffi: ^2.0.0
+  connectivity_plus: ^5.0.2
 
 dev_dependencies:
   build_runner: ^2.4.2

flutter/test/integrations/connectivity_integration_test.dart

@@ -0,0 +1,149 @@
+import 'package:connectivity_plus/connectivity_plus.dart';
+import 'package:flutter/widgets.dart';
+import 'package:flutter_test/flutter_test.dart';
+import 'package:mockito/mockito.dart';
+import 'package:sentry/sentry.dart';
+import 'package:sentry_flutter/src/integrations/connectivity_integration.dart';
+import 'package:sentry_flutter/src/sentry_flutter_options.dart';
+
+import '../mocks.dart';
+import '../mocks.mocks.dart';
+
+void main() {
+  WidgetsFlutterBinding.ensureInitialized();
+
+  late Fixture fixture;
+
+  setUp(() {
+    fixture = Fixture();
+  });
+
+  verifyBreadcrumb(Breadcrumb crumb, String connectivityData) {
+    expect(crumb.category, 'device.connectivity');
+    expect(crumb.type, 'connectivity');
+    expect(crumb.level, SentryLevel.info);
+    expect(crumb.data?['connectivity'], connectivityData);
+  }
+
+  test('adds integration', () {
+    final sut = fixture.getSut();
+    sut(fixture.hub, fixture.options);
+
+    expect(fixture.options.sdk.integrations.contains('connectivityIntegration'),
+        true);
+  });
+
+  test(
+      '$ConnectivityIntegration: connectivity changed `bluetooth` adds `wifi` breadcrumb',
+      () {
+    final integration = fixture.getSut();
+    integration.call(fixture.hub, fixture.options);
+
+    integration.addBreadcrumb(ConnectivityResult.bluetooth);
+
+    final crumb = verify(
+      fixture.hub.addBreadcrumb(captureAny),
+    ).captured.first as Breadcrumb;
+
+    verifyBreadcrumb(crumb, 'wifi');
+  });
+
+  test(
+      '$ConnectivityIntegration: connectivity changed `wifi` adds `wifi` breadcrumb',
+      () {
+    final integration = fixture.getSut();
+    integration.call(fixture.hub, fixture.options);
+
+    integration.addBreadcrumb(ConnectivityResult.wifi);
+
+    final crumb = verify(
+      fixture.hub.addBreadcrumb(captureAny),
+    ).captured.first as Breadcrumb;
+
+    verifyBreadcrumb(crumb, 'wifi');
+  });
+
+  test(
+      '$ConnectivityIntegration: connectivity changed `vpn` adds `vpn` breadcrumb',
+      () {
+    final integration = fixture.getSut();
+    integration.call(fixture.hub, fixture.options);
+
+    integration.addBreadcrumb(ConnectivityResult.vpn);
+
+    final crumb = verify(
+      fixture.hub.addBreadcrumb(captureAny),
+    ).captured.first as Breadcrumb;
+
+    verifyBreadcrumb(crumb, 'wifi');
+  });
+
+  test(
+      '$ConnectivityIntegration: connectivity changed `ethernet` adds `ethernet` breadcrumb',
+      () {
+    final integration = fixture.getSut();
+    integration.call(fixture.hub, fixture.options);
+
+    integration.addBreadcrumb(ConnectivityResult.ethernet);
+
+    final crumb = verify(
+      fixture.hub.addBreadcrumb(captureAny),
+    ).captured.first as Breadcrumb;
+
+    verifyBreadcrumb(crumb, 'ethernet');
+  });
+
+  test(
+      '$ConnectivityIntegration: connectivity changed `mobile` adds  `cellular` breadcrumb',
+      () {
+    final integration = fixture.getSut();
+    integration.call(fixture.hub, fixture.options);
+
+    integration.addBreadcrumb(ConnectivityResult.mobile);
+
+    final crumb = verify(
+      fixture.hub.addBreadcrumb(captureAny),
+    ).captured.first as Breadcrumb;
+
+    verifyBreadcrumb(crumb, 'cellular');
+  });
+
+  test(
+      '$ConnectivityIntegration: connectivity changed `other` adds `other` breadcrumb',
+      () {
+    final integration = fixture.getSut();
+    integration.call(fixture.hub, fixture.options);
+
+    integration.addBreadcrumb(ConnectivityResult.other);
+
+    final crumb = verify(
+      fixture.hub.addBreadcrumb(captureAny),
+    ).captured.first as Breadcrumb;
+
+    verifyBreadcrumb(crumb, 'other');
+  });
+
+  test(
+      '$ConnectivityIntegration: connectivity changed `none` adds `none` breadcrumb',
+      () {
+    final integration = fixture.getSut();
+    integration.call(fixture.hub, fixture.options);
+
+    integration.addBreadcrumb(ConnectivityResult.none);
+
+    final crumb = verify(
+      fixture.hub.addBreadcrumb(captureAny),
+    ).captured.first as Breadcrumb;
+
+    verifyBreadcrumb(crumb, 'none');
+  });
+}
+
+class Fixture {
+  final hub = MockHub();
+  final options = SentryFlutterOptions(dsn: fakeDsn);
+
+  ConnectivityIntegration getSut() {
+    return ConnectivityIntegration();
+  }
+}

flutter/test/sentry_flutter_options_test.dart

@@ -32,7 +32,7 @@ void main() {
       expect(options.enableAutoNativeBreadcrumbs, isFalse);
     });
 
-    testWidgets('useFlutterBreadcrumbTracking', (WidgetTester tester) async {
+    testWidgets('useNativeBreadcrumbTracking', (WidgetTester tester) async {
       final options = SentryFlutterOptions();
       options.useNativeBreadcrumbTracking();
 

flutter/test/sentry_flutter_test.dart

@@ -3,6 +3,7 @@
 import 'package:flutter_test/flutter_test.dart';
 import 'package:package_info_plus/package_info_plus.dart';
 import 'package:sentry_flutter/sentry_flutter.dart';
+import 'package:sentry_flutter/src/integrations/connectivity_integration.dart';
 import 'package:sentry_flutter/src/integrations/integrations.dart';
 import 'package:sentry_flutter/src/integrations/screenshot_integration.dart';
 import 'package:sentry_flutter/src/profiling.dart';
@@ -43,6 +44,11 @@ final nativeIntegrations = [
   NativeSdkIntegration,
 ];
 
+// These should be added to every platform except Android & iOS/macOS.
+final nonMobileIntegrations = [
+  ConnectivityIntegration,
+];
+
 void main() {
   TestWidgetsFlutterBinding.ensureInitialized();
 
@@ -88,7 +94,11 @@ void main() {
             ...platformAgnosticIntegrations,
             ...nonWebIntegrations,
           ],
-          shouldNotHaveIntegrations: iOsAndMacOsIntegrations);
+          shouldNotHaveIntegrations: [
+            ...iOsAndMacOsIntegrations,
+            ...nonWebIntegrations,
+          ],
+      );
 
       integrations
           .indexWhere((element) => element is WidgetsFlutterBindingIntegration);
@@ -138,7 +148,10 @@ void main() {
           ...platformAgnosticIntegrations,
           ...nonWebIntegrations,
         ],
-        shouldNotHaveIntegrations: androidIntegrations,
+        shouldNotHaveIntegrations: [
+          ...androidIntegrations,
+          ...nonWebIntegrations,
+        ],
       );
 
       testBefore(
@@ -187,7 +200,10 @@ void main() {
           ...platformAgnosticIntegrations,
           ...nonWebIntegrations,
         ],
-        shouldNotHaveIntegrations: androidIntegrations,
+        shouldNotHaveIntegrations: [
+          ...androidIntegrations,
+          ...nonWebIntegrations,
+        ]
       );
 
       testBefore(
@@ -234,6 +250,7 @@ void main() {
         shouldHaveIntegrations: [
           ...platformAgnosticIntegrations,
           ...nonWebIntegrations,
+          ...nonMobileIntegrations,
         ],
         shouldNotHaveIntegrations: [
           ...androidIntegrations,
@@ -285,6 +302,7 @@ void main() {
         shouldHaveIntegrations: [
           ...platformAgnosticIntegrations,
           ...nonWebIntegrations,
+          ...nonMobileIntegrations,
         ],
         shouldNotHaveIntegrations: [
           ...androidIntegrations,
@@ -336,7 +354,10 @@ void main() {
 
       testConfiguration(
         integrations: integrations,
-        shouldHaveIntegrations: platformAgnosticIntegrations,
+        shouldHaveIntegrations: [
+          ...platformAgnosticIntegrations,
+          ...nonMobileIntegrations,
+        ],
         shouldNotHaveIntegrations: [
           ...androidIntegrations,
           ...iOsAndMacOsIntegrations,