ActionSheet [nfc]: Update signature of actionsheet to not use streamName.

Also involves necessary changes at call sites, and some actionsheet
functions.

Author: AkashDhiman

src/message/__tests__/messageActionSheet-test.js

@@ -1,7 +1,6 @@
 // @flow strict-local
 import deepFreeze from 'deep-freeze';
 import { HOME_NARROW } from '../../utils/narrow';
-import { streamNameOfStreamMessage } from '../../utils/recipient';
 
 import * as eg from '../../__tests__/lib/exampleData';
 import { constructMessageActionButtons, constructTopicActionButtons } from '../messageActionSheet';
@@ -62,10 +61,11 @@ describe('constructActionButtons', () => {
 });
 
 describe('constructTopicActionButtons', () => {
-  const streamMessage = eg.streamMessage();
-  const streamName = streamNameOfStreamMessage(streamMessage);
+  const stream = eg.makeStream();
+  const streamMessage = eg.streamMessage({ stream });
   const topic = streamMessage.subject;
   const streamId = streamMessage.stream_id;
+  const streams = deepFreeze(new Map([[streamMessage.stream_id, stream]]));
 
   const baseState = (() => {
     const r = (state, action) => reducer(state, action, eg.plusReduxState);
@@ -77,8 +77,7 @@ describe('constructTopicActionButtons', () => {
   test('show mark as read if topic is unread', () => {
     const unreadStreams = baseState.streams;
     const buttons = constructTopicActionButtons({
-      backgroundData: { ...baseBackgroundData, unreadStreams },
-      streamName,
+      backgroundData: { ...baseBackgroundData, streams, unreadStreams },
       streamId,
       topic,
     });
@@ -87,19 +86,17 @@ describe('constructTopicActionButtons', () => {
 
   test('do not show mark as read if topic is read', () => {
     const buttons = constructTopicActionButtons({
-      backgroundData: baseBackgroundData,
-      streamName,
+      backgroundData: { ...baseBackgroundData, streams },
       streamId,
       topic,
     });
     expect(buttonTitles(buttons)).not.toContain('Mark topic as read');
   });
 
   test('show Unmute topic option if topic is muted', () => {
-    const mute = deepFreeze([[streamName, topic]]);
+    const mute = deepFreeze([[stream.name, topic]]);
     const buttons = constructTopicActionButtons({
-      backgroundData: { ...baseBackgroundData, mute },
-      streamName,
+      backgroundData: { ...baseBackgroundData, streams, mute },
       streamId,
       topic,
     });
@@ -108,30 +105,27 @@ describe('constructTopicActionButtons', () => {
 
   test('show mute topic option if topic is not muted', () => {
     const buttons = constructTopicActionButtons({
-      backgroundData: { ...baseBackgroundData, mute: [] },
-      streamName,
+      backgroundData: { ...baseBackgroundData, streams, mute: [] },
       streamId,
       topic,
     });
     expect(buttonTitles(buttons)).toContain('Mute topic');
   });
 
   test('show Unmute stream option if stream is not in home view', () => {
-    const subscriptions = [{ ...eg.subscription, in_home_view: false }];
+    const subscriptions = [{ ...eg.subscription, in_home_view: false, ...stream }];
     const buttons = constructTopicActionButtons({
-      backgroundData: { ...baseBackgroundData, subscriptions },
-      streamName,
+      backgroundData: { ...baseBackgroundData, subscriptions, streams },
       streamId,
       topic,
     });
     expect(buttonTitles(buttons)).toContain('Unmute stream');
   });
 
   test('show mute stream option if stream is in home view', () => {
-    const subscriptions = [{ ...eg.subscription, in_home_view: true }];
+    const subscriptions = [{ ...eg.subscription, in_home_view: true, ...stream }];
     const buttons = constructTopicActionButtons({
-      backgroundData: { ...baseBackgroundData, subscriptions },
-      streamName,
+      backgroundData: { ...baseBackgroundData, subscriptions, streams },
       streamId,
       topic,
     });
@@ -141,8 +135,7 @@ describe('constructTopicActionButtons', () => {
   test('show delete topic option if current user is an admin', () => {
     const ownUser = { ...eg.selfUser, is_admin: true };
     const buttons = constructTopicActionButtons({
-      backgroundData: { ...baseBackgroundData, ownUser },
-      streamName,
+      backgroundData: { ...baseBackgroundData, ownUser, streams },
       streamId,
       topic,
     });
@@ -151,8 +144,7 @@ describe('constructTopicActionButtons', () => {
 
   test('do not show delete topic option if current user is not an admin', () => {
     const buttons = constructTopicActionButtons({
-      backgroundData: baseBackgroundData,
-      streamName,
+      backgroundData: { ...baseBackgroundData, streams },
       streamId,
       topic,
     });

src/message/messageActionSheet.js

@@ -1,5 +1,6 @@
 /* @flow strict-local */
 import { Clipboard, Share, Alert } from 'react-native';
+import invariant from 'invariant';
 
 import * as NavigationService from '../nav/NavigationService';
 import type {
@@ -39,7 +40,6 @@ export type ShowActionSheetWithOptions = (
 
 type TopicArgs = {
   auth: Auth,
-  streamName: string,
   streamId: number,
   topic: string,
   subscriptions: Subscription[],
@@ -228,7 +228,6 @@ cancel.errorMessage = 'Failed to hide menu';
 
 export const constructTopicActionButtons = ({
   backgroundData: { mute, ownUser, streams, subscriptions, unreadStreams },
-  streamName,
   streamId,
   topic,
 }: {|
@@ -240,7 +239,6 @@ export const constructTopicActionButtons = ({
     ownUser: User,
     ...
   }>,
-  streamName: string,
   streamId: number,
   topic: string,
 |}): Button<TopicArgs>[] => {
@@ -252,12 +250,14 @@ export const constructTopicActionButtons = ({
   if (unreadCount !== undefined && unreadCount > 0) {
     buttons.push(markTopicAsRead);
   }
-  if (isTopicMuted(streamName, topic, mute)) {
+  const stream = streams.get(streamId);
+  invariant(stream !== undefined, 'Stream with provided streamId not found.');
+  if (isTopicMuted(stream.name, topic, mute)) {
     buttons.push(unmuteTopic);
   } else {
     buttons.push(muteTopic);
   }
-  const sub = subscriptions.find(x => x.name === streamName);
+  const sub = subscriptions.find(x => x.stream_id === streamId);
   if (sub && !sub.in_home_view) {
     buttons.push(unmuteStream);
   } else {
@@ -402,7 +402,6 @@ export const showTopicActionSheet = ({
   callbacks,
   backgroundData,
   topic,
-  streamName,
   streamId,
 }: {|
   showActionSheetWithOptions: ShowActionSheetWithOptions,
@@ -420,26 +419,25 @@ export const showTopicActionSheet = ({
     flags: FlagsState,
     ...
   }>,
-  streamName: string,
   streamId: number,
   topic: string,
 |}): void => {
   const buttonList = constructTopicActionButtons({
     backgroundData,
-    streamName,
     streamId,
     topic,
   });
+  const stream = backgroundData.streams.get(streamId);
+  invariant(stream !== undefined, 'Stream with provided streamId not found.');
   showActionSheetWithOptions(
     {
-      title: `#${streamName} > ${topic}`,
+      title: `#${stream.name} > ${topic}`,
       options: buttonList.map(button => callbacks._(button.title)),
       cancelButtonIndex: buttonList.length - 1,
     },
     makeButtonCallback(buttonList, {
       ...backgroundData,
       ...callbacks,
-      streamName,
       streamId,
       topic,
     }),

src/streams/TopicItem.js

@@ -72,7 +72,6 @@ export default function TopicItem(props: Props) {
           showActionSheetWithOptions,
           callbacks: { dispatch, _ },
           backgroundData,
-          streamName,
           streamId: currentStream[0],
           topic: name,
         });

src/title/TitleStream.js

@@ -83,7 +83,6 @@ const TitleStream = (props: Props) => {
                 showActionSheetWithOptions,
                 callbacks: { dispatch, _ },
                 backgroundData,
-                streamName: stream.name,
                 streamId: stream.stream_id,
                 topic: topicOfNarrow(narrow),
               });

src/webview/handleOutboundEvents.js

@@ -224,7 +224,6 @@ const handleLongPress = (
         showActionSheetWithOptions,
         callbacks: { dispatch, _ },
         backgroundData,
-        streamName,
         streamId: currentStream[0],
         topic: message.subject,
       });