Skip to content

Commit c8d1ac2

Browse files
committed
fix: tests
1 parent ec5a85e commit c8d1ac2

File tree

3 files changed

+11
-11
lines changed

3 files changed

+11
-11
lines changed

package/src/components/Channel/__tests__/ownCapabilities.test.js

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -111,11 +111,11 @@ describe('Own capabilities', () => {
111111
});
112112

113113
describe(`${allOwnCapabilities.banChannelMembers} capability`, () => {
114-
// it(`should render "Ban User" message action when ${allOwnCapabilities.banChannelMembers} capability is enabled`, async () => {
115-
// await generateChannelWithCapabilities([allOwnCapabilities.banChannelMembers]);
116-
// const { queryByLabelText } = await renderChannelAndOpenMessageActionsList(receivedMessage);
117-
// expect(!!queryByLabelText('banUser action list item')).toBeTruthy();
118-
// });
114+
it(`should render "Ban User" message action when ${allOwnCapabilities.banChannelMembers} capability is enabled`, async () => {
115+
await generateChannelWithCapabilities([allOwnCapabilities.banChannelMembers]);
116+
const { queryByLabelText } = await renderChannelAndOpenMessageActionsList(receivedMessage);
117+
expect(!!queryByLabelText('banUser action list item')).toBeTruthy();
118+
});
119119

120120
it(`should not render "Ban User" message action when ${allOwnCapabilities.banChannelMembers} capability is disabled`, async () => {
121121
await generateChannelWithCapabilities();

package/src/components/Message/hooks/useMessageActions.tsx

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -276,8 +276,9 @@ export const useMessageActions = ({
276276
const banUser: MessageActionType = {
277277
action: onBanUser,
278278
actionType: 'banUser',
279-
icon: <UserDelete width={20} height={20} stroke={semantics.textSecondary} />,
279+
icon: <UserDelete width={20} height={20} stroke={semantics.accentError} />,
280280
title: message.user?.banned ? t('Unban User') : t('Ban User'),
281+
titleStyle: { color: semantics.accentError },
281282
type: 'destructive',
282283
};
283284

package/src/components/Message/utils/messageActions.ts

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ export type MessageActionsParams = {
3232
export type MessageActionsProp = (param: MessageActionsParams) => MessageActionType[];
3333

3434
export const messageActions = ({
35-
// banUser,
35+
banUser,
3636
copyMessage,
3737
deleteMessage,
3838
editMessage,
@@ -94,10 +94,9 @@ export const messageActions = ({
9494
actions.push(markUnread);
9595
}
9696

97-
// TODO: V9: Check if we really want this.
98-
// if (!isMyMessage && ownCapabilities.banChannelMembers) {
99-
// actions.push(banUser);
100-
// }
97+
if (!isMyMessage && ownCapabilities.banChannelMembers) {
98+
actions.push(banUser);
99+
}
101100

102101
if (
103102
(isMyMessage && ownCapabilities.deleteOwnMessage) ||

0 commit comments

Comments
 (0)