feat: remove builder

Author: ahmedAlaaInstabug

android/src/test/java/com/instabug/flutter/BugReportingApiTest.java

@@ -200,15 +200,15 @@ public void testSetCommentMinimumCharacterCount() {
     public void testSetProactiveReportingConfigurations() {
         // given
         boolean enabled = true;
-        long gapBetweekDialogs = 20;
+        long gapBetweenDialogs = 20;
         long modeDelay = 30;
 
         // when
-        api.setProactiveReportingConfigurations(enabled, gapBetweekDialogs, modeDelay);
+        api.setProactiveReportingConfigurations(enabled, gapBetweenDialogs, modeDelay);
 
         // then
         mBugReporting.verify(() -> BugReporting.setProactiveReportingConfigurations(argThat(config ->
-                config.getModalsGap() == gapBetweekDialogs &&
+                config.getModalsGap() == gapBetweenDialogs &&
                         config.getDetectionGap() == modeDelay &&
                         config.isEnabled() == enabled
         )));

example/lib/main.dart

@@ -58,11 +58,7 @@ void main() {
       };
 
       BugReporting.setProactiveReportingConfigurations(
-          ProactiveReportingConfigsBuilder()
-              .setModalDelayAfterDetection(1)
-              .setGapBetweenModals(1)
-              .isEnabled(true)
-              .build());
+          ProactiveReportingConfigs());
 
       runApp(const MyApp());
     },

lib/src/models/proactive_reporting_config.dart

@@ -5,62 +5,9 @@ class ProactiveReportingConfigs {
   final int modalDelayAfterDetection; // Time in seconds
   final bool enabled;
 
-  // Private constructor to ensure it can only be created through the builder
-  const ProactiveReportingConfigs._({
-    required this.gapBetweenModals,
-    required this.modalDelayAfterDetection,
-    required this.enabled,
+  const ProactiveReportingConfigs({
+    this.gapBetweenModals = 24,
+    this.modalDelayAfterDetection = 20,
+    this.enabled = true,
   });
 }
-
-// Builder class for ProactiveReportingConfigs
-class ProactiveReportingConfigsBuilder {
-  int gapBetweenModals = 30; // Default: 30 seconds
-  int modalDelayAfterDetection = 15; // Default: 15 seconds
-  bool enabled = true; // Default: enabled
-
-  // Logger method to handle logging
-  void _logWarning(String message) {
-    if (kDebugMode) {
-      print('Warning: $message');
-    }
-  }
-
-  /// Controls the time gap between showing 2 proactive reporting dialogs in seconds
-  ProactiveReportingConfigsBuilder setGapBetweenModals(int gap) {
-    if (gap <= 0) {
-      _logWarning(
-        'gapBetweenModals must be a positive number. Using default value of 30 seconds.',
-      );
-      return this;
-    }
-    gapBetweenModals = gap;
-    return this;
-  }
-
-  /// Controls the time gap between detecting a frustrating experience
-  ProactiveReportingConfigsBuilder setModalDelayAfterDetection(int delay) {
-    if (delay <= 0) {
-      _logWarning(
-        'modalDelayAfterDetection must be a positive number. Using default value of 15 seconds.',
-      );
-      return this;
-    }
-    modalDelayAfterDetection = delay;
-    return this;
-  }
-
-  /// Controls the state of the feature
-  ProactiveReportingConfigsBuilder isEnabled(bool value) {
-    enabled = value;
-    return this;
-  }
-
-  ProactiveReportingConfigs build() {
-    return ProactiveReportingConfigs._(
-      gapBetweenModals: gapBetweenModals,
-      modalDelayAfterDetection: modalDelayAfterDetection,
-      enabled: enabled,
-    );
-  }
-}

test/bug_reporting_test.dart

@@ -203,11 +203,10 @@ void main() {
   test('[setProactiveReportingConfigurations] should call host method',
       () async {
     await BugReporting.setProactiveReportingConfigurations(
-      ProactiveReportingConfigsBuilder()
-          .setGapBetweenModals(1)
-          .setModalDelayAfterDetection(1)
-          .isEnabled(true)
-          .build(),
+      const ProactiveReportingConfigs(
+        gapBetweenModals: 1,
+        modalDelayAfterDetection: 1,
+      ),
     );
 
     verify(mHost.setProactiveReportingConfigurations(true, 1, 1)).called(1);