Bug fixes & improvements (#15)

* IM-0 - add lint and tsc check to travis

* IM-0 - clean before each init

* IM-0 - bump version

* IM-0 - code clean

Co-authored-by: Imran Mentese <imran.mentese@arhs-spikeseed.com>

Author: imranMnts

.travis.yml

@@ -4,6 +4,8 @@ node_js:
 install:
   - yarn
 script:
+  - yarn run tsc
+  - yarn run lint
   - yarn run test
 after_success:
   - npm run test:cov && bash <(curl -s https://codecov.io/bash) -e TRAVIS_NODE_VERSION

CHANGELOG.md

@@ -1,5 +1,10 @@
 # Changelog
 
+### Version 1.2.1
+    
+- BugFixes
+    - Fix small issue when app refreshed many times
+
 ### Version 1.2.0
 
 - News

package.json

@@ -1,11 +1,12 @@
 {
   "name": "react-native-logging-tools",
   "title": "React Native library created to ease logging",
-  "version": "1.2.0",
+  "version": "1.2.1",
   "main": "src/index.ts",
   "typings": "index.d.ts",
   "scripts": {
     "prettier:fix": "prettier --config ./.prettierrc.js --write --check \"src/**/*.ts\" ",
+    "lint": "tslint -p tsconfig.json 'src/**/*.tsx' 'src/**/*.ts'",
     "lint:fix": "tslint -p tsconfig.json --fix 'src/**/*.tsx' 'src/**/*.ts'",
     "precommit": "yarn run prettier:fix && yarn run lint:fix && yarn run tsc",
     "test": "TZ=UTC jest",

src/__tests__/index.spec.ts

@@ -17,7 +17,6 @@ import {
 import * as Init from '../modules/init';
 import { ITealium } from '../model/tealium';
 import { log } from '../modules/events';
-import { excludeLogs, loggers } from '../modules/init';
 
 describe('index test suite', () => {
   const analytics = {
@@ -72,18 +71,18 @@ describe('index test suite', () => {
 
   it('should init properly empty', () => {
     init({});
-    expect(loggers).toEqual([]);
+    expect(Init.loggers).toEqual([]);
   });
 
   it('should init properly', () => {
     init({ config: {}, analytics: [], errorReporters: [] });
-    expect(loggers).toEqual([]);
+    expect(Init.loggers).toEqual([]);
   });
 
   it('should init properly with wrong analytics and errorReporters', () => {
     // @ts-ignore
     init({ config: {}, analytics: [{}], errorReporters: [{}] });
-    expect(loggers).toEqual([]);
+    expect(Init.loggers).toEqual([]);
   });
 
   it('should init properly and log event with reject', () => {
@@ -121,7 +120,7 @@ describe('index test suite', () => {
         excludeLogs: { instabug: [DEBUG_LOG] },
       },
     });
-    expect(excludeLogs).toEqual({ instabug: [DEBUG_LOG] });
+    expect(Init.excludeLogs).toEqual({ instabug: [DEBUG_LOG] });
   });
 
   // Firebase & Sentry

src/flipper/__tests__/index.spec.ts

@@ -3,7 +3,7 @@ import { getCurrentDateTime } from '../../helpers/functions';
 
 jest.mock('../../helpers/functions', () => ({
   getCurrentDateTime: jest.fn(),
-}))
+}));
 
 describe('flipper test suite', () => {
   it('should init properly flipper and send event', () => {

src/helpers/functions.ts

@@ -13,8 +13,6 @@ export const isFunction = (func: any) => func && typeof func === 'function';
  *
  * @return return current date and hour
  */
-export const getCurrentDateTime = (date = new Date()) => {
-  return `${date.getFullYear()}-${
-    date.getMonth() + 1
-  }-${date.getDate()} ${date.getHours()}:${date.getMinutes()}:${date.getSeconds()}`;
-};
+export const getCurrentDateTime = (date = new Date()) =>
+  `${date.getFullYear()}-${date.getMonth() +
+    1}-${date.getDate()} ${date.getHours()}:${date.getMinutes()}:${date.getSeconds()}`;

src/modules/init/index.ts

@@ -14,11 +14,18 @@ export let Reactotron: any;
 export let AsyncStorage: any;
 export let isSensitiveBuild: boolean;
 
-export const loggers: Function[] = [];
-export const recordErrors: Function[] = [];
+export let loggers: Function[] = [];
+export let recordErrors: Function[] = [];
 export let excludeLogs: IExcludeLogs = {};
 
+function clean() {
+  loggers = [];
+  recordErrors = [];
+}
+
 export function init(initConfig: IInit): void {
+  clean();
+
   if (initConfig.config) {
     if (initConfig.config.useFlipperPlugin) {
       flipperConnectionManager = new FlipperConnectionManager();