Merge pull request #452 from Iterable/evan/MOB-5314-linting-changes-to-test-files-and-config-files

[MOB-5314] linting changes to test files and config files

Author: evangreer91

babel.config.js

@@ -1,3 +1,3 @@
 module.exports = {
-  presets: ['module:metro-react-native-babel-preset'],
-};
+  presets: ['module:metro-react-native-babel-preset']
+}

jest.config.js

@@ -1,6 +1,6 @@
 module.exports = {
   preset: 'react-native',
-  setupFiles: ["<rootDir>/ts/__mocks__/jest.setup.ts"],
+  setupFiles: ['<rootDir>/ts/__mocks__/jest.setup.ts'],
   moduleFileExtensions: ['ts', 'tsx', 'js', 'jsx', 'json', 'node'],
-  testMatch: ["<rootDir>/ts/__tests__/**/*.(test|spec).[jt]s?(x)"]
-};
+  testMatch: ['<rootDir>/ts/__tests__/**/*.(test|spec).[jt]s?(x)']
+}

ts/__mocks__/MockLinking.ts

@@ -1,8 +1,8 @@
+// eslint-disable-next-line @typescript-eslint/no-extraneous-class
 export class MockLinking {
   static canOpenURL = jest.fn()
   static openURL = jest.fn()
 
   static addEventListener = jest.fn()
   static removeEventListener = jest.fn()
 }
-

ts/__mocks__/MockRNIterableAPI.ts

@@ -1,6 +1,7 @@
 import { IterableAttributionInfo } from '../Iterable'
 import IterableInAppMessage from '../IterableInAppMessage'
 
+// eslint-disable-next-line @typescript-eslint/no-extraneous-class
 export class MockRNIterableAPI {
   static email?: string
   static userId?: string
@@ -10,24 +11,24 @@ export class MockRNIterableAPI {
   static messages?: IterableInAppMessage[]
   static clickedUrl?: string
 
-  static getEmail(): Promise<string | undefined> {
-    return new Promise((resolve, _) => {
+  static async getEmail (): Promise<string | undefined> {
+    return await new Promise((resolve, reject) => {
       resolve(MockRNIterableAPI.email)
     })
   }
 
-  static setEmail(email: string, authToken?: string | undefined) {
+  static setEmail (email: string, authToken?: string | undefined): void {
     MockRNIterableAPI.email = email
     MockRNIterableAPI.token = authToken
   }
 
-  static getUserId(): Promise<string | undefined> {
-    return new Promise((resolve, _) => {
+  static async getUserId (): Promise<string | undefined> {
+    return await new Promise((resolve, reject) => {
       resolve(MockRNIterableAPI.userId)
     })
   }
 
-  static setUserId(userId: string, authToken?: string | undefined) {
+  static setUserId (userId: string, authToken?: string | undefined): void {
     MockRNIterableAPI.userId = userId
     MockRNIterableAPI.token = authToken
   }
@@ -48,36 +49,36 @@ export class MockRNIterableAPI {
 
   static trackEvent = jest.fn()
 
-  static getLastPushPayload(): Promise<any | undefined> {
-    return new Promise((resolve, _) => {
+  static async getLastPushPayload (): Promise<any | undefined> {
+    return await new Promise((resolve, reject) => {
       resolve(MockRNIterableAPI.lastPushPayload)
     })
   }
 
-  static getAttributionInfo(): Promise<IterableAttributionInfo | undefined> {
-    return new Promise((resolve, _) => {
+  static async getAttributionInfo (): Promise<IterableAttributionInfo | undefined> {
+    return await new Promise((resolve, reject) => {
       resolve(MockRNIterableAPI.attributionInfo)
     })
   }
 
-  static setAttributionInfo(attributionInfo?: IterableAttributionInfo) {
+  static setAttributionInfo (attributionInfo?: IterableAttributionInfo): void {
     MockRNIterableAPI.attributionInfo = attributionInfo
   }
 
   static initializeWithApiKey = jest.fn()
 
   static setInAppShowResponse = jest.fn()
 
-  static getInAppMessages(): Promise<IterableInAppMessage[] | undefined> {
-    return new Promise((resolve, _) => {
+  static async getInAppMessages (): Promise<IterableInAppMessage[] | undefined> {
+    return await new Promise((resolve, reject) => {
       resolve(MockRNIterableAPI.messages)
     })
   }
 
   static setAutoDisplayPaused = jest.fn()
 
-  static showMessage(message: IterableInAppMessage, consume: boolean): Promise<string | undefined> {
-    return new Promise((resolve, _) => {
+  static async showMessage (message: IterableInAppMessage, consume: boolean): Promise<string | undefined> {
+    return await new Promise((resolve, reject) => {
       resolve(MockRNIterableAPI.clickedUrl)
     })
   }
@@ -98,13 +99,13 @@ export class MockRNIterableAPI {
 
   // set messages function is to set the messages static property
   // this is for testing purposes only
-  static setMessages(messages: IterableInAppMessage[]) {
+  static setMessages (messages: IterableInAppMessage[]): void {
     MockRNIterableAPI.messages = messages
   }
-  
+
   // setClickedUrl function is to set the messages static property
   // this is for testing purposes only
-  static setClickedUrl(clickedUrl: string) {
+  static setClickedUrl (clickedUrl: string): void {
     MockRNIterableAPI.clickedUrl = clickedUrl
   }
 }

ts/__mocks__/jest.setup.ts

@@ -1,8 +1,8 @@
 import { MockRNIterableAPI } from './MockRNIterableAPI'
 import { MockLinking } from './MockLinking'
-jest.mock('react-native/Libraries/EventEmitter/NativeEventEmitter.js')
 
 import * as ReactNative from 'react-native'
+jest.mock('react-native/Libraries/EventEmitter/NativeEventEmitter.js')
 
 jest.doMock('react-native', () => {
   // Extend ReactNative
@@ -11,10 +11,10 @@ jest.doMock('react-native', () => {
       // Mock RNIterableAPI
       NativeModules: {
         ...ReactNative.NativeModules,
-        RNIterableAPI: MockRNIterableAPI,
+        RNIterableAPI: MockRNIterableAPI
       },
-      Linking: MockLinking,
+      Linking: MockLinking
     },
-    ReactNative,
+    ReactNative
   )
 })