model: Remove various fallbacks for ancient, sometimes prehistoric servers

This completes all our `TODO(#5102)`s.

Author: chrisbobbe

src/alertWords/__tests__/alertWordsReducer-test.js

@@ -26,26 +26,6 @@ describe('alertWordsReducer', () => {
         ),
       ).toEqual(['word', '@mobile-core', 'alert']);
     });
-
-    // TODO(#5102): Delete; see comment on implementation.
-    test('when no `alert_words` data is given reset state', () => {
-      const prevState = deepFreeze(['word']);
-      const actualState = alertWordsReducer(
-        prevState,
-        eg.mkActionRegisterComplete({
-          // Hmm, we should need a Flow suppression here. This property is
-          // marked required in InitialData, and this explicit undefined is
-          // meant to defy that; see TODO(#5102) above.
-          // mkActionRegisterComplete is designed to accept input with this or
-          // any property *omitted*… and I think, as a side effect of handling
-          // that, Flow mistakenly accepts an explicit undefined here, so it
-          // doesn't catch the resulting malformed InitialData.
-          alert_words: undefined,
-        }),
-      );
-
-      expect(actualState).toEqual([]);
-    });
   });
 
   describe('EVENT_ALERT_WORDS', () => {

src/alertWords/alertWordsReducer.js

@@ -14,15 +14,7 @@ export default (
       return initialState;
 
     case REGISTER_COMPLETE:
-      return (
-        action.data.alert_words
-        // TODO(#5102): Delete fallback once we enforce any threshold for
-        //   ancient servers we refuse to connect to. It was added in #2878
-        //   (2018-11-16), but it wasn't clear even then, it seems, whether
-        //   any servers actually omit the data. The API doc doesn't mention
-        //   any servers that omit it, and our Flow types mark it required.
-        || initialState
-      );
+      return action.data.alert_words;
 
     case EVENT_ALERT_WORDS:
       return action.alert_words || initialState;

src/api/initialDataTypes.js

@@ -419,10 +419,6 @@ export type InitialDataRealmUser = $ReadOnly<{|
 |}>;
 
 export type InitialDataRealmUserGroups = $ReadOnly<{|
-  // New in Zulip 1.8.
-  // TODO(#5102): In userGroupsReducer, we still have a fallback for pre-1.8
-  //   servers; remove that, and remove the above comment, which will be
-  //   irrelevant.
   realm_user_groups: $ReadOnlyArray<UserGroup>,
 |}>;
 

src/unread/unreadHuddlesReducer.js

@@ -94,13 +94,7 @@ export default (
       return initialState;
 
     case REGISTER_COMPLETE:
-      return (
-        (action.data.unread_msgs && action.data.unread_msgs.huddles)
-        // TODO(#5102): Delete fallback once we refuse to connect to Zulip
-        //   servers before 1.7.0, when it seems this feature was added; see
-        //   comment on InitialDataUpdateMessageFlags.
-        || initialState
-      );
+      return action.data.unread_msgs.huddles;
 
     case EVENT_NEW_MESSAGE:
       return eventNewMessage(state, action);

src/unread/unreadMentionsReducer.js

@@ -52,13 +52,7 @@ export default (
       return initialState;
 
     case REGISTER_COMPLETE:
-      return (
-        (action.data.unread_msgs && action.data.unread_msgs.mentions)
-        // TODO(#5102): Delete fallback once we refuse to connect to Zulip
-        //   servers before 1.7.0, when it seems this feature was added; see
-        //   comment on InitialDataUpdateMessageFlags.
-        || initialState
-      );
+      return action.data.unread_msgs.mentions;
 
     case EVENT_NEW_MESSAGE: {
       const { flags } = action.message;

src/unread/unreadModel.js

@@ -129,11 +129,7 @@ function streamsReducer(
       return initialStreamsState;
 
     case REGISTER_COMPLETE: {
-      // TODO(#5102): Delete fallback once we refuse to connect to Zulip
-      //   servers before 1.7.0, when it seems this feature was added; see
-      //   comment on InitialDataUpdateMessageFlags.
-      // flowlint-next-line unnecessary-optional-chain:off
-      const data = action.data.unread_msgs?.streams ?? [];
+      const data = action.data.unread_msgs.streams;
 
       // First, collect together all the data for a given stream, just in a
       // plain old Array.

src/unread/unreadPmsReducer.js

@@ -94,13 +94,7 @@ export default (
       return initialState;
 
     case REGISTER_COMPLETE:
-      return (
-        (action.data.unread_msgs && action.data.unread_msgs.pms)
-        // TODO(#5102): Delete fallback once we refuse to connect to Zulip
-        //   servers before 1.7.0, when it seems this feature was added; see
-        //   comment on InitialDataUpdateMessageFlags.
-        || initialState
-      );
+      return action.data.unread_msgs.pms;
 
     case EVENT_NEW_MESSAGE:
       return eventNewMessage(state, action);

src/user-groups/__tests__/userGroupsReducer-test.js

@@ -22,26 +22,6 @@ describe('userGroupsReducer', () => {
         userGroupsReducer(prevState, eg.mkActionRegisterComplete({ realm_user_groups: [group] })),
       ).toEqual([group]);
     });
-
-    // TODO(#5102): Remove this test case, which is for pre-1.8 servers.
-    test('when no data is given reset state', () => {
-      const prevState = deepFreeze([eg.makeUserGroup()]);
-      expect(
-        userGroupsReducer(
-          prevState,
-          eg.mkActionRegisterComplete({
-            // Hmm, we should need a Flow suppression here. This property is
-            // marked required in InitialData, and this explicit undefined is
-            // meant to defy that; see TODO(#5102) above.
-            // mkActionRegisterComplete is designed to accept input with this or
-            // any property *omitted*… and I think, as a side effect of handling
-            // that, Flow mistakenly accepts an explicit undefined here, so it
-            // doesn't catch the resulting malformed InitialData.
-            realm_user_groups: undefined,
-          }),
-        ),
-      ).toEqual([]);
-    });
   });
 
   describe('RESET_ACCOUNT_DATA', () => {

src/user-groups/userGroupsReducer.js

@@ -52,8 +52,7 @@ export default (
       return initialState;
 
     case REGISTER_COMPLETE:
-      // TODO(#5102): Remove fallback for pre-1.8 servers
-      return action.data.realm_user_groups || initialState;
+      return action.data.realm_user_groups;
 
     case EVENT_USER_GROUP_ADD:
       return [...state, action.group];