Revert "Support setSdkDebugLogsLevel on Android" (#317)

This reverts commit 170b8d2.

Author: TheBuggedYRN

CHANGELOG.md

@@ -6,7 +6,6 @@
 * Fixes APM network logging on Android
 * Fixes a NullPointerException when overriding a string key that doesn't exist on Android
 * Removes redundant native logs
-* Deprecates Instabug.setDebugEnabled and APM.setLogLevel APIs in favour of Instabug.setSdkDebugLogsLevel, which controls the verbosity of SDK logs on both platforms
 
 ## 11.5.0 (2022-11-24)
 

android/src/main/java/com/instabug/flutter/modules/InstabugApi.java

@@ -247,10 +247,14 @@ public void run() {
         );
     }
 
+    @Override
+    public void setDebugEnabled(@NonNull Boolean enabled) {
+        Instabug.setDebugEnabled(enabled);
+    }
+
     @Override
     public void setSdkDebugLogsLevel(@NonNull String level) {
-        final int sdkLogLevel = ArgsRegistry.sdkLogLevels.get(level);
-        Instabug.setSdkDebugLogsLevel(sdkLogLevel);
+        // iOS Only
     }
 
     @Override

android/src/main/java/com/instabug/flutter/util/ArgsRegistry.java

@@ -201,11 +201,4 @@ public T get(Object key) {
         put("CustomTextPlaceHolderKey.messagesNotificationAndOthers", Key.CHATS_MULTIPLE_MESSAGE_NOTIFICATION);
         put("CustomTextPlaceHolderKey.insufficientContentMessage", Key.COMMENT_FIELD_INSUFFICIENT_CONTENT);
     }};
-
-    public static final ArgsMap<Integer> sdkLogLevels = new ArgsMap<Integer>() {{
-        put("IBGSDKDebugLogsLevel.none", com.instabug.library.LogLevel.NONE);
-        put("IBGSDKDebugLogsLevel.error", com.instabug.library.LogLevel.ERROR);
-        put("IBGSDKDebugLogsLevel.debug", com.instabug.library.LogLevel.DEBUG);
-        put("IBGSDKDebugLogsLevel.verbose", com.instabug.library.LogLevel.VERBOSE);
-    }};
 }

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

@@ -369,13 +369,14 @@ public void testGetUserAttributes() {
         mInstabug.verify(Instabug::getAllUserAttributes);
     }
 
+    @SuppressWarnings("deprecation")
     @Test
-    public void testSetSdkDebugLogsLevel() {
-        String level = "IBGSDKDebugLogsLevel.debug";
+    public void testSetDebugEnabled() {
+        boolean isEnabled = true;
 
-        api.setSdkDebugLogsLevel(level);
+        api.setDebugEnabled(isEnabled);
 
-        mInstabug.verify(() -> Instabug.setSdkDebugLogsLevel(com.instabug.library.LogLevel.DEBUG));
+        mInstabug.verify(() -> Instabug.setDebugEnabled(isEnabled));
     }
 
     @Test

ios/Classes/Modules/InstabugApi.m

@@ -141,6 +141,10 @@ - (void)getUserAttributesWithCompletion:(nonnull void (^)(NSDictionary<NSString
     completion(Instabug.userAttributes, nil);
 }
 
+- (void)setDebugEnabledEnabled:(NSNumber *)enabled error:(FlutterError *_Nullable *_Nonnull)error {
+    // Android Only
+}
+
 - (void)setSdkDebugLogsLevelLevel:(NSString *)level error:(FlutterError *_Nullable *_Nonnull)error {
     IBGSDKDebugLogsLevel resolvedLevel = (ArgsRegistry.sdkLogLevels[level]).integerValue;
     [Instabug setSdkDebugLogsLevel:resolvedLevel];

lib/src/modules/apm.dart

@@ -36,9 +36,6 @@ class APM {
 
   /// Sets log Level to determine level of details in a log
   /// [logLevel] Enum value to determine the level
-  @Deprecated(
-    "Use [Instabug.setSdkDebugLogsLevel] instead. ",
-  )
   static Future<void> setLogLevel(LogLevel logLevel) async {
     return _host.setLogLevel(logLevel.toString());
   }

lib/src/modules/instabug.dart

@@ -291,17 +291,9 @@ class Instabug {
   /// Android only
   /// Enable/disable SDK logs
   /// [debugEnabled] desired state of debug mode.
-  @Deprecated(
-    "Use [Instabug.setSdkDebugLogsLevel] instead. This will work on both Android and iOS. ",
-  )
   static Future<void> setDebugEnabled(bool debugEnabled) async {
     if (IBGBuildInfo.instance.isAndroid) {
-      if (debugEnabled) {
-        return _host
-            .setSdkDebugLogsLevel(IBGSDKDebugLogsLevel.verbose.toString());
-      } else {
-        return _host.setSdkDebugLogsLevel(IBGSDKDebugLogsLevel.none.toString());
-      }
+      return _host.setDebugEnabled(debugEnabled);
     }
   }
 

pigeons/instabug.api.dart

@@ -39,6 +39,7 @@ abstract class InstabugHostApi {
   @async
   Map<String, String>? getUserAttributes();
 
+  void setDebugEnabled(bool enabled);
   void setSdkDebugLogsLevel(String level);
 
   void setReproStepsMode(String mode);

test/apm_test.dart

@@ -61,7 +61,6 @@ void main() {
   test('[setLogLevel] should call host method', () async {
     const level = LogLevel.debug;
 
-    // ignore: deprecated_member_use_from_same_package
     await APM.setLogLevel(level);
 
     verify(

test/instabug_test.dart

@@ -270,17 +270,14 @@ void main() {
     ).called(1);
   });
 
-  test('[setDebugEnabled] should call setSdkDebugLogsLevel host method',
-      () async {
+  test('[setDebugEnabled] should call host method', () async {
     const enabled = true;
-    const level = IBGSDKDebugLogsLevel.verbose;
     when(mBuildInfo.isAndroid).thenReturn(true);
 
-    // ignore: deprecated_member_use_from_same_package
     await Instabug.setDebugEnabled(enabled);
 
     verify(
-      mHost.setSdkDebugLogsLevel(level.toString()),
+      mHost.setDebugEnabled(enabled),
     ).called(1);
   });