chore: ensure testEventSink is not null before posting error reason

Author: SelaseKay

packages/firebase_crashlytics/firebase_crashlytics/android/src/main/java/io/flutter/plugins/firebase/crashlytics/FlutterFirebaseCrashlyticsPlugin.java

@@ -172,7 +172,9 @@ private Task<Void> recordError(final Map<String, Object> arguments) {
             Exception exception;
             if (reason != null) {
               final String crashlyticsErrorReason = "thrown " + reason;
-              mainHandler.post(() -> testEventSink.success(crashlyticsErrorReason));
+              if (testEventSink != null) {
+                mainHandler.post(() -> testEventSink.success(crashlyticsErrorReason));
+              }
               // Set a "reason" (to match iOS) to show where the exception was thrown.
               crashlytics.setCustomKey(Constants.FLUTTER_ERROR_REASON, crashlyticsErrorReason);
               exception =

packages/firebase_crashlytics/firebase_crashlytics/ios/firebase_crashlytics/Sources/firebase_crashlytics/FLTFirebaseCrashlyticsPlugin.m

@@ -139,9 +139,11 @@ - (void)recordError:(id)arguments withMethodCallResult:(FLTFirebaseMethodCallRes
   if (![reason isEqual:[NSNull null]]) {
     NSString *crashlyticsErrorReason = [NSString stringWithFormat:@"thrown %@", reason];
 
-    self.testEventSink(crashlyticsErrorReason);
+    if (self.testEventSink) {
+      self.testEventSink(crashlyticsErrorReason);
+    }
     // Log additional custom value to match Android.
-    [[FIRCrashlytics crashlytics] setCustomValue:[NSString stringWithFormat:@"thrown %@", reason]
+    [[FIRCrashlytics crashlytics] setCustomValue:crashlyticsErrorReason
                                           forKey:@"flutter_error_reason"];
     reason = [NSString stringWithFormat:@"%@. Error thrown %@.", dartExceptionMessage, reason];
   } else {

tests/integration_test/firebase_crashlytics/firebase_crashlytics_e2e_test.dart

@@ -110,8 +110,7 @@ void main() {
 
               final completer = Completer<String>();
 
-              eventStream.listen((event) {
-                 print('Received event: $event');
+              final subscription = eventStream.listen((event) {
                 completer.complete(event.toString());
               });
 
@@ -122,9 +121,10 @@ void main() {
               );
 
               final event = await completer.future;
-              expect(event, 'thrown foooo reason');
+              expect(event, 'thrown foo reason');
+              await subscription.cancel();
           },
-          skip: kIsWeb || defaultTargetPlatform == TargetPlatform.macOS || !isCI,
+          skip: kIsWeb || defaultTargetPlatform == TargetPlatform.macOS,
         );
       });