Add test coverage for BugReporting

Author: AliAbdelfattah

lib/BugReporting.dart

@@ -1,10 +1,10 @@
 // ignore_for_file: avoid_classes_with_only_static_members
 
 import 'dart:async';
-import 'dart:io' show Platform;
 
 import 'package:flutter/services.dart';
 import 'package:instabug_flutter/Instabug.dart';
+import 'package:instabug_flutter/utils/platform_manager.dart';
 
 enum InvocationOption {
   commentFieldRequired,
@@ -183,7 +183,7 @@ class BugReporting {
   /// [iPhoneShakingThreshold] iPhoneShakingThreshold double
   static Future<void> setShakingThresholdForiPhone(
       double iPhoneShakingThreshold) async {
-    if (Platform.isIOS) {
+    if (PlatformManager.instance.isIOS()) {
       final List<dynamic> params = <dynamic>[iPhoneShakingThreshold];
       await _channel.invokeMethod<Object>(
           'setShakingThresholdForiPhone:', params);
@@ -195,7 +195,7 @@ class BugReporting {
   /// [iPadShakingThreshold] iPhoneShakingThreshold double
   static Future<void> setShakingThresholdForiPad(
       double iPadShakingThreshold) async {
-    if (Platform.isIOS) {
+    if (PlatformManager.instance.isIOS()) {
       final List<dynamic> params = <dynamic>[iPadShakingThreshold];
       await _channel.invokeMethod<Object>(
           'setShakingThresholdForiPad:', params);
@@ -209,7 +209,7 @@ class BugReporting {
   /// [androidThreshold] iPhoneShakingThreshold int
   static Future<void> setShakingThresholdForAndroid(
       int androidThreshold) async {
-    if (Platform.isAndroid) {
+    if (PlatformManager.instance.isAndroid()) {
       final List<dynamic> params = <dynamic>[androidThreshold];
       await _channel.invokeMethod<Object>(
           'setShakingThresholdForAndroid:', params);

test/instabug_flutter_test.dart

@@ -68,8 +68,7 @@ void main() {
     log.clear();
   });
 
-  test('startWithToken:invocationEvents: should be called on iOS',
-      () async {
+  test('startWithToken:invocationEvents: should be called on iOS', () async {
     when(mockPlatform.isIOS()).thenAnswer((_) => true);
 
     Instabug.start(appToken, invocationEvents);
@@ -433,6 +432,51 @@ void main() {
     ]);
   });
 
+  test('setShakingThresholdForiPhone: Test', () async {
+    const iPhoneShakingThreshold = 1.6;
+    final List<dynamic> args = <dynamic>[iPhoneShakingThreshold];
+
+    when(mockPlatform.isIOS()).thenAnswer((_) => true);
+
+    BugReporting.setShakingThresholdForiPhone(iPhoneShakingThreshold);
+    expect(log, <Matcher>[
+      isMethodCall(
+        'setShakingThresholdForiPhone:',
+        arguments: args,
+      )
+    ]);
+  });
+
+  test('setShakingThresholdForiPad: Test', () async {
+    const iPadShakingThreshold = 1.6;
+    final List<dynamic> args = <dynamic>[iPadShakingThreshold];
+
+    when(mockPlatform.isIOS()).thenAnswer((_) => true);
+
+    BugReporting.setShakingThresholdForiPad(iPadShakingThreshold);
+    expect(log, <Matcher>[
+      isMethodCall(
+        'setShakingThresholdForiPad:',
+        arguments: args,
+      )
+    ]);
+  });
+
+  test('setShakingThresholdForAndroid: Test', () async {
+    const androidThreshold = 1000;
+    final List<dynamic> args = <dynamic>[androidThreshold];
+
+    when(mockPlatform.isAndroid()).thenAnswer((_) => true);
+
+    BugReporting.setShakingThresholdForAndroid(androidThreshold);
+    expect(log, <Matcher>[
+      isMethodCall(
+        'setShakingThresholdForAndroid:',
+        arguments: args,
+      )
+    ]);
+  });
+
   test('setOnInvokeCallback Test', () async {
     BugReporting.setOnInvokeCallback(() => () {});
     expect(log, <Matcher>[
@@ -467,6 +511,17 @@ void main() {
     ]);
   });
 
+  test('setInvocationEvents Test', () async {
+    BugReporting.setInvocationEvents(null);
+    final List<dynamic> args = <dynamic>[<String>[]];
+    expect(log, <Matcher>[
+      isMethodCall(
+        'setInvocationEvents:',
+        arguments: args,
+      )
+    ]);
+  });
+
   test(
       'setEnabledAttachmentTypes:extraScreenShot:galleryImage:screenRecording: Test',
       () async {
@@ -521,6 +576,17 @@ void main() {
     ]);
   });
 
+  test('setInvocationOptions Test: empty', () async {
+    BugReporting.setInvocationOptions(null);
+    final List<dynamic> args = <dynamic>[<String>[]];
+    expect(log, <Matcher>[
+      isMethodCall(
+        'setInvocationOptions:',
+        arguments: args,
+      )
+    ]);
+  });
+
   test('showBugReportingWithReportTypeAndOptions:options Test', () async {
     BugReporting.show(
         ReportType.bug, <InvocationOption>[InvocationOption.emailFieldHidden]);
@@ -536,6 +602,18 @@ void main() {
     ]);
   });
 
+  test('showBugReportingWithReportTypeAndOptions:options Test: empty options',
+      () async {
+    BugReporting.show(ReportType.bug, null);
+    final List<dynamic> args = <dynamic>[ReportType.bug.toString(), <String>[]];
+    expect(log, <Matcher>[
+      isMethodCall(
+        'showBugReportingWithReportTypeAndOptions:options:',
+        arguments: args,
+      )
+    ]);
+  });
+
   test('setSurveysEnabled: Test', () async {
     const isEnabled = false;
     final List<dynamic> args = <dynamic>[isEnabled];