Fix some issues that were left over from the switch to weakMapMemoize

- Looks like some of the code in `computationComparisons.spec.tsx` wasn't properly updated after we switched the default memoization function from `lruMemoize` to `weakMapMemoize`. These have been updated to their correct implementations.
- Correct residual naming inconsistencies from the previous `defaultMemoize` to `lruMemoize` renaming. All references to `default` have been updated to `lru`.

Author: aryaemami59

test/computationComparisons.spec.tsx

@@ -3,7 +3,7 @@
  */
 
 import rtl from '@testing-library/react'
-import React, { useLayoutEffect, useMemo } from 'react'
+import { memo, useLayoutEffect, useMemo } from 'react'
 import type { TypedUseSelectorHook } from 'react-redux'
 import { Provider, shallowEqual, useSelector } from 'react-redux'
 import type { OutputSelector } from 'reselect'
@@ -14,7 +14,7 @@ import {
   weakMapMemoize
 } from 'reselect'
 import type { RootState, Todo } from './testUtils'
-import { addTodo, setupStore, toggleCompleted } from './testUtils'
+import { setupStore, toggleCompleted } from './testUtils'
 
 describe('Computations and re-rendering with React components', () => {
   let store: ReturnType<typeof setupStore>
@@ -27,37 +27,40 @@ describe('Computations and re-rendering with React components', () => {
   })
 
   type SelectTodoIds = OutputSelector<
-    [(state: RootState) => RootState['todos']],
+    [(state: RootState) => Todo[]],
     number[],
     typeof lruMemoize | typeof weakMapMemoize,
     typeof lruMemoize | typeof weakMapMemoize
   >
 
   type SelectTodoById = OutputSelector<
-    [
-      (state: RootState) => RootState['todos'],
-      (state: RootState, id: number) => number
-    ],
+    [(state: RootState) => Todo[], (state: RootState, id: number) => number],
     readonly [todo: Todo | undefined],
     typeof lruMemoize | typeof weakMapMemoize,
     typeof lruMemoize | typeof weakMapMemoize
   >
 
   const selectTodos = (state: RootState) => state.todos
-  const mapTodoIds = (todos: RootState['todos']) => todos.map(({ id }) => id)
-  const selectTodoId = (todos: RootState, id: number) => id
-  const mapTodoById = (todos: RootState['todos'], id: number) => {
+  const mapTodoIds = (todos: Todo[]) => todos.map(({ id }) => id)
+  const selectTodoId = (state: RootState, id: number) => id
+  const mapTodoById = (todos: Todo[], id: number) => {
     // Intentionally return this wrapped in an array to force a new reference each time
     return [todos.find(todo => todo.id === id)] as const
   }
 
-  const selectTodoIdsDefault = createSelector([selectTodos], mapTodoIds)
-  console.log(`selectTodoIdsDefault name: ${selectTodoIdsDefault.name}`)
+  const selectTodoIdsLru = createSelector([selectTodos], mapTodoIds, {
+    argsMemoize: lruMemoize,
+    memoize: lruMemoize
+  })
 
-  const selectTodoIdsResultEquality = createSelector(
+  const selectTodoIdsLruResultEquality = createSelector(
     [selectTodos],
     mapTodoIds,
-    { memoizeOptions: { resultEqualityCheck: shallowEqual } }
+    {
+      memoizeOptions: { resultEqualityCheck: shallowEqual },
+      memoize: lruMemoize,
+      argsMemoize: lruMemoize
+    }
   )
 
   const selectTodoIdsWeakMap = createSelector([selectTodos], mapTodoIds, {
@@ -75,16 +78,18 @@ describe('Computations and re-rendering with React components', () => {
     }
   )
 
-  const selectTodoByIdDefault = createSelector(
+  const selectTodoByIdLru = createSelector(
     [selectTodos, selectTodoId],
-    mapTodoById
+    mapTodoById,
+    { memoize: lruMemoize, argsMemoize: lruMemoize }
   )
 
-  const selectTodoByIdResultEquality = createSelector(
+  const selectTodoByIdLruResultEquality = createSelector(
     [selectTodos, selectTodoId],
     mapTodoById,
     {
       memoize: lruMemoize,
+      argsMemoize: lruMemoize,
       memoizeOptions: { resultEqualityCheck: shallowEqual, maxSize: 500 }
     }
   )
@@ -101,7 +106,7 @@ describe('Computations and re-rendering with React components', () => {
   let listRenders = 0
   let listItemMounts = 0
 
-  const TodoListItem = React.memo(function TodoListItem({
+  const TodoListItem = memo(function TodoListItem({
     id,
     selectTodoById
   }: {
@@ -150,11 +155,11 @@ describe('Computations and re-rendering with React components', () => {
   }
 
   const testCases: [string, SelectTodoIds, SelectTodoById][] = [
-    ['default', selectTodoIdsDefault, selectTodoByIdDefault],
+    ['lru', selectTodoIdsLru, selectTodoByIdLru],
     [
-      'resultEquality',
-      selectTodoIdsResultEquality,
-      selectTodoByIdResultEquality
+      'lruResultEquality',
+      selectTodoIdsLruResultEquality,
+      selectTodoByIdLruResultEquality
     ],
     ['weakMap', selectTodoIdsWeakMap, selectTodoByIdWeakMap],