Fix flaky exception identifier test on web (#2201)

* Fix test
getsentry · Jul 30, 2024 · 6a5a65d · 6a5a65d
1 parent c19bfb6
commit 6a5a65d
Showing 1 changed file with 44 additions and 33 deletions.
diff --git a/flutter/test/sentry_flutter_test.dart b/flutter/test/sentry_flutter_test.dart
@@ -666,39 +666,50 @@ void main() {
     await expectLater(SentryFlutter.pauseAppHangTracking(), completes);
   });
 
-  test(
-      'should add DartExceptionTypeIdentifier and FlutterExceptionTypeIdentifier by default',
-      () async {
-    SentryOptions? actualOptions;
-    await SentryFlutter.init(
-      (options) {
-        options.dsn = fakeDsn;
-        options.automatedTestMode = true;
-        actualOptions = options;
-      },
-      appRunner: appRunner,
-    );
-
-    expect(actualOptions!.exceptionTypeIdentifiers.length, 2);
-    // Flutter identifier should be first as it's more specific
-    expect(
-      actualOptions!.exceptionTypeIdentifiers.first,
-      isA<CachingExceptionTypeIdentifier>().having(
-        (c) => c.identifier,
-        'wrapped identifier',
-        isA<FlutterExceptionTypeIdentifier>(),
-      ),
-    );
-    expect(
-      actualOptions!.exceptionTypeIdentifiers[1],
-      isA<CachingExceptionTypeIdentifier>().having(
-        (c) => c.identifier,
-        'wrapped identifier',
-        isA<DartExceptionTypeIdentifier>(),
-      ),
-    );
-
-    await Sentry.close();
+  group('exception identifiers', () {
+    setUp(() async {
+      loadTestPackage();
+      await Sentry.close();
+    });
+
+    test(
+        'should add DartExceptionTypeIdentifier and FlutterExceptionTypeIdentifier by default',
+        () async {
+      SentryOptions? actualOptions;
+      await SentryFlutter.init(
+        (options) {
+          options.dsn = fakeDsn;
+          options.automatedTestMode = true;
+          actualOptions = options;
+        },
+        appRunner: appRunner,
+        platformChecker: getPlatformChecker(
+          platform: MockPlatform.android(),
+          isWeb: true,
+        ),
+      );
+
+      expect(actualOptions!.exceptionTypeIdentifiers.length, 2);
+      // Flutter identifier should be first as it's more specific
+      expect(
+        actualOptions!.exceptionTypeIdentifiers.first,
+        isA<CachingExceptionTypeIdentifier>().having(
+          (c) => c.identifier,
+          'wrapped identifier',
+          isA<FlutterExceptionTypeIdentifier>(),
+        ),
+      );
+      expect(
+        actualOptions!.exceptionTypeIdentifiers[1],
+        isA<CachingExceptionTypeIdentifier>().having(
+          (c) => c.identifier,
+          'wrapped identifier',
+          isA<DartExceptionTypeIdentifier>(),
+        ),
+      );
+
+      await Sentry.close();
+    });
   });
 }