chore: apply pr comments

Author: andrewdanieldev

android/src/test/java/com/instabug/reactlibrary/RNInstabugNetworkLoggerModuleTest.java

@@ -204,12 +204,4 @@ public void testRegisterNetworkLogsListenerCalled() {
             internalAPMMock.verify(() -> InternalAPM._registerNetworkLogSanitizer(any()));
         }
     }
-
-
-//    @Test
-//    public void testUpdateNetworkLogSnapshotInvalidJson() {
-//        String invalidJsonString = "{\"id\":\"testId\"";
-//
-//        assertThrows(RuntimeException.class, () -> rnInstabugNetworkLoggerModule.updateNetworkLogSnapshot(invalidJsonString));
-//    }
 }

src/modules/Instabug.ts

@@ -192,7 +192,7 @@ const handleNetworkInterceptionMode = (config: InstabugConfig) => {
 function handleAndroidJSInterception() {
   if (isNativeInterceptionFeatureEnabled && hasAPMNetworkPlugin) {
     shouldEnableNativeInterception = true;
-    console.warn(
+    Logger.warn(
       InstabugConstants.IBG_APM_TAG + InstabugConstants.SWITCHED_TO_NATIVE_INTERCEPTION_MESSAGE,
     );
   }
@@ -206,11 +206,11 @@ function handleAndroidNativeInterception() {
   if (isNativeInterceptionFeatureEnabled) {
     shouldEnableNativeInterception = hasAPMNetworkPlugin;
     if (!hasAPMNetworkPlugin) {
-      console.error(InstabugConstants.IBG_APM_TAG + InstabugConstants.PLUGIN_NOT_INSTALLED_MESSAGE);
+      Logger.error(InstabugConstants.IBG_APM_TAG + InstabugConstants.PLUGIN_NOT_INSTALLED_MESSAGE);
     }
   } else {
     shouldEnableNativeInterception = false; // rollback to use JS interceptor for APM & Core.
-    console.error(
+    Logger.error(
       InstabugConstants.IBG_APM_TAG + InstabugConstants.NATIVE_INTERCEPTION_DISABLED_MESSAGE,
     );
   }
@@ -256,7 +256,7 @@ const handleInterceptionModeForIOS = (config: InstabugConfig) => {
     } else {
       shouldEnableNativeInterception = false;
       NetworkLogger.setEnabled(true); // rollback to JS interceptor
-      console.error(
+      Logger.error(
         InstabugConstants.IBG_APM_TAG + InstabugConstants.NATIVE_INTERCEPTION_DISABLED_MESSAGE,
       );
     }

src/modules/NetworkLogger.ts

@@ -172,7 +172,7 @@ export const resetNetworkListener = () => {
     _networkListener = null;
     NativeNetworkLogger.resetNetworkLogsListener();
   } else {
-    console.error(
+    Logger.error(
       `${InstabugConstants.IBG_APM_TAG}: The \`resetNetworkListener()\` method is intended solely for testing purposes.`,
     );
   }