session [nfc]: Add reducer param for (already-passed) GlobalState

To see how this param is already passed, look for

  applyReducer('session', …)

in src/boot/reducers.js.

Author: chrisbobbe

src/session/__tests__/sessionReducer-test.js

@@ -27,8 +27,11 @@ describe('sessionReducer', () => {
         // global
         { type: GOT_PUSH_TOKEN, pushToken: '456' },
         { type: APP_ORIENTATION, orientation: 'LANDSCAPE' },
-      ].reduce(sessionReducer, eg.baseReduxState.session);
-      expect(sessionReducer(prevState, eg.action.reset_account_data)).toEqual({
+      ].reduce(
+        (state, action) => sessionReducer(state, action, eg.plusReduxState),
+        eg.baseReduxState.session,
+      );
+      expect(sessionReducer(prevState, eg.action.reset_account_data, eg.plusReduxState)).toEqual({
         ...prevState,
         ...initialPerAccountSessionState,
       });
@@ -37,14 +40,14 @@ describe('sessionReducer', () => {
 
   test('DEAD_QUEUE', () => {
     const state = deepFreeze({ ...baseState, loading: true });
-    const newState = sessionReducer(state, deepFreeze({ type: DEAD_QUEUE }));
+    const newState = sessionReducer(state, deepFreeze({ type: DEAD_QUEUE }), eg.plusReduxState);
     expect(newState).toEqual({ ...baseState, loading: false });
   });
 
   test('REGISTER_COMPLETE', () => {
     const state = deepFreeze({ ...baseState, loading: true });
     const action = eg.mkActionRegisterComplete({ queue_id: '100' });
-    const newState = sessionReducer(state, action);
+    const newState = sessionReducer(state, action, eg.plusReduxState);
     expect(newState).toEqual({
       ...baseState,
       loading: false,
@@ -55,45 +58,56 @@ describe('sessionReducer', () => {
   test('APP_ONLINE', () => {
     const state = deepFreeze({ ...baseState, isOnline: false });
     const action = deepFreeze({ type: APP_ONLINE, isOnline: true });
-    const newState = sessionReducer(state, action);
+    const newState = sessionReducer(state, action, eg.plusReduxState);
     expect(newState).toEqual({ ...baseState, isOnline: true });
   });
 
   test('REGISTER_ABORT', () => {
     const state = deepFreeze({ ...baseState, loading: true });
-    const newState = sessionReducer(state, deepFreeze({ type: REGISTER_ABORT, reason: 'server' }));
+    const newState = sessionReducer(
+      state,
+      deepFreeze({ type: REGISTER_ABORT, reason: 'server' }),
+      eg.plusReduxState,
+    );
     expect(newState).toEqual({ ...baseState, loading: false });
   });
 
   test('REGISTER_START', () => {
     const state = deepFreeze({ ...baseState, loading: false });
-    const newState = sessionReducer(state, deepFreeze({ type: REGISTER_START }));
+    const newState = sessionReducer(state, deepFreeze({ type: REGISTER_START }), eg.plusReduxState);
     expect(newState).toEqual({ ...baseState, loading: true });
   });
 
   test('APP_ORIENTATION', () => {
     const state = deepFreeze({ ...baseState, orientation: 'PORTRAIT' });
     const orientation = 'LANDSCAPE';
     const action = deepFreeze({ type: APP_ORIENTATION, orientation });
-    expect(sessionReducer(state, action)).toEqual({ ...baseState, orientation });
+    expect(sessionReducer(state, action, eg.plusReduxState)).toEqual({ ...baseState, orientation });
   });
 
   test('GOT_PUSH_TOKEN', () => {
     const pushToken = 'pushToken';
     const action = deepFreeze({ type: GOT_PUSH_TOKEN, pushToken });
-    expect(sessionReducer(baseState, action)).toEqual({ ...baseState, pushToken });
+    expect(sessionReducer(baseState, action, eg.plusReduxState)).toEqual({
+      ...baseState,
+      pushToken,
+    });
   });
 
   test('TOGGLE_OUTBOX_SENDING', () => {
     const state = deepFreeze({ ...baseState, outboxSending: false });
     expect(
-      sessionReducer(state, deepFreeze({ type: TOGGLE_OUTBOX_SENDING, sending: true })),
+      sessionReducer(
+        state,
+        deepFreeze({ type: TOGGLE_OUTBOX_SENDING, sending: true }),
+        eg.plusReduxState,
+      ),
     ).toEqual({ ...baseState, outboxSending: true });
   });
 
   test('DISMISS_SERVER_COMPAT_NOTICE', () => {
     const action = deepFreeze({ type: DISMISS_SERVER_COMPAT_NOTICE });
-    expect(sessionReducer(baseState, action)).toEqual({
+    expect(sessionReducer(baseState, action, eg.plusReduxState)).toEqual({
       ...baseState,
       hasDismissedServerCompatNotice: true,
     });

src/session/sessionReducer.js

@@ -1,4 +1,5 @@
 /* @flow strict-local */
+import type { GlobalState } from '../reduxTypes';
 import type { Orientation, Action } from '../types';
 import {
   REHYDRATE,
@@ -134,8 +135,11 @@ const initialState: SessionState = {
   ...initialPerAccountSessionState,
 };
 
-// eslint-disable-next-line default-param-last
-export default (state: SessionState = initialState, action: Action): SessionState => {
+export default (
+  state: SessionState = initialState, // eslint-disable-line default-param-last
+  action: Action,
+  globalState: GlobalState,
+): SessionState => {
   switch (action.type) {
     case DEAD_QUEUE:
       return {