🎨 (src) fix eslint errors

Author: tpucci

src/FullScreenPortal.mdx

@@ -16,7 +16,7 @@ route: /api/full-screen-portal
 
 ## Properties
 
-### `fullScreenStack: IStreamListener<IStop[]>`
+### `fullScreenStack: IStreamListener<StopInterface[]>`
 
 MobX Strem Listener of list of React Gondola full screen Stops. This stream is observed by MobX and fires the rendering of the full screen stack in the `FullScreenPortal.render` method.
 
@@ -25,10 +25,10 @@ MobX Strem Listener of list of React Gondola full screen Stops. This stream is o
 Wrap your application main canal with `FullScreenPortal`. `FullScreenPortal` must appear at the top of your tree in order to render full screen pages.
 
 ```tsx
-import React, { Component } from "react";
-import { FullScreenPortal } from "react-gondola";
+import React, { Component } from 'react';
+import { FullScreenPortal } from 'react-gondola';
 
-import { MainCanal } from "./canals/MainCanal";
+import { MainCanal } from './canals/MainCanal';
 
 export default class App extends Component {
   render() {

src/FullScreenPortal.tsx

@@ -5,9 +5,7 @@ import { fromStream } from 'mobx-utils';
 import { Navigation } from './Navigation';
 
 export class FullScreenPortal extends ReactComponent {
-  fullScreenStack = fromStream(
-    Navigation.instance.fullScreenDelegate.fullSceenStack$
-  );
+  fullScreenStack = fromStream(Navigation.instance.fullScreenDelegate.fullSceenStack$);
 
   render() {
     return (
@@ -18,11 +16,9 @@ export class FullScreenPortal extends ReactComponent {
             if (this.fullScreenStack.current) {
               return (
                 this.fullScreenStack.current &&
-                this.fullScreenStack.current.map(
-                  ({ Component, name, isAuthorized }) => (
-                    <Component isAuthorized={isAuthorized} key={name} />
-                  )
-                )
+                this.fullScreenStack.current.map(({ Component, name, isAuthorized }) => (
+                  <Component isAuthorized={isAuthorized} key={name} />
+                ))
               );
             }
             return null;

src/Navigation/BackContext.ts

@@ -1,12 +1,12 @@
 import { createContext } from 'react';
 import { Observable } from 'rxjs';
-import { IBackEvent } from './BackHandlerDelegate';
+import { BackEvent } from './BackHandlerDelegate';
 import { Navigation } from './Navigation';
 
-export interface IBackContext {
-  back$: Observable<IBackEvent>;
+export interface BackContextInterface {
+  back$: Observable<BackEvent>;
 }
 
-export const BackContext = createContext<IBackContext>(
+export const BackContext = createContext<BackContextInterface>(
   Navigation.instance.backHandlerDelegate.defaultBackContext
 );

src/Navigation/BackHandlerDelegate.ts

@@ -2,7 +2,7 @@ import { BackHandler } from 'react-native';
 import { fromEventPattern, Observable } from 'rxjs';
 import { share } from 'rxjs/operators';
 
-export interface IBackEvent {
+export interface BackEvent {
   target: string | null;
 }
 
@@ -16,10 +16,10 @@ export class BackHandlerDelegate {
    * See https://facebook.github.io/react-native/docs/backhandler.html.
    * @param handler
    */
-  private addEventListener = (handler: (event: IBackEvent) => void) => {
+  private addEventListener = (handler: (event: BackEvent) => void) => {
     BackHandler.addEventListener('hardwareBackPress', () => {
       handler({
-        target: null
+        target: null,
       });
       if (this.onBackCallback) {
         this.onBackCallback();
@@ -30,9 +30,9 @@ export class BackHandlerDelegate {
     });
   };
 
-  private back$: Observable<IBackEvent> = fromEventPattern<IBackEvent>(
-    this.addEventListener
-  ).pipe(share());
+  private back$: Observable<BackEvent> = fromEventPattern<BackEvent>(this.addEventListener).pipe(
+    share()
+  );
 
   private onBackCallback?: () => any;
   setOnBackCallback = (cb: () => any) => {

src/Navigation/FullScreenDelegate.ts

@@ -1,39 +1,34 @@
 import { Subject, Observable } from 'rxjs';
 import { mergeAll, map, scan } from 'rxjs/operators';
-import { IStop } from './Stop';
+import { StopInterface } from './Stop';
 
-type Stack = IStop[];
+type Stack = StopInterface[];
 
-interface IFullScreenStackProperties {
+interface FullScreenStackProperties {
   canalId: string;
   fullScreenStack: Stack;
 }
 
-interface IFullScreenStackMap {
+interface FullScreenStackMap {
   [canalId: string]: Stack;
 }
 
 export class FullScreenDelegate {
-  canalsFullScreenStackProperties$ = new Subject<
-    Observable<IFullScreenStackProperties>
-  >();
+  canalsFullScreenStackProperties$ = new Subject<Observable<FullScreenStackProperties>>();
 
-  fullSceenStack$: Observable<
-    Stack
-  > = this.canalsFullScreenStackProperties$.pipe(
+  fullSceenStack$: Observable<Stack> = this.canalsFullScreenStackProperties$.pipe(
     mergeAll(),
     scan(
       (
-        fullScreenStackMap: IFullScreenStackMap,
-        fullScreenStackProperties: IFullScreenStackProperties
+        fullScreenStackMap: FullScreenStackMap,
+        fullScreenStackProperties: FullScreenStackProperties
       ) => ({
         ...fullScreenStackMap,
-        [fullScreenStackProperties.canalId]:
-          fullScreenStackProperties.fullScreenStack
+        [fullScreenStackProperties.canalId]: fullScreenStackProperties.fullScreenStack,
       }),
       {}
     ),
-    map((fullScreenStackMap: IFullScreenStackMap) =>
+    map((fullScreenStackMap: FullScreenStackMap) =>
       Object.keys(fullScreenStackMap).reduce(
         (fullScreenStack: Stack, canalId: string) =>
           fullScreenStack.concat(fullScreenStackMap[canalId]),

src/Navigation/Stop.d.ts

@@ -1,12 +1,12 @@
 import { ComponentType } from 'react';
 
-export interface IStop {
-  Component: ComponentType<IStopComponentProps>;
+export interface StopInterface {
+  Component: ComponentType<StopComponentProps>;
   isFullScreen?: boolean;
   name: string;
   isAuthorized: boolean;
 }
 
-export interface IStopComponentProps {
+export interface StopComponentProps {
   isAuthorized: boolean;
 }

src/Navigation/__tests__/BackHandlerDelegate.test.tsx

@@ -2,9 +2,7 @@ import BackHandler from 'react-native/Libraries/Utilities/__mocks__/BackHandler'
 import { BackHandlerDelegate } from '../BackHandlerDelegate';
 
 jest.mock('react-native', () => ({
-  BackHandler: require.requireActual(
-    'react-native/Libraries/Utilities/__mocks__/BackHandler'
-  )
+  BackHandler: require.requireActual('react-native/Libraries/Utilities/__mocks__/BackHandler'),
 }));
 
 describe('BackHandlerDelegate', () => {

src/StopValidator.ts

@@ -2,7 +2,7 @@ import React, { ComponentType } from 'react';
 
 import { TransitionComponentType } from './transitions/Transition';
 
-export interface IStop<T extends string> {
+export interface StopInterface<T extends string> {
   name: T;
   Component: ComponentType<any>;
   isFullScreen?: boolean;
@@ -12,21 +12,16 @@ export interface IStop<T extends string> {
 }
 
 export class StopValidator {
-  static validate(stop: IStop<string>) {
+  static validate(stop: StopInterface<string>) {
     if (!stop.name || typeof stop.name !== 'string') {
       throw new Error('ERR_INVALID_NAME');
     }
-    if (
-      !(
-        React.isValidElement(stop.Component) ||
-        typeof stop.Component === 'function'
-      )
-    ) {
+    if (!(React.isValidElement(stop.Component) || typeof stop.Component === 'function')) {
       throw new Error('ERR_INVALID_COMPONENT');
     }
   }
 
-  static validateList(stops: Array<IStop<string>>) {
+  static validateList(stops: StopInterface<string>[]) {
     for (let index = 0; index < stops.length; index++) {
       const stop = stops[index];
       try {

src/__tests__/createCanal.test.tsx

@@ -7,7 +7,7 @@ import { stopCreator } from './utils/stopCreator';
 import { createCanal } from '../createCanal';
 import { Subject } from 'rxjs';
 import { BackContext } from '../Navigation/BackContext';
-import { IBackEvent } from '../Navigation/BackHandlerDelegate';
+import { BackEvent } from '../Navigation/BackHandlerDelegate';
 
 describe('createCanal', () => {
   it('throws an error if first arg is not a Component', () => {
@@ -41,21 +41,13 @@ describe('createCanal', () => {
   });
 
   it('renders only the first page if all authorizations are given but the one for the second page', () => {
-    const Canal = createCanal([
-      stopCreator('a'),
-      stopCreator('b'),
-      stopCreator('c')
-    ]);
+    const Canal = createCanal([stopCreator('a'), stopCreator('b'), stopCreator('c')]);
     const testRenderer = TestRenderer.create(<Canal a c />);
     expect(testRenderer.toJSON()).toMatchSnapshot();
   });
 
   it('only rerender if style was modified', () => {
-    const Canal = createCanal([
-      stopCreator('a'),
-      stopCreator('b'),
-      stopCreator('c')
-    ]);
+    const Canal = createCanal([stopCreator('a'), stopCreator('b'), stopCreator('c')]);
     const testRenderer = TestRenderer.create(<Canal a />);
     const renderSpy = jest.spyOn(
       // @ts-ignore
@@ -98,9 +90,7 @@ describe('createCanal', () => {
       // @ts-ignore
       createCanal([stopCreator('a'), stopCreator('a')]);
     } catch (error) {
-      expect(error.message).toBe(
-        '`createCanal` found duplicated `name: a` key.'
-      );
+      expect(error.message).toBe('`createCanal` found duplicated `name: a` key.');
     }
     expect.assertions(1);
   });
@@ -111,9 +101,7 @@ describe('createCanal', () => {
     const Canal = createCanal([stopCreator('a'), stopB]);
     const testRenderer = TestRenderer.create(<Canal a />);
     // @ts-ignore
-    testRenderer.root.children[0].instance.fullScreenStackProperties$.subscribe(
-      spy
-    );
+    testRenderer.root.children[0].instance.fullScreenStackProperties$.subscribe(spy);
     // @ts-ignore
     const [_, StopB] = testRenderer.root.children[0].instance.stopsList;
     testRenderer.update(<Canal a />);
@@ -124,9 +112,9 @@ describe('createCanal', () => {
           Component: StopB.Component,
           isAuthorized: false,
           isFullScreen: true,
-          name: 'b'
-        }
-      ]
+          name: 'b',
+        },
+      ],
     });
     testRenderer.update(<Canal a />);
     testRenderer.update(<Canal a b />);
@@ -137,9 +125,9 @@ describe('createCanal', () => {
           Component: StopB.Component,
           isAuthorized: true,
           isFullScreen: true,
-          name: 'b'
-        }
-      ]
+          name: 'b',
+        },
+      ],
     });
     testRenderer.update(<Canal a b />);
     testRenderer.update(<Canal a b />);
@@ -151,9 +139,7 @@ describe('createCanal', () => {
     const Canal = createCanal([stopCreator('a'), stopCreator('b', true)]);
     const testRenderer = TestRenderer.create(<Canal a />);
     // @ts-ignore
-    testRenderer.root.children[0].instance.fullScreenStackProperties$.subscribe(
-      spy
-    );
+    testRenderer.root.children[0].instance.fullScreenStackProperties$.subscribe(spy);
     // @ts-ignore
     const [_, StopB] = testRenderer.root.children[0].instance.stopsList;
     testRenderer.update(<Canal a b />);
@@ -166,9 +152,9 @@ describe('createCanal', () => {
           Component: StopB.Component,
           isAuthorized: true,
           isFullScreen: true,
-          name: 'b'
-        }
-      ]
+          name: 'b',
+        },
+      ],
     });
     expect(spy).toHaveBeenCalledWith({
       canalId: '0',
@@ -177,14 +163,14 @@ describe('createCanal', () => {
           Component: StopB.Component,
           isAuthorized: false,
           isFullScreen: true,
-          name: 'b'
-        }
-      ]
+          name: 'b',
+        },
+      ],
     });
   });
 
   it('passes back events on', () => {
-    const back$ = new Subject<IBackEvent>();
+    const back$ = new Subject<BackEvent>();
     const spy = jest.fn();
     const Canal = createCanal([stopCreator('a'), stopCreator('b')]);
     const testRenderer = TestRenderer.create(
@@ -196,7 +182,7 @@ describe('createCanal', () => {
     testRenderer.root.children[0].instance.back$.subscribe(spy);
     back$.next({ target: null });
     expect(spy).toHaveBeenCalledWith({
-      target: 'a'
+      target: 'a',
     });
     testRenderer.update(
       <BackContext.Provider value={{ back$ }}>
@@ -205,7 +191,7 @@ describe('createCanal', () => {
     );
     back$.next({ target: null });
     expect(spy).toHaveBeenCalledWith({
-      target: 'b'
+      target: 'b',
     });
     testRenderer.update(
       <BackContext.Provider value={{ back$ }}>
@@ -214,7 +200,7 @@ describe('createCanal', () => {
     );
     back$.next({ target: null });
     expect(spy).toHaveBeenCalledWith({
-      target: null
+      target: null,
     });
   });
 });