Use PlatformManager with startWithToken

Author: AliAbdelfattah

.gitignore

@@ -30,6 +30,7 @@
 .pub/
 pubspec.lock
 build/
+coverage/
 
 # Android related
 **/android/**/gradle-wrapper.jar

CHANGELOG.md

@@ -1,11 +1,13 @@
 ## master
 
 * Change all `void ... async` methods to `Future<void> ... async` so that callers can use `await`.
+* Fixes a crash when Instabug.start API is called on Android.
 
 ## v9.1.9 (2021-05-11)
 
 * Adds support for overriding the replies notification string values through `repliesNotificationTeamName`, `repliesNotificationReplyButton`, `repliesNotificationDismissButton`
 * Removes the use of `android:requestLegacyExternalStorage` attribute on Android
+
 ## v9.1.8 (2021-02-17)
 
 * Fixes an issue with iOS invocation events causing the welcome message not to show.

lib/Instabug.dart

@@ -6,6 +6,7 @@ import 'dart:typed_data';
 import 'dart:ui';
 
 import 'package:flutter/services.dart';
+import 'package:instabug_flutter/utils/platform_manager.dart';
 
 enum InvocationEvent {
   shake,
@@ -102,7 +103,7 @@ class Instabug {
   /// the SDK's UI.
   static Future<void> start(
       String token, List<InvocationEvent> invocationEvents) async {
-    if (Platform.isIOS) {
+    if (PlatformManager.instance.isIOS()) {
       final List<String> invocationEventsStrings =
           invocationEvents.map((e) => e.toString()).toList(growable: false);
       final List<dynamic> params = <dynamic>[token, invocationEventsStrings];

test/instabug_flutter_test.dart

@@ -16,8 +16,16 @@ import 'package:instabug_flutter/Replies.dart';
 import 'package:instabug_flutter/Surveys.dart';
 import 'package:instabug_flutter/models/crash_data.dart';
 import 'package:instabug_flutter/models/exception_data.dart';
+import 'package:instabug_flutter/utils/platform_manager.dart';
+import 'package:mockito/annotations.dart';
+import 'package:mockito/mockito.dart';
 import 'package:stack_trace/stack_trace.dart';
 
+import 'instabug_flutter_test.mocks.dart';
+
+@GenerateMocks([
+  PlatformManager,
+])
 void main() {
   WidgetsFlutterBinding.ensureInitialized();
   final List<MethodCall> log = <MethodCall>[];
@@ -32,6 +40,7 @@ void main() {
   const Map<String, String> userAttributePair = <String, String>{
     'gender': 'female'
   };
+  late MockPlatformManager mockPlatform;
 
   setUpAll(() async {
     const MethodChannel('instabug_flutter')
@@ -50,11 +59,19 @@ void main() {
     });
   });
 
+  setUp(() {
+    mockPlatform = MockPlatformManager();
+    PlatformManager.setPlatformInstance(mockPlatform);
+  });
+
   tearDown(() async {
     log.clear();
   });
 
-  test('startWithToken:invocationEvents: Test', () async {
+  test('startWithToken:invocationEvents: should be called on iOS',
+      () async {
+    when(mockPlatform.isIOS()).thenAnswer((_) => true);
+
     Instabug.start(appToken, invocationEvents);
     final List<dynamic> args = <dynamic>[
       appToken,