stream-chat-react-native-core 9.5.0-beta.7 → 9.5.0-beta.9

package/src/components/ChannelDetails/__tests__/{ChannelEditDetails.test.tsx → ChannelEditDetailsFormContent.test.tsx}

@@ -17,7 +17,7 @@ import { TranslationProvider } from '../../../contexts/translationContext/Transl
 import { generateFileReference } from '../../../mock-builders/attachments';
 import { NativeHandlers } from '../../../native';
 import { EditChannelDetailsStore } from '../../../state-store/edit-channel-details-store';
-import { ChannelEditDetails } from '../components/ChannelEditDetails';
+import { ChannelEditDetailsFormContent } from '../components/ChannelEditDetailsFormContent';
 import type { ChannelEditImageSheetProps } from '../components/ChannelEditImageSheet';
 
 type SheetProbeRecord = ChannelEditImageSheetProps;
@@ -80,10 +80,10 @@ const renderComponent = ({ channel }: { channel: Channel }) => {
             { client: { on: () => ({ unsubscribe: () => undefined }), userID: 'me' } } as never
           }
         >
-          <ChannelDetailsContextProvider value={{ channel }}>
-            <ChannelEditDetailsContext.Provider value={{ store }}>
+          <ChannelDetailsContextProvider channel={channel}>
+            <ChannelEditDetailsContext.Provider value={{ store, submit: jest.fn() }}>
               <WithComponents overrides={{ ChannelEditImageSheet: SheetProbe }}>
-                <ChannelEditDetails />
+                <ChannelEditDetailsFormContent />
               </WithComponents>
             </ChannelEditDetailsContext.Provider>
           </ChannelDetailsContextProvider>
@@ -96,7 +96,7 @@ const renderComponent = ({ channel }: { channel: Channel }) => {
 
 const latestSheetProps = () => sheetCalls[sheetCalls.length - 1];
 
-describe('ChannelEditDetails', () => {
+describe('ChannelEditDetailsFormContent', () => {
   beforeEach(() => {
     sheetCalls.length = 0;
   });
@@ -128,14 +128,14 @@ describe('ChannelEditDetails', () => {
               { client: { on: () => ({ unsubscribe: () => undefined }), userID: 'me' } } as never
             }
           >
-            <ChannelDetailsContextProvider value={{ channel: buildChannel() }}>
+            <ChannelDetailsContextProvider channel={buildChannel()}>
               <ChannelEditDetailsContext.Provider
-                value={{ store: new EditChannelDetailsStore(buildChannel()) }}
+                value={{ store: new EditChannelDetailsStore(buildChannel()), submit: jest.fn() }}
               >
                 <WithComponents
                   overrides={{ ChannelEditImageSheet: SheetProbe, ChannelEditName: NameProbe }}
                 >
-                  <ChannelEditDetails />
+                  <ChannelEditDetailsFormContent />
                 </WithComponents>
               </ChannelEditDetailsContext.Provider>
             </ChannelDetailsContextProvider>

package/src/components/ChannelDetails/__tests__/ChannelEditImageSheet.test.tsx

@@ -97,7 +97,7 @@ const renderSheet = ({
           userLanguage: 'en',
         }}
       >
-        <ChannelEditDetailsContext.Provider value={{ store }}>
+        <ChannelEditDetailsContext.Provider value={{ store, submit: jest.fn() }}>
           <WithComponents overrides={{ ChannelDetailsActionItem: ActionItemProbe }}>
             <ChannelEditImageSheet onClose={onClose} visible={visible} />
           </WithComponents>

package/src/components/ChannelDetails/__tests__/ChannelEditName.test.tsx

@@ -37,7 +37,7 @@ const renderComponent = ({ channel }: { channel: Channel }) => {
             { client: { on: () => ({ unsubscribe: () => undefined }), userID: 'me' } } as never
           }
         >
-          <ChannelEditDetailsContext.Provider value={{ store }}>
+          <ChannelEditDetailsContext.Provider value={{ store, submit: jest.fn() }}>
             <ChannelEditName />
           </ChannelEditDetailsContext.Provider>
         </ChatContext.Provider>

package/src/components/ChannelDetails/__tests__/members/ChannelAddMembersButton.test.tsx

@@ -0,0 +1,163 @@
+import React from 'react';
+import { StyleSheet } from 'react-native';
+
+import { fireEvent, render, screen } from '@testing-library/react-native';
+
+type ReactTestInstance = ReturnType<typeof screen.getByTestId>;
+import type { Channel } from 'stream-chat';
+
+import { AccessibilityProvider } from '../../../../contexts/accessibilityContext/AccessibilityContext';
+import { ChannelDetailsContextProvider } from '../../../../contexts/channelDetailsContext/channelDetailsContext';
+import {
+  allOwnCapabilities,
+  OwnCapabilitiesContextValue,
+  OwnCapability,
+} from '../../../../contexts/ownCapabilitiesContext/OwnCapabilitiesContext';
+import { ThemeProvider } from '../../../../contexts/themeContext/ThemeContext';
+import { defaultTheme } from '../../../../contexts/themeContext/utils/theme';
+import { TranslationProvider } from '../../../../contexts/translationContext/TranslationContext';
+import { ChannelAddMembersButton } from '../../components/members/ChannelAddMembersButton';
+
+// Replace the modal wrapper with a lightweight probe that renders regardless of `visible`
+// (and ignores its children, so the real add-members form never mounts). This lets tests
+// distinguish "modal mounted" from "modal not mounted" and observe its open/closed state.
+jest.mock('../../components/modal/Modal', () => ({
+  ChannelDetailsModal: ({ visible }: { visible: boolean }) => {
+    const { Text: RNText } = require('react-native');
+    return <RNText testID='add-members-modal'>{visible ? 'visible' : 'hidden'}</RNText>;
+  },
+}));
+
+const buildChannel = (overrides?: Partial<OwnCapabilitiesContextValue>): Channel => {
+  const ownCapabilities = overrides
+    ? Object.entries(overrides)
+        .filter(([, enabled]) => enabled)
+        .map(([key]) => allOwnCapabilities[key as OwnCapability])
+    : undefined;
+  return {
+    cid: 'messaging:test',
+    data: ownCapabilities ? { own_capabilities: ownCapabilities } : {},
+    on: () => ({ unsubscribe: () => undefined }),
+    state: { members: {} },
+  } as unknown as Channel;
+};
+
+const TEST_ID = 'channel-details-add-members-button';
+
+// Walks up the rendered tree to confirm some host ancestor received `width: 'auto'`
+// (the forwarded style overriding the Button's default `width: '100%'`).
+const hasAncestorWithStyle = (instance: ReactTestInstance | null): boolean => {
+  for (let node = instance; node; node = node.parent) {
+    if (typeof node.type === 'string') {
+      const flattened = StyleSheet.flatten(node.props.style) ?? {};
+      if (flattened.width === 'auto') {
+        return true;
+      }
+    }
+  }
+  return false;
+};
+
+const renderButton = ({
+  capabilities,
+  onPress,
+  style,
+  variant,
+}: {
+  capabilities?: Partial<OwnCapabilitiesContextValue>;
+  onPress?: () => void;
+  style?: React.ComponentProps<typeof ChannelAddMembersButton>['style'];
+  variant?: 'icon' | 'text';
+} = {}) =>
+  render(
+    <ThemeProvider theme={defaultTheme}>
+      <AccessibilityProvider value={{ enabled: true }}>
+        <TranslationProvider
+          value={{
+            t: ((key: string) => key) as never,
+            tDateTimeParser: ((input: unknown) => input) as never,
+            userLanguage: 'en',
+          }}
+        >
+          <ChannelDetailsContextProvider channel={buildChannel(capabilities)}>
+            <ChannelAddMembersButton
+              onPress={onPress}
+              style={style}
+              testID={TEST_ID}
+              variant={variant}
+            />
+          </ChannelDetailsContextProvider>
+        </TranslationProvider>
+      </AccessibilityProvider>
+    </ThemeProvider>,
+  );
+
+describe('ChannelAddMembersButton', () => {
+  it('renders nothing when the user lacks the update-channel-members capability', () => {
+    renderButton();
+
+    expect(screen.queryByTestId(TEST_ID)).toBeNull();
+  });
+
+  it('renders the text variant when the user has the capability', () => {
+    renderButton({ capabilities: { updateChannelMembers: true }, variant: 'text' });
+
+    expect(screen.getByTestId(TEST_ID)).toBeTruthy();
+    expect(screen.getByText('Add')).toBeTruthy();
+  });
+
+  it('renders the icon variant when the user has the capability', () => {
+    renderButton({ capabilities: { updateChannelMembers: true }, variant: 'icon' });
+
+    expect(screen.getByTestId(TEST_ID)).toBeTruthy();
+    // The icon variant has no visible label.
+    expect(screen.queryByText('Add')).toBeNull();
+  });
+
+  it('defaults to the text variant', () => {
+    renderButton({ capabilities: { updateChannelMembers: true } });
+
+    expect(screen.getByText('Add')).toBeTruthy();
+  });
+
+  it.each(['text', 'icon'] as const)(
+    'forwards the style prop to the underlying Button (%s variant)',
+    (variant) => {
+      renderButton({
+        capabilities: { updateChannelMembers: true },
+        style: { width: 'auto' },
+        variant,
+      });
+
+      // The style lands on the Button's outer wrapper View, a host ancestor of the
+      // testID-carrying Pressable. Search ancestors for the one that received it.
+      expect(hasAncestorWithStyle(screen.getByTestId(TEST_ID))).toBe(true);
+    },
+  );
+
+  it('invokes the onPress override and does not open the modal', () => {
+    const onPress = jest.fn();
+
+    renderButton({ capabilities: { updateChannelMembers: true }, onPress });
+
+    fireEvent.press(screen.getByTestId(TEST_ID));
+
+    expect(onPress).toHaveBeenCalledTimes(1);
+  });
+
+  it('mounts the built-in modal and opens it on press when no override is provided', () => {
+    renderButton({ capabilities: { updateChannelMembers: true } });
+
+    expect(screen.getByTestId('add-members-modal')).toHaveTextContent('hidden');
+
+    fireEvent.press(screen.getByTestId(TEST_ID));
+
+    expect(screen.getByTestId('add-members-modal')).toHaveTextContent('visible');
+  });
+
+  it('does not mount the built-in modal when a custom onPress is provided', () => {
+    renderButton({ capabilities: { updateChannelMembers: true }, onPress: jest.fn() });
+
+    expect(screen.queryByTestId('add-members-modal')).toBeNull();
+  });
+});

package/src/components/ChannelDetails/__tests__/members/{ChannelAddMembersModal.test.tsx → ChannelAddMembersForm.test.tsx}

@@ -21,13 +21,13 @@ import { TranslationProvider } from '../../../../contexts/translationContext/Tra
 import { useChannelActions } from '../../../../hooks/actions/useChannelActions';
 import { generateMember } from '../../../../mock-builders/generator/member';
 import { generateUser } from '../../../../mock-builders/generator/user';
-import { ChannelAddMembersModal } from '../../components/members/ChannelAddMembersModal';
+import { ChannelAddMembersForm } from '../../components/members/ChannelAddMembersForm';
 
 jest.mock('../../../../hooks/actions/useChannelActions');
 const mockedUseChannelActions = jest.mocked(useChannelActions);
 
-// Stands in for the real ChannelAddMembers list: drives the shared selection store
-// directly instead of running the search source / list.
+// Stands in for the real ChannelAddMembersFormContent list: drives the shared
+// selection store directly instead of running the search source / list.
 const AddMembersProbe = () => {
   const { selectionStore } = useChannelAddMembersContext();
   return (
@@ -76,16 +76,14 @@ const applyCapabilities = (
   return channel;
 };
 
-const renderModal = ({
+const renderForm = ({
   capabilities,
   channel,
   onClose = jest.fn(),
-  visible = true,
 }: {
   channel: Channel;
   capabilities?: Partial<OwnCapabilitiesContextValue>;
   onClose?: () => void;
-  visible?: boolean;
 }) =>
   render(
     <ThemeProvider theme={defaultTheme}>
@@ -100,11 +98,9 @@ const renderModal = ({
           <ChatContext.Provider
             value={{ client: { notifications: new NotificationManager(), userID: 'me' } } as never}
           >
-            <ChannelDetailsContextProvider
-              value={{ channel: applyCapabilities(channel, capabilities) }}
-            >
-              <WithComponents overrides={{ ChannelAddMembers: AddMembersProbe }}>
-                <ChannelAddMembersModal onClose={onClose} visible={visible} />
+            <ChannelDetailsContextProvider channel={applyCapabilities(channel, capabilities)}>
+              <WithComponents overrides={{ ChannelAddMembersFormContent: AddMembersProbe }}>
+                <ChannelAddMembersForm onClose={onClose} />
               </WithComponents>
             </ChannelDetailsContextProvider>
           </ChatContext.Provider>
@@ -118,13 +114,11 @@ const makeMembers = (count: number) =>
     generateMember({ user: generateUser({ id: `u-${idx}`, name: `User ${idx}` }) }),
   );
 
-describe('ChannelAddMembersModal', () => {
+describe('ChannelAddMembersForm', () => {
   let addMembersSpy: jest.Mock;
 
   beforeEach(() => {
-    addMembersSpy = jest.fn(async (_ids: string[], options?: { onSuccess?: () => unknown }) => {
-      await options?.onSuccess?.();
-    });
+    addMembersSpy = jest.fn().mockResolvedValue(undefined);
     mockedUseChannelActions.mockReturnValue({
       addMembers: addMembersSpy,
     } as unknown as ReturnType<typeof useChannelActions>);
@@ -135,10 +129,10 @@ describe('ChannelAddMembersModal', () => {
     mockedUseChannelActions.mockReset();
   });
 
-  it('enables the confirm button only while ChannelAddMembers reports a selection', () => {
+  it('enables the confirm button only while the form content reports a selection', () => {
     const channel = buildChannel(makeMembers(3), 3);
 
-    renderModal({ channel });
+    renderForm({ channel });
 
     const confirm = screen.getByTestId('channel-details-add-members-confirm-button');
     expect(confirm.props.accessibilityState).toMatchObject({ disabled: true });
@@ -154,11 +148,11 @@ describe('ChannelAddMembersModal', () => {
     ).toMatchObject({ disabled: true });
   });
 
-  it('calls addMembers from useChannelActions with the selected user ids and closes the sheet on confirm', async () => {
+  it('calls addMembers from useChannelActions with the selected user ids and closes the form on confirm', async () => {
     const channel = buildChannel(makeMembers(3), 3);
     const onClose = jest.fn();
 
-    renderModal({ channel, onClose });
+    renderForm({ channel, onClose });
 
     fireEvent.press(screen.getByTestId('probe-select-one'));
 
@@ -169,7 +163,7 @@ describe('ChannelAddMembersModal', () => {
 
     expect(addMembersSpy).toHaveBeenCalledWith(
       ['picked-1'],
-      expect.objectContaining({ onSuccess: expect.any(Function) }),
+      expect.objectContaining({ onFailure: expect.any(Function) }),
     );
     expect(channel.addMembers).not.toHaveBeenCalled();
     await waitFor(() => expect(onClose).toHaveBeenCalledTimes(1));
@@ -179,19 +173,23 @@ describe('ChannelAddMembersModal', () => {
     const channel = buildChannel(makeMembers(3), 3);
     const onClose = jest.fn();
 
-    renderModal({ channel, onClose });
+    renderForm({ channel, onClose });
 
     fireEvent.press(screen.getByLabelText('a11y/Close'));
     expect(onClose).toHaveBeenCalledTimes(1);
   });
 
-  it('keeps the sheet open and re-enables confirm when addMembers does not invoke onSuccess', async () => {
+  it('keeps the form open and re-enables confirm when addMembers invokes onFailure', async () => {
     const warnSpy = jest.spyOn(console, 'warn').mockImplementation(() => undefined);
-    addMembersSpy.mockResolvedValueOnce(undefined);
+    addMembersSpy.mockImplementationOnce(
+      async (_ids: string[], options?: { onFailure?: (error: unknown) => unknown }) => {
+        await options?.onFailure?.(new Error('failed'));
+      },
+    );
     const channel = buildChannel(makeMembers(3), 3);
     const onClose = jest.fn();
 
-    renderModal({ channel, onClose });
+    renderForm({ channel, onClose });
 
     fireEvent.press(screen.getByTestId('probe-select-one'));
 
@@ -203,7 +201,7 @@ describe('ChannelAddMembersModal', () => {
 
     expect(addMembersSpy).toHaveBeenCalledWith(
       ['picked-1'],
-      expect.objectContaining({ onSuccess: expect.any(Function) }),
+      expect.objectContaining({ onFailure: expect.any(Function) }),
     );
     expect(onClose).not.toHaveBeenCalled();
     expect(screen.getByTestId('add-members-probe')).toBeTruthy();

package/src/components/ChannelDetails/__tests__/members/{ChannelAddMembers.test.tsx → ChannelAddMembersFormContent.test.tsx}

@@ -11,7 +11,7 @@ import { TranslationProvider } from '../../../../contexts/translationContext/Tra
 import { generateUser } from '../../../../mock-builders/generator/user';
 import { SelectionStore } from '../../../../state-store/selection-store';
 import type { AddMemberSearchResultItemProps } from '../../components/members/AddMemberSearchResultItem';
-import { ChannelAddMembers } from '../../components/members/ChannelAddMembers';
+import { ChannelAddMembersFormContent } from '../../components/members/ChannelAddMembersFormContent';
 
 const mockRowProbe: AddMemberSearchResultItemProps[] = [];
 
@@ -98,13 +98,15 @@ const tree = (
       }}
     >
       <ChannelAddMembersContext.Provider value={{ searchSource, selectionStore } as never}>
-        <ChannelAddMembers additionalFlatListProps={props.additionalFlatListProps as never} />
+        <ChannelAddMembersFormContent
+          additionalFlatListProps={props.additionalFlatListProps as never}
+        />
       </ChannelAddMembersContext.Provider>
     </TranslationProvider>
   </ThemeProvider>
 );
 
-describe('ChannelAddMembers', () => {
+describe('ChannelAddMembersFormContent', () => {
   beforeEach(() => {
     mockRowProbe.length = 0;
   });

package/src/components/ChannelDetails/__tests__/members/ChannelMemberActionsSheet.test.tsx

@@ -81,7 +81,7 @@ const renderSheet = ({
             } as never
           }
         >
-          <ChannelDetailsContextProvider value={{ channel }}>
+          <ChannelDetailsContextProvider channel={channel}>
             <WithComponents overrides={{ ChannelDetailsActionItem: ActionItemProbe }}>
               <ChannelMemberActionsSheet member={member} onClose={onClose} visible={visible} />
             </WithComponents>
@@ -168,9 +168,14 @@ describe('ChannelMemberActionsSheet', () => {
               userLanguage: 'en',
             }}
           >
-            <ChannelDetailsContextProvider value={{ channel, getChannelMemberActionItems }}>
+            <ChannelDetailsContextProvider channel={channel}>
               <WithComponents overrides={{ ChannelDetailsActionItem: ActionItemProbe }}>
-                <ChannelMemberActionsSheet member={member} onClose={jest.fn()} visible />
+                <ChannelMemberActionsSheet
+                  getChannelMemberActionItems={getChannelMemberActionItems}
+                  member={member}
+                  onClose={jest.fn()}
+                  visible
+                />
               </WithComponents>
             </ChannelDetailsContextProvider>
           </TranslationProvider>

package/src/components/ChannelDetails/__tests__/members/ChannelMemberItem.test.tsx

@@ -12,8 +12,8 @@ import { defaultTheme } from '../../../../contexts/themeContext/utils/theme';
 import { TranslationProvider } from '../../../../contexts/translationContext/TranslationContext';
 import { generateMember } from '../../../../mock-builders/generator/member';
 import { generateUser } from '../../../../mock-builders/generator/user';
-import type { GetMemberRoleLabel } from '../../ChannelDetails';
 import { ChannelMemberItem } from '../../components/members/ChannelMemberItem';
+import type { GetMemberRoles } from '../../hooks/members/useMemberRoles';
 
 Dayjs.extend(relativeTime);
 
@@ -36,13 +36,13 @@ const defaultChannel = {
 const renderRow = ({
   channel = defaultChannel,
   currentUserId,
-  getMemberRoleLabel,
+  getMemberRoles,
   mutedUsers = [],
   ...props
 }: React.ComponentProps<typeof ChannelMemberItem> & {
   channel?: Channel;
   currentUserId?: string;
-  getMemberRoleLabel?: GetMemberRoleLabel;
+  getMemberRoles?: GetMemberRoles;
   mutedUsers?: Array<{ target: { id: string }; user: { id: string } }>;
 }) =>
   render(
@@ -70,8 +70,8 @@ const renderRow = ({
             } as never
           }
         >
-          <ChannelDetailsContextProvider value={{ channel, getMemberRoleLabel }}>
-            <ChannelMemberItem {...props} />
+          <ChannelDetailsContextProvider channel={channel}>
+            <ChannelMemberItem getMemberRoles={getMemberRoles} {...props} />
           </ChannelDetailsContextProvider>
         </ChatContext.Provider>
       </TranslationProvider>
@@ -96,7 +96,7 @@ describe('ChannelMemberItem accessibility', () => {
 
   it('includes the role label in the accessible label between name and status', () => {
     renderRow({
-      getMemberRoleLabel: () => 'Admin',
+      getMemberRoles: () => [{ key: 'admin', label: 'Admin' }],
       member: memberFor(),
     });
     expect(screen.getByLabelText('Alice, Admin, Offline')).toBeTruthy();
@@ -131,7 +131,7 @@ describe('ChannelMemberItem muted indicator', () => {
 describe('ChannelMemberItem large variant', () => {
   it('renders the role label in the large profile header', () => {
     renderRow({
-      getMemberRoleLabel: () => 'Admin',
+      getMemberRoles: () => [{ key: 'admin', label: 'Admin' }],
       member: memberFor(),
       size: 'lg',
     });
@@ -162,17 +162,17 @@ describe('ChannelMemberItem activity status', () => {
 });
 
 describe('ChannelMemberItem role label rendering', () => {
-  it('renders the role label string returned by useMemberRoleLabel', () => {
+  it('renders a badge for each role returned by useMemberRoles', () => {
     renderRow({
-      getMemberRoleLabel: () => 'Admin',
+      getMemberRoles: () => [{ key: 'admin', label: 'Admin' }],
       member: memberFor(),
     });
     expect(screen.getByText('Admin')).toBeTruthy();
   });
 
-  it('renders no role label when the hook returns null', () => {
+  it('renders no role badges when the hook returns an empty array', () => {
     renderRow({
-      getMemberRoleLabel: () => null,
+      getMemberRoles: () => [],
       member: memberFor(),
     });
     expect(screen.queryByText('Admin')).toBeNull();

package/src/components/ChannelDetails/__tests__/members/ChannelMemberList.test.tsx

@@ -27,6 +27,12 @@ const mockProviderProbe: { channel: unknown }[] = [];
 const mockNotificationTargetProbe: { hostId?: string; panel?: string }[] = [];
 const mockAddNotification = jest.fn();
 
+let mockMemberCount = 1;
+
+jest.mock('../../../../hooks/useChannelMemberCount', () => ({
+  useChannelMemberCount: () => mockMemberCount,
+}));
+
 jest.mock('../../../Notifications/hooks/useNotificationApi', () => ({
   useNotificationApi: () => ({ addNotification: mockAddNotification }),
 }));
@@ -66,7 +72,6 @@ jest.mock('../../../../contexts/channelMemberListContext/ChannelMemberListContex
     return children;
   },
   useChannelMemberListContext: () => ({
-    channel: mockChannel,
     searchSource: mockCurrentSearchSource,
   }),
 }));
@@ -94,6 +99,7 @@ jest.mock('../../../UIComponents/SearchInput', () => {
 });
 
 type FakeSearchSource = {
+  resetStateAndActivate: jest.Mock;
   search: jest.Mock;
   state: StateStore<
     Pick<
@@ -122,6 +128,7 @@ const makeSearchSource = (
   // The component calls state.partialNext on search input change; spy on it.
   jest.spyOn(state, 'partialNext');
   return {
+    resetStateAndActivate: jest.fn(),
     search: jest.fn(),
     state: state as FakeSearchSource['state'],
   };
@@ -147,7 +154,6 @@ const tree = (
   searchSource: FakeSearchSource,
   props: {
     additionalFlatListProps?: object;
-    onMemberPress?: (member: ChannelMemberResponse) => void;
   } = {},
 ) => {
   mockCurrentSearchSource = searchSource;
@@ -161,12 +167,7 @@ const tree = (
         }}
       >
         <ChannelDetailsContextProvider
-          value={
-            {
-              channel: mockChannel,
-              onMemberPress: props.onMemberPress,
-            } as unknown as ChannelDetailsContextValue
-          }
+          channel={mockChannel as unknown as ChannelDetailsContextValue['channel']}
         >
           <WithComponents
             overrides={{
@@ -188,6 +189,7 @@ describe('ChannelMemberList', () => {
     mockSheetProbe.length = 0;
     mockProviderProbe.length = 0;
     mockNotificationTargetProbe.length = 0;
+    mockMemberCount = 1;
   });
 
   afterEach(() => jest.clearAllMocks());
@@ -298,7 +300,7 @@ describe('ChannelMemberList', () => {
     expect(screen.queryByTestId('channel-member-list')).toBeNull();
   });
 
-  it('opens the per-member actions sheet on press when no onMemberPress override is provided, and closes it', () => {
+  it('opens the per-member actions sheet on press, and closes it', () => {
     const bob = generateMember({ user: generateUser({ id: 'bob', name: 'Bob' }) });
     render(tree(makeSearchSource({ items: [bob] })));
 
@@ -311,18 +313,6 @@ describe('ChannelMemberList', () => {
     expect(screen.queryByTestId('member-actions-sheet-probe')).toBeNull();
   });
 
-  it('calls onMemberPress instead of opening the sheet when an override is provided', () => {
-    const alice = generateMember({ user: generateUser({ id: 'alice', name: 'Alice' }) });
-    const onMemberPress = jest.fn();
-    render(tree(makeSearchSource({ items: [alice] }), { onMemberPress }));
-
-    fireEvent.press(screen.getByTestId('member-alice'));
-
-    expect(onMemberPress).toHaveBeenCalledTimes(1);
-    expect(onMemberPress.mock.calls[0][0].user?.id).toBe('alice');
-    expect(screen.queryByTestId('member-actions-sheet-probe')).toBeNull();
-  });
-
   it('targets the channel-details panel with a channel-scoped notification host', () => {
     render(tree(makeSearchSource()));
 
@@ -360,4 +350,28 @@ describe('ChannelMemberList', () => {
 
     expect(mockAddNotification).not.toHaveBeenCalled();
   });
+
+  it('refreshes the list when the member count changes and there is no search query', () => {
+    const searchSource = makeSearchSource({ searchQuery: '' });
+    const { rerender } = render(tree(searchSource));
+    expect(searchSource.search).toHaveBeenCalledTimes(1);
+    searchSource.search.mockClear();
+
+    mockMemberCount = 2;
+    rerender(tree(searchSource));
+
+    expect(searchSource.search).toHaveBeenCalledTimes(1);
+    expect(searchSource.search).toHaveBeenCalledWith();
+  });
+
+  it('does not refresh the list when the member count changes while a search query is active', () => {
+    const searchSource = makeSearchSource({ searchQuery: 'alice' });
+    const { rerender } = render(tree(searchSource));
+    searchSource.search.mockClear();
+
+    mockMemberCount = 2;
+    rerender(tree(searchSource));
+
+    expect(searchSource.search).not.toHaveBeenCalled();
+  });
 });