Remove deps of useSelector

reduxjs · May 2, 2019 · d849ede · d849ede
1 parent 06a674e
commit d849ede
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 38 deletions.
diff --git a/src/hooks/useSelector.js b/src/hooks/useSelector.js
@@ -24,8 +24,6 @@ const useIsomorphicLayoutEffect =
  * useful if you provide a selector that memoizes values).
  *
  * @param {Function} selector the selector function
- * @param {any[]} deps (optional) dependencies array to control referential stability
- * of the selector
  *
  * @returns {any} the selected state
  *
@@ -36,11 +34,11 @@ const useIsomorphicLayoutEffect =
  * import { RootState } from './store'
  *
  * export const CounterComponent = () => {
- *   const counter = useSelector(state => state.counter, [])
+ *   const counter = useSelector(state => state.counter)
  *   return <div>{counter}</div>
  * }
  */
-export function useSelector(selector, deps) {
+export function useSelector(selector) {
   invariant(selector, `You must pass a selector to useSelectors`)
 
   const { store, subscription: contextSub } = useReduxContext()
@@ -51,15 +49,13 @@ export function useSelector(selector, deps) {
     contextSub
   ])
 
-  const memoizedSelector = useMemo(() => selector, deps)
-
   const latestSubscriptionCallbackError = useRef()
-  const latestSelector = useRef(memoizedSelector)
+  const latestSelector = useRef(selector)
 
   let selectedState = undefined
 
   try {
-    selectedState = memoizedSelector(store.getState())
+    selectedState = selector(store.getState())
   } catch (err) {
     let errorMessage = `An error occured while selecting the store state: ${
       err.message
@@ -77,7 +73,7 @@ export function useSelector(selector, deps) {
   const latestSelectedState = useRef(selectedState)
 
   useIsomorphicLayoutEffect(() => {
-    latestSelector.current = memoizedSelector
+    latestSelector.current = selector
     latestSelectedState.current = selectedState
     latestSubscriptionCallbackError.current = undefined
   })

diff --git a/test/hooks/useSelector.spec.js b/test/hooks/useSelector.spec.js
@@ -1,6 +1,6 @@
 /*eslint-disable react/prop-types*/
 
-import React, { useReducer } from 'react'
+import React from 'react'
 import { createStore } from 'redux'
 import { renderHook, act } from 'react-hooks-testing-library'
 import * as rtl from 'react-testing-library'
@@ -154,34 +154,6 @@ describe('React', () => {
 
           expect(renderedItems.length).toBe(1)
         })
-
-        it('re-uses the selector if deps do not change', () => {
-          let selectorId = 0
-          let forceRender
-
-          const Comp = () => {
-            const [, f] = useReducer(c => c + 1, 0)
-            forceRender = f
-            const renderedSelectorId = selectorId++
-            const value = useSelector(() => renderedSelectorId, [])
-            renderedItems.push(value)
-            return <div />
-          }
-
-          rtl.render(
-            <ProviderMock store={store}>
-              <Comp />
-            </ProviderMock>
-          )
-
-          rtl.act(forceRender)
-
-          // this line verifies the susbcription callback uses the same memoized selector and therefore
-          // does not cause a re-render
-          store.dispatch({ type: '' })
-
-          expect(renderedItems).toEqual([0, 0])
-        })
       })
 
       describe('edge cases', () => {