model tests: Fill in coverage of all interesting REGISTER_COMPLETE handlers

Author: chrisbobbe

src/caughtup/__tests__/caughtUpReducer-test.js

@@ -25,6 +25,18 @@ describe('caughtUpReducer', () => {
     expect(caughtUpReducer(prevState, eg.action.reset_account_data)).toEqual(initialState);
   });
 
+  describe('REGISTER_COMPLETE', () => {
+    const initialState = eg.baseReduxState.caughtUp;
+    const prevState = caughtUpReducer(initialState, {
+      ...eg.action.message_fetch_complete,
+      foundNewest: true,
+      foundOldest: true,
+    });
+    expect(prevState).not.toEqual(initialState);
+
+    expect(caughtUpReducer(prevState, eg.action.register_complete)).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__/flagsReducer-test.js

@@ -7,6 +7,17 @@ import flagsReducer from '../flagsReducer';
 import { EVENT_UPDATE_MESSAGE_FLAGS } from '../../actionConstants';
 
 describe('flagsReducer', () => {
+  describe('REGISTER_COMPLETE', () => {
+    const initialState = eg.baseReduxState.flags;
+    const prevState = flagsReducer(
+      initialState,
+      eg.mkActionEventNewMessage(eg.streamMessage({ flags: ['read', 'starred'] })),
+    );
+    expect(prevState).not.toEqual(initialState);
+
+    expect(flagsReducer(prevState, eg.action.register_complete)).toEqual(initialState);
+  });
+
   describe('MESSAGE_FETCH_COMPLETE', () => {
     test('flags from all messages are extracted and stored by id', () => {
       const message1 = eg.streamMessage();

src/chat/__tests__/narrowsReducer-test.js

@@ -39,6 +39,17 @@ describe('narrowsReducer', () => {
     expect(narrowsReducer(prevState, eg.action.reset_account_data)).toEqual(initialState);
   });
 
+  describe('REGISTER_COMPLETE', () => {
+    const initialState = eg.baseReduxState.narrows;
+    const prevState = narrowsReducer(initialState, {
+      ...eg.action.message_fetch_complete,
+      messages: [eg.streamMessage()],
+    });
+    expect(prevState).not.toEqual(initialState);
+
+    expect(narrowsReducer(prevState, eg.action.register_complete)).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/message/__tests__/messagesReducer-test.js

@@ -27,6 +27,20 @@ describe('messagesReducer', () => {
     ).toEqual(eg.baseReduxState.messages);
   });
 
+  describe('REGISTER_COMPLETE', () => {
+    const initialState = eg.baseReduxState.messages;
+    const prevState = messagesReducer(
+      initialState,
+      { ...eg.action.message_fetch_complete, messages: [eg.streamMessage()] },
+      eg.baseReduxState,
+    );
+    expect(prevState).not.toEqual(initialState);
+
+    expect(messagesReducer(prevState, eg.action.register_complete, eg.baseReduxState)).toEqual(
+      initialState,
+    );
+  });
+
   describe('EVENT_NEW_MESSAGE', () => {
     test('appends message to state, if any narrow is caught up to newest', () => {
       const message1 = eg.streamMessage();

src/streams/__tests__/streamsReducer-test.js

@@ -19,6 +19,16 @@ describe('streamsReducer', () => {
     });
   });
 
+  describe('REGISTER_COMPLETE', () => {
+    test('stores initial streams data', () => {
+      const streams = [eg.makeStream(), eg.makeStream()];
+
+      const prevState = eg.baseReduxState.streams;
+      const action = eg.mkActionRegisterComplete({ streams });
+      expect(streamsReducer(prevState, action)).toEqual(streams);
+    });
+  });
+
   describe('EVENT -> stream -> create', () => {
     test('add new stream', () => {
       const stream1 = eg.makeStream({ name: 'some stream', stream_id: 1 });