From abe07200e6411028c05c6fa3ded65db6b054db82 Mon Sep 17 00:00:00 2001 From: Badr Rahal Date: Mon, 1 Oct 2018 21:12:36 +0200 Subject: [PATCH] Removed unused imports and const declarations (#5053) --- .../src/__tests__/NavigationContainer-test.js | 2 +- .../src/navigators/__tests__/SwitchNavigator-test.js | 2 +- .../src/navigators/__tests__/TabNavigator-test.js | 1 - .../src/navigators/createNavigator.js | 4 +--- .../src/navigators/createSwitchNavigator.js | 1 - .../src/routers/__tests__/PathHandling-test.js | 12 ------------ .../src/routers/__tests__/StackRouter-test.js | 4 ++-- .../src/routers/__tests__/TabRouter-test.js | 1 - .../src/routers/createConfigGetter.js | 2 +- .../src/views/AnimatedValueSubscription.js | 2 -- .../react-navigation/src/views/NavigationContext.js | 1 - .../src/views/ResourceSavingSceneView.js | 1 - .../src/views/__tests__/NavigationEvents-test.js | 1 - .../react-navigation/src/views/withNavigation.js | 1 - .../src/views/withNavigationFocus.js | 1 - 15 files changed, 6 insertions(+), 30 deletions(-) diff --git a/packages/react-navigation/src/__tests__/NavigationContainer-test.js b/packages/react-navigation/src/__tests__/NavigationContainer-test.js index 589c05d7..4c7807f6 100644 --- a/packages/react-navigation/src/__tests__/NavigationContainer-test.js +++ b/packages/react-navigation/src/__tests__/NavigationContainer-test.js @@ -1,5 +1,5 @@ import React from 'react'; -import { StyleSheet, View } from 'react-native'; +import { View } from 'react-native'; import renderer from 'react-test-renderer'; diff --git a/packages/react-navigation/src/navigators/__tests__/SwitchNavigator-test.js b/packages/react-navigation/src/navigators/__tests__/SwitchNavigator-test.js index da28dfc1..e9717926 100644 --- a/packages/react-navigation/src/navigators/__tests__/SwitchNavigator-test.js +++ b/packages/react-navigation/src/navigators/__tests__/SwitchNavigator-test.js @@ -1,4 +1,4 @@ -import React, { Component } from 'react'; +import React from 'react'; import { View } from 'react-native'; import renderer from 'react-test-renderer'; diff --git a/packages/react-navigation/src/navigators/__tests__/TabNavigator-test.js b/packages/react-navigation/src/navigators/__tests__/TabNavigator-test.js index 2f132b71..200bc75b 100644 --- a/packages/react-navigation/src/navigators/__tests__/TabNavigator-test.js +++ b/packages/react-navigation/src/navigators/__tests__/TabNavigator-test.js @@ -1,5 +1,4 @@ import React, { Component } from 'react'; -import { View } from 'react-native'; import renderer from 'react-test-renderer'; const { diff --git a/packages/react-navigation/src/navigators/createNavigator.js b/packages/react-navigation/src/navigators/createNavigator.js index 0c8ef75a..f5939041 100644 --- a/packages/react-navigation/src/navigators/createNavigator.js +++ b/packages/react-navigation/src/navigators/createNavigator.js @@ -1,8 +1,6 @@ import React from 'react'; import { polyfill } from 'react-lifecycles-compat'; -import getChildEventSubscriber from '../getChildEventSubscriber'; - function createNavigator(NavigatorView, router, navigationConfig) { class Navigator extends React.Component { static router = router; @@ -16,7 +14,7 @@ function createNavigator(NavigatorView, router, navigationConfig) { static getDerivedStateFromProps(nextProps, prevState) { const prevDescriptors = prevState.descriptors; const { navigation, screenProps } = nextProps; - const { dispatch, state, addListener } = navigation; + const { state } = navigation; const { routes } = state; if (typeof routes === 'undefined') { throw new TypeError( diff --git a/packages/react-navigation/src/navigators/createSwitchNavigator.js b/packages/react-navigation/src/navigators/createSwitchNavigator.js index 9e12d83f..ad4d9a73 100644 --- a/packages/react-navigation/src/navigators/createSwitchNavigator.js +++ b/packages/react-navigation/src/navigators/createSwitchNavigator.js @@ -1,4 +1,3 @@ -import React from 'react'; import createNavigator from '../navigators/createNavigator'; import SwitchRouter from '../routers/SwitchRouter'; import SwitchView from '../views/SwitchView/SwitchView'; diff --git a/packages/react-navigation/src/routers/__tests__/PathHandling-test.js b/packages/react-navigation/src/routers/__tests__/PathHandling-test.js index a5070039..5d09237a 100644 --- a/packages/react-navigation/src/routers/__tests__/PathHandling-test.js +++ b/packages/react-navigation/src/routers/__tests__/PathHandling-test.js @@ -4,9 +4,7 @@ import React from 'react'; import SwitchRouter from '../SwitchRouter'; import StackRouter from '../StackRouter'; -import StackActions from '../StackActions'; import NavigationActions from '../../NavigationActions'; -import { urlToPathAndParams } from '../pathUtils'; import { _TESTING_ONLY_normalize_keys } from '../KeyGenerator'; beforeEach(() => { @@ -442,16 +440,6 @@ const performRouterTest = createTestRouter => { }); test('URI encoded path param gets parsed and correctly printed', () => { - const router = createTestRouter({ - main: { - screen: () =>
, - }, - person: { - path: 'people/:name', - screen: () =>
, - }, - }); - const action = testRouter.getActionForPathAndParams('people/Henry%20L'); expect(action).toEqual({ routeName: 'person', diff --git a/packages/react-navigation/src/routers/__tests__/StackRouter-test.js b/packages/react-navigation/src/routers/__tests__/StackRouter-test.js index 123ae55b..e86d6147 100644 --- a/packages/react-navigation/src/routers/__tests__/StackRouter-test.js +++ b/packages/react-navigation/src/routers/__tests__/StackRouter-test.js @@ -492,7 +492,7 @@ describe('StackRouter', () => { }, state ); - const barKey = state2.routes[1].routes[0].key; + const state3 = router.getStateForAction( { type: StackActions.POP_TO_TOP, @@ -520,7 +520,7 @@ describe('StackRouter', () => { }, state ); - const barKey = state2.routes[1].routes[0].key; + const state3 = router.getStateForAction( { type: StackActions.POP_TO_TOP, diff --git a/packages/react-navigation/src/routers/__tests__/TabRouter-test.js b/packages/react-navigation/src/routers/__tests__/TabRouter-test.js index 48a00691..e4a67cab 100644 --- a/packages/react-navigation/src/routers/__tests__/TabRouter-test.js +++ b/packages/react-navigation/src/routers/__tests__/TabRouter-test.js @@ -3,7 +3,6 @@ import React from 'react'; import TabRouter from '../TabRouter'; -import StackActions from '../../routers/StackActions'; import NavigationActions from '../../NavigationActions'; const INIT_ACTION = { type: NavigationActions.INIT }; diff --git a/packages/react-navigation/src/routers/createConfigGetter.js b/packages/react-navigation/src/routers/createConfigGetter.js index c8d49bc9..519412bb 100644 --- a/packages/react-navigation/src/routers/createConfigGetter.js +++ b/packages/react-navigation/src/routers/createConfigGetter.js @@ -26,7 +26,7 @@ export default (routeConfigs, navigatorScreenConfig) => ( navigation, screenProps ) => { - const { state, dispatch } = navigation; + const { state } = navigation; const route = state; invariant( diff --git a/packages/react-navigation/src/views/AnimatedValueSubscription.js b/packages/react-navigation/src/views/AnimatedValueSubscription.js index dc4b2882..1715af8f 100644 --- a/packages/react-navigation/src/views/AnimatedValueSubscription.js +++ b/packages/react-navigation/src/views/AnimatedValueSubscription.js @@ -1,5 +1,3 @@ -import { Animated } from 'react-native'; - export default class AnimatedValueSubscription { constructor(value, callback) { this._value = value; diff --git a/packages/react-navigation/src/views/NavigationContext.js b/packages/react-navigation/src/views/NavigationContext.js index 098b82e4..2ee5a853 100644 --- a/packages/react-navigation/src/views/NavigationContext.js +++ b/packages/react-navigation/src/views/NavigationContext.js @@ -1,4 +1,3 @@ -import React from 'react'; import createReactContext from 'create-react-context'; const NavigationContext = createReactContext(); diff --git a/packages/react-navigation/src/views/ResourceSavingSceneView.js b/packages/react-navigation/src/views/ResourceSavingSceneView.js index fc7fa475..56177412 100644 --- a/packages/react-navigation/src/views/ResourceSavingSceneView.js +++ b/packages/react-navigation/src/views/ResourceSavingSceneView.js @@ -1,6 +1,5 @@ import React from 'react'; import { Platform, StyleSheet, View } from 'react-native'; -import PropTypes from 'prop-types'; import { polyfill } from 'react-lifecycles-compat'; import SceneView from './SceneView'; diff --git a/packages/react-navigation/src/views/__tests__/NavigationEvents-test.js b/packages/react-navigation/src/views/__tests__/NavigationEvents-test.js index 8a4c8968..2182b206 100644 --- a/packages/react-navigation/src/views/__tests__/NavigationEvents-test.js +++ b/packages/react-navigation/src/views/__tests__/NavigationEvents-test.js @@ -1,5 +1,4 @@ import React from 'react'; -import { View } from 'react-native'; import renderer from 'react-test-renderer'; import NavigationEvents from '../NavigationEvents'; import { NavigationProvider } from '../NavigationContext'; diff --git a/packages/react-navigation/src/views/withNavigation.js b/packages/react-navigation/src/views/withNavigation.js index 318ff478..2a580a4c 100644 --- a/packages/react-navigation/src/views/withNavigation.js +++ b/packages/react-navigation/src/views/withNavigation.js @@ -1,5 +1,4 @@ import React from 'react'; -import propTypes from 'prop-types'; import hoistStatics from 'hoist-non-react-statics'; import invariant from '../utils/invariant'; import { NavigationConsumer } from './NavigationContext'; diff --git a/packages/react-navigation/src/views/withNavigationFocus.js b/packages/react-navigation/src/views/withNavigationFocus.js index e4f88378..63c0b547 100644 --- a/packages/react-navigation/src/views/withNavigationFocus.js +++ b/packages/react-navigation/src/views/withNavigationFocus.js @@ -1,5 +1,4 @@ import React from 'react'; -import propTypes from 'prop-types'; import hoistStatics from 'hoist-non-react-statics'; import invariant from '../utils/invariant'; import withNavigation from './withNavigation';