model tests: Cover RESET_ACCOUNT_DATA for all reducers that use it

In #5606, I updated all the existing tests for ACCOUNT_SWITCH, etc.,
to test RESET_ACCOUNT_DATA instead, naturally.

…But that didn't give complete coverage of the RESET_ACCOUNT_DATA
action, because some tests weren't testing for ACCOUNT_SWITCH, etc.,
in the first place. So, add the coverage now.

Author: chrisbobbe

src/alertWords/__tests__/alertWordsReducer-test.js

@@ -7,6 +7,15 @@ import { EVENT_ALERT_WORDS } from '../../actionConstants';
 import * as eg from '../../__tests__/lib/exampleData';
 
 describe('alertWordsReducer', () => {
+  describe('RESET_ACCOUNT_DATA', () => {
+    const initialState = eg.baseReduxState.alertWords;
+    const action1 = eg.mkActionRegisterComplete({ alert_words: ['word', '@mobile-core', 'alert'] });
+    const prevState = alertWordsReducer(initialState, action1);
+    expect(prevState).not.toEqual(initialState);
+
+    expect(alertWordsReducer(prevState, eg.action.reset_account_data)).toEqual(initialState);
+  });
+
   describe('REGISTER_COMPLETE', () => {
     test('when `alert_words` data is provided init state with it', () => {
       const prevState = deepFreeze([]);

src/caughtup/__tests__/caughtUpReducer-test.js

@@ -16,6 +16,15 @@ import {
 import { objectFromEntries } from '../../jsBackport';
 
 describe('caughtUpReducer', () => {
+  describe('RESET_ACCOUNT_DATA', () => {
+    const initialState = eg.baseReduxState.caughtUp;
+    const action1 = { ...eg.action.message_fetch_complete, foundNewest: true, foundOldest: true };
+    const prevState = caughtUpReducer(initialState, action1);
+    expect(prevState).not.toEqual(initialState);
+
+    expect(caughtUpReducer(prevState, eg.action.reset_account_data)).toEqual(initialState);
+  });
+
   describe('MESSAGE_FETCH_START', () => {
     test('when fetch starts caught up does not change', () => {
       const prevState = deepFreeze({ [HOME_NARROW_STR]: { older: true, newer: true } });

src/chat/__tests__/fetchingReducer-test.js

@@ -14,6 +14,15 @@ import { MESSAGE_FETCH_START, MESSAGE_FETCH_ERROR } from '../../actionConstants'
 import { DEFAULT_FETCHING } from '../fetchingSelectors';
 
 describe('fetchingReducer', () => {
+  describe('RESET_ACCOUNT_DATA', () => {
+    const initialState = eg.baseReduxState.fetching;
+    const action1 = { type: MESSAGE_FETCH_START, narrow: HOME_NARROW, numBefore: 10, numAfter: 10 };
+    const prevState = fetchingReducer(initialState, action1);
+    expect(prevState).not.toEqual(initialState);
+
+    expect(fetchingReducer(prevState, eg.action.reset_account_data)).toEqual(initialState);
+  });
+
   describe('MESSAGE_FETCH_START', () => {
     test('if messages are fetched before or after the corresponding flag is set', () => {
       const initialState = deepFreeze({

src/chat/__tests__/narrowsReducer-test.js

@@ -30,6 +30,15 @@ describe('narrowsReducer', () => {
   const egTopic = eg.streamMessage().subject;
   const topicNarrowStr = keyFromNarrow(topicNarrow(eg.stream.stream_id, egTopic));
 
+  describe('RESET_ACCOUNT_DATA', () => {
+    const initialState = eg.baseReduxState.narrows;
+    const action1 = { ...eg.action.message_fetch_complete, messages: [eg.streamMessage()] };
+    const prevState = narrowsReducer(initialState, action1);
+    expect(prevState).not.toEqual(initialState);
+
+    expect(narrowsReducer(prevState, eg.action.reset_account_data)).toEqual(initialState);
+  });
+
   describe('EVENT_NEW_MESSAGE', () => {
     test('if not caught up in narrow, do not add message in home narrow', () => {
       const message = eg.streamMessage({ id: 3, flags: [] });

src/drafts/__tests__/draftsReducer-test.js

@@ -11,6 +11,15 @@ describe('draftsReducer', () => {
   const testNarrow = topicNarrow(eg.stream.stream_id, 'denmark2');
   const testNarrowStr = keyFromNarrow(testNarrow);
 
+  describe('RESET_ACCOUNT_DATA', () => {
+    const initialState = eg.baseReduxState.drafts;
+    const action1 = { type: DRAFT_UPDATE, content: 'Hello', narrow: testNarrow };
+    const prevState = draftsReducer(initialState, action1);
+    expect(prevState).not.toEqual(initialState);
+
+    expect(draftsReducer(prevState, eg.action.reset_account_data)).toEqual(initialState);
+  });
+
   describe('DRAFT_UPDATE', () => {
     test('add a new draft key drafts', () => {
       const prevState = NULL_OBJECT;

src/message/__tests__/messagesReducer-test.js

@@ -17,6 +17,16 @@ import { makeUserId } from '../../api/idTypes';
 import { randString } from '../../utils/misc';
 
 describe('messagesReducer', () => {
+  test('RESET_ACCOUNT_DATA', () => {
+    expect(
+      messagesReducer(
+        eg.makeMessagesState([eg.streamMessage()]),
+        eg.action.reset_account_data,
+        eg.baseReduxState,
+      ),
+    ).toEqual(eg.baseReduxState.messages);
+  });
+
   describe('EVENT_NEW_MESSAGE', () => {
     test('appends message to state, if any narrow is caught up to newest', () => {
       const message1 = eg.streamMessage();

src/outbox/__tests__/outboxReducer-test.js

@@ -7,6 +7,15 @@ import { MESSAGE_SEND_START } from '../../actionConstants';
 import * as eg from '../../__tests__/lib/exampleData';
 
 describe('outboxReducer', () => {
+  describe('RESET_ACCOUNT_DATA', () => {
+    const initialState = eg.baseReduxState.outbox;
+    const action1 = { type: MESSAGE_SEND_START, outbox: eg.streamOutbox({}) };
+    const prevState = outboxReducer(initialState, action1);
+    expect(prevState).not.toEqual(initialState);
+
+    expect(outboxReducer(prevState, eg.action.reset_account_data)).toEqual(initialState);
+  });
+
   describe('REGISTER_COMPLETE', () => {
     test('filters out isSent', () => {
       const message1 = eg.streamOutbox({ content: 'New one' });

src/pm-conversations/__tests__/pmConversationsModel-test.js

@@ -23,6 +23,10 @@ describe('reducer', () => {
   const someKey = keyOfExactUsers([eg.makeUser().user_id]);
   const someState = { map: Immutable.Map([[someKey, 123]]), sorted: Immutable.List([someKey]) };
 
+  test('RESET_ACCOUNT_DATA', () => {
+    expect(reducer(someState, eg.action.reset_account_data)).toEqual(initialState);
+  });
+
   describe('REGISTER_COMPLETE', () => {
     test('no data (old server)', () => {
       /* eslint-disable-next-line no-unused-vars */

src/typing/__tests__/typingReducer-test.js

@@ -32,6 +32,20 @@ describe('typingReducer', () => {
       : { ...base, op: 'stop', type: EVENT_TYPING_STOP };
   };
 
+  describe('RESET_ACCOUNT_DATA', () => {
+    const initialState = eg.baseReduxState.typing;
+    const action1 = egTypingAction({
+      op: 'start',
+      sender: user1,
+      recipients: [user1, user2],
+      time: 123456789,
+    });
+    const prevState = typingReducer(initialState, action1);
+    expect(prevState).not.toEqual(initialState);
+
+    expect(typingReducer(prevState, eg.action.reset_account_data)).toEqual(initialState);
+  });
+
   describe('EVENT_TYPING_START', () => {
     test('adds sender as currently typing user', () => {
       const prevState = NULL_OBJECT;