refactor: remove IterableLogger instantiation from tests and clean up test structure

Author: lposen

src/__tests__/IterableInApp.test.ts

@@ -1,7 +1,5 @@
 import { NativeEventEmitter } from 'react-native';
 
-import { IterableLogger } from '../core';
-
 import { MockRNIterableAPI } from '../__mocks__/MockRNIterableAPI';
 
 import {
@@ -21,7 +19,6 @@ import {
 describe('Iterable In App', () => {
   beforeEach(() => {
     jest.clearAllMocks();
-    Iterable.logger = new IterableLogger(new IterableConfig());
   });
 
   test('trackInAppOpen_params_methodCalledWithParams', () => {
@@ -202,9 +199,11 @@ describe('Iterable In App', () => {
     // WHEN the simulated local queue is set to the in-app messages
     MockRNIterableAPI.setMessages(messages);
     // THEN Iterable.inAppManager.getMessages returns the list of in-app messages
-    return await Iterable.inAppManager?.getMessages().then((messagesObtained) => {
-      expect(messagesObtained).toEqual(messages);
-    });
+    return await Iterable.inAppManager
+      ?.getMessages()
+      .then((messagesObtained) => {
+        expect(messagesObtained).toEqual(messages);
+      });
   });
 
   test('showMessage_messageAndConsume_returnsClickedUrl', async () => {
@@ -222,9 +221,11 @@ describe('Iterable In App', () => {
     // WHEN the simulated clicked url is set to the clicked url
     MockRNIterableAPI.setClickedUrl(clickedUrl);
     // THEN Iterable,inAppManager.showMessage returns the simulated clicked url
-    return await Iterable.inAppManager?.showMessage(message, consume).then((url) => {
-      expect(url).toEqual(clickedUrl);
-    });
+    return await Iterable.inAppManager
+      ?.showMessage(message, consume)
+      .then((url) => {
+        expect(url).toEqual(clickedUrl);
+      });
   });
 
   test('removeMessage_params_methodCalledWithParams', () => {

src/core/classes/Iterable.test.ts

@@ -2,41 +2,32 @@ import { NativeEventEmitter, Platform } from 'react-native';
 
 import { MockLinking } from '../../__mocks__/MockLinking';
 import { MockRNIterableAPI } from '../../__mocks__/MockRNIterableAPI';
-import { IterableLogger } from '..';
 // import from the same location that consumers import from
 import {
   Iterable,
   IterableAction,
   IterableActionContext,
   IterableActionSource,
   IterableAttributionInfo,
+  IterableAuthResponse,
   IterableCommerceItem,
   IterableConfig,
   IterableDataRegion,
   IterableEventName,
-  IterableLogLevel,
-  IterableInAppMessage,
   IterableInAppCloseSource,
   IterableInAppDeleteSource,
   IterableInAppLocation,
+  IterableInAppMessage,
+  IterableInAppShowResponse,
   IterableInAppTrigger,
   IterableInAppTriggerType,
-  IterableAuthResponse,
-  IterableInAppShowResponse,
+  IterableLogLevel,
 } from '../..';
 import { TestHelper } from '../../__tests__/TestHelper';
 
-const getDefaultConfig = () => {
-  const config = new IterableConfig();
-  config.logReactNativeSdkCalls = false;
-  return config;
-};
-
 describe('Iterable', () => {
   beforeEach(() => {
     jest.clearAllMocks();
-    const config = getDefaultConfig();
-    Iterable.logger = new IterableLogger(config);
   });
 
   afterEach(() => {