Compare commits

...

2 Commits

Author SHA1 Message Date
grendervill 7512857363 test(e2e): матрица 11.6 для иконки беседы и личных шаблонов (Фаза 7)
Две проверки на стенде двумя браузерами, без F5 (trackLoads = 0):

* группа из трёх пробных аккаунтов: владелец ставит иконку и переименовывает
  беседу через меню в шапке, наблюдатель-участник видит иконку в шапке и в
  списке бесед, новое имя — там же, событием DM_CHANNEL_UPDATE; участнику
  меню владельца недоступно;
* личные шаблоны: первое устройство сохраняет структуру сервера шаблоном,
  второе (свой вход формой) открывает окно создания сервера, видит шаблон в
  «Моих шаблонах» и создаёт по нему сервер — комната и роль из шаблона на
  месте. У личных шаблонов нет события шлюза: список читается при открытии
  окна, перезагрузка страницы не нужна.

Имя беседы сверяется ещё и по REST, тестовые данные убираются (группа
распускается вместе с иконкой, серверы и шаблон удаляются, данные прошлых
прогонов подчищаются в beforeAll).
2026-09-26 17:38:28 +03:00
grendervill c0137007ce feat(web): переименование групповой беседы кнопкой (Фаза 7)
Пункт «Переименовать» в меню владельца в шапке групповой беседы и диалог с
текущим именем: `PATCH /channels/{id}` (D-075) уже был, интерфейса не было.
Ответ сервера кладём в стор бесед — имя меняется и в шапке, и в списке; при
отказе окно остаётся открытым с причиной, пустое имя отправить нельзя.
Меню владельца было меню иконки: теперь это общее меню беседы (⋯) с
переименованием, загрузкой и снятием иконки — прежние testid иконки смотрят
на те же пункты, поэтому обновлены только ссылки на само меню.

i18n ru/en, Vitest: `web/tests/dmRename.test.tsx` (переименование, пустое имя,
отмена и черновик, отказ сервера, права участника, DM_CHANNEL_UPDATE без
перезагрузки) и обновлённые ссылки в `web/tests/dmIcon.test.tsx`.
2026-09-26 17:33:06 +03:00
7 changed files with 798 additions and 29 deletions
+379
View File
@@ -0,0 +1,379 @@
import { expect, test, type APIRequestContext, type Page } from '@playwright/test';
import {
apiAs,
createChannel,
createGuild,
createRole,
displayName,
dropStaleGuilds,
ensureProbe,
listChannels,
listRoles,
openApp,
openGuild,
pngBytes,
postWithRetry,
repeat429,
signIn,
stepUp,
trackLoads,
type Account,
type ProbeLogin,
type Session,
} from './support';
/**
* Realtime-матрица AGENT.md 11.6 для остатка Фазы 7: иконка и имя групповой
* беседы (D-070, D-075) и личные шаблоны серверов (7.3). Действие A видно
* второму браузеру без F5 и повторного входа — за прогон ни одной полной
* загрузки документа (`trackLoads` = 0). Запуск только по флагу, тест требует
* стенда:
*
* GLCHAT_E2E_REALTIME=1 GLCHAT_URL=https://gl.mhspx.su \
* PLAYWRIGHT_BROWSERS_PATH=../.cache/ms-playwright \
* npx playwright test e2e/realtime-phase7.spec.ts
*/
const enabled = process.env.GLCHAT_E2E_REALTIME === '1';
const API = '/api/v1';
const BASE_URL = process.env.GLCHAT_URL ?? 'https://gl.mhspx.su';
/** Префикс тестовых данных: по нему убираются серверы, шаблоны и группы прошлых прогонов. */
const PREFIX = 'Realtime E2E ';
/** Постоянные пробные аккаунты: A — владелец, B — второй браузер, C — третий участник. */
const OWNER_LOGIN: ProbeLogin = { username: 'rtprobe', password: 'Rt-Probe-Owner-9x' };
const MEMBER_LOGIN: ProbeLogin = { username: 'rtprobe2', password: 'Rt-Probe-Member-9x' };
const THIRD_LOGIN: ProbeLogin = { username: 'rtprobe3', password: 'Rt-Probe-Third-9x' };
let owner: Account;
let member: Account;
let third: Account;
test.describe('realtime: иконка и имя групповой беседы, личные шаблоны (Фаза 7)', () => {
test.skip(!enabled, 'нужен развёрнутый инстанс: GLCHAT_E2E_REALTIME=1');
// Лимит API — 120 запросов в минуту на сессию (AGENT.md 8.6): пауза между
// тестами даёт ведру лимита наполниться, иначе интерфейс получает 429.
test.afterEach(async () => {
await new Promise((resolve) => setTimeout(resolve, 4_000));
});
test.beforeAll(async ({ playwright }) => {
const ownerApi = await playwright.request.newContext({ baseURL: BASE_URL });
const memberApi = await playwright.request.newContext({ baseURL: BASE_URL });
const thirdApi = await playwright.request.newContext({ baseURL: BASE_URL });
try {
// Лимит входов — 5 в минуту на IP (AGENT.md 8.6): входы разносим по времени.
const ownerProbe = await ensureProbe(ownerApi, OWNER_LOGIN.username, OWNER_LOGIN.password);
await new Promise((resolve) => setTimeout(resolve, 2_000));
const memberProbe = await ensureProbe(
memberApi,
MEMBER_LOGIN.username,
MEMBER_LOGIN.password,
);
await new Promise((resolve) => setTimeout(resolve, 2_000));
const thirdProbe = await ensureProbe(thirdApi, THIRD_LOGIN.username, THIRD_LOGIN.password);
const ownerName = await displayName(ownerApi);
const memberName = await displayName(memberApi);
const thirdName = await displayName(thirdApi);
// Уборка данных прошлых прогонов: удаление серверов требует свежий step-up.
await stepUp(ownerApi, OWNER_LOGIN.password);
await dropStaleGuilds(ownerApi, PREFIX);
await dropStaleTemplates(ownerApi, PREFIX);
await dropStaleGroups(ownerApi, PREFIX, ownerProbe.id);
await stepUp(memberApi, MEMBER_LOGIN.password);
await dropStaleGuilds(memberApi, PREFIX);
// Браузерные сессии берём из тех же api-контекстов: лишние входы не
// тратятся, а наблюдатель входит формой уже внутри теста.
owner = {
...ownerProbe,
displayName: ownerName,
state: await ownerApi.storageState(),
browserState: await ownerApi.storageState(),
};
member = {
...memberProbe,
displayName: memberName,
state: await memberApi.storageState(),
browserState: await memberApi.storageState(),
};
third = {
...thirdProbe,
displayName: thirdName,
state: await thirdApi.storageState(),
browserState: await thirdApi.storageState(),
};
} finally {
await ownerApi.dispose();
await memberApi.dispose();
await thirdApi.dispose();
}
});
test('26. Иконка и имя групповой беседы: у наблюдателя без перезагрузки', async ({
browser,
playwright,
}) => {
const ownerApi = await apiAs(playwright, owner.state);
const sessions: Session[] = [];
let groupId = '';
try {
// Группа — это минимум три человека (AGENT.md 7.8): владелец и два гостя.
const created = await postWithRetry(ownerApi, `${API}/users/@me/channels/group`, {
name: `${PREFIX}группа`,
recipient_ids: [member.id, third.id],
});
expect(created.ok(), `групповая беседа: ${await created.text()}`).toBeTruthy();
groupId = ((await created.json()) as { channel: { id: string } }).channel.id;
// Владелец — на подготовленной сессии, наблюдатель входит формой: пути разные.
sessions.push(await openApp(browser, owner.browserState));
sessions.push(await signIn(browser, { email: member.email, password: member.password }));
const [actor, observer] = sessions as [Session, Session];
await openGroupConversation(actor.page, groupId);
await openGroupConversation(observer.page, groupId);
// Считаем полные загрузки документа: F5 в сценарии быть не должно.
const loads = trackLoads(observer.page);
loads.reset();
// Владелец ставит иконку беседы через меню в шапке.
await actor.page.getByTestId('dm-owner-menu').click();
await actor.page.getByTestId('dm-icon-upload').click();
await actor.page.getByTestId('dm-icon-input').setInputFiles({
name: 'group-icon.png',
mimeType: 'image/png',
buffer: pngBytes(64, 64, [30, 110, 210]),
});
const actorHeader = actor.page.getByTestId('dm-group-header');
await expect(actorHeader.locator('img')).toHaveAttribute('src', /\/files\/\d+/u, {
timeout: 30_000,
});
// Наблюдатель: иконка приходит событием DM_CHANNEL_UPDATE в шапку и список.
const observerHeader = observer.page.getByTestId('dm-group-header');
await expect(observerHeader.locator('img')).toHaveAttribute('src', /\/files\/\d+/u, {
timeout: 20_000,
});
await expect(observer.page.getByTestId(`dm-link-${groupId}`).locator('img')).toHaveAttribute(
'src',
/\/files\/\d+/u,
{ timeout: 20_000 },
);
// Участник-не-владелец меню владельца не получает: права проверяет сервер.
await expect(observer.page.getByTestId('dm-owner-menu')).toHaveCount(0);
// Переименование: владелец через диалог, наблюдатель видит новое имя.
const renamed = `${PREFIX}кино ${Date.now()}`;
await actor.page.getByTestId('dm-owner-menu').click();
await actor.page.getByTestId('dm-group-rename').click();
const dialog = actor.page.getByRole('dialog', { name: 'Переименовать беседу' });
await expect(dialog).toBeVisible({ timeout: 20_000 });
// Поле открывается с текущим именем — его и правит владелец.
await expect(dialog.getByLabel('Название группы')).toHaveValue(`${PREFIX}группа`);
await dialog.getByLabel('Название группы').fill(renamed);
await dialog.getByRole('button', { name: 'Сохранить' }).click();
await expect(dialog).toBeHidden({ timeout: 20_000 });
await expect(actorHeader).toContainText(renamed, { timeout: 20_000 });
await expect(observerHeader).toContainText(renamed, { timeout: 20_000 });
await expect(observer.page.getByTestId(`dm-link-${groupId}`)).toContainText(renamed, {
timeout: 20_000,
});
// Имя сохранено на сервере, а не только нарисовано в браузере владельца.
const list = await repeat429(() => ownerApi.get(`${API}/users/@me/channels`));
expect(list.ok(), `список бесед: ${await list.text()}`).toBeTruthy();
const channels = ((await list.json()) as { channels: { id: string; name: string }[] })
.channels;
expect(channels.find((channel) => channel.id === groupId)?.name).toBe(renamed);
// Ни один шаг матрицы не потребовал перезагрузки страницы.
expect(loads.count(), 'наблюдатель обновлялся без полной загрузки документа').toBe(0);
} finally {
for (const session of sessions) {
await session.context.close();
}
if (groupId !== '') {
await dropGroup(ownerApi, { id: groupId, member_ids: [member.id, third.id] }, owner.id);
}
await ownerApi.dispose();
}
});
test('27. Личные шаблоны: сервер по шаблону на другом устройстве без перезагрузки', async ({
browser,
playwright,
}) => {
const ownerApi = await apiAs(playwright, owner.state);
const sessions: Session[] = [];
const stamp = Date.now();
const sourceName = `${PREFIX}источник ${stamp}`;
const templateName = `${PREFIX}шаблон ${stamp}`;
const roomName = `tpl-room-${stamp}`;
const roleName = `tpl-role-${stamp}`;
const createdName = `${PREFIX}по шаблону ${stamp}`;
let sourceGuildId = '';
let createdGuildId = '';
let templateId = '';
try {
// Исходный сервер с отличимой структурой: по ней видно, что новый сервер
// собран именно из личного шаблона.
const source = await createGuild(ownerApi, sourceName);
sourceGuildId = source.id;
await createChannel(ownerApi, sourceGuildId, { name: roomName, type: 'text' });
await createRole(ownerApi, sourceGuildId, {
name: roleName,
permissions: 'VIEW_CHANNEL|SEND_MESSAGES',
});
// Первое устройство: сохраняем структуру личным шаблоном в настройках.
const device1 = await openApp(browser, owner.browserState);
sessions.push(device1);
await openGuild(device1.page, sourceName);
await device1.page.getByLabel('Настройки сервера').click();
const form = device1.page.getByTestId('guild-templates-form');
await expect(form).toBeVisible({ timeout: 30_000 });
await form.getByLabel('Название шаблона').fill(templateName);
await form.getByRole('button', { name: 'Сохранить шаблон' }).click();
await expect(device1.page.getByText('Шаблон сохранён.')).toBeVisible({ timeout: 20_000 });
// id шаблона берём по REST: им же наблюдатель выберет его в окне создания.
const mine = await repeat429(() => ownerApi.get(`${API}/guild-templates/mine`));
expect(mine.ok(), `свои шаблоны: ${await mine.text()}`).toBeTruthy();
const templates = ((await mine.json()) as { templates: { id: string; name: string }[] })
.templates;
templateId = templates.find((template) => template.name === templateName)?.id ?? '';
expect(templateId, 'шаблон сохранён на сервере').not.toBe('');
// Второе устройство того же пользователя: свой вход формой.
const device2 = await signIn(browser, { email: owner.email, password: owner.password });
sessions.push(device2);
const loads = trackLoads(device2.page);
loads.reset();
// Свои шаблоны видны в окне создания сервера: список читается при
// открытии, страницу перезагружать не нужно (у личных шаблонов нет
// отдельного события шлюза — они приватны владельцу).
await device2.page.getByLabel('Создать сервер').click();
const createDialog = device2.page.getByRole('dialog', { name: 'Создать сервер' });
const option = createDialog.getByTestId(`guild-template-personal-${templateId}`);
await expect(option).toBeVisible({ timeout: 20_000 });
await option.getByRole('radio').click();
await createDialog.getByLabel('Название сервера').fill(createdName);
await createDialog.getByRole('button', { name: 'Создать' }).click();
await expect(device2.page.getByTestId('guild-header')).toContainText(createdName, {
timeout: 30_000,
});
createdGuildId = device2.page.url().split('/app/')[1]?.split('/')[0] ?? '';
expect(createdGuildId, 'новый сервер открылся в приложении').not.toBe('');
// Структура нового сервера совпала с шаблоном: комната и роль на месте.
const createdChannels = await listChannels(ownerApi, createdGuildId);
expect(createdChannels.map((channel) => channel.name)).toContain(roomName);
const createdRoles = await listRoles(ownerApi, createdGuildId);
expect(createdRoles.map((role) => role.name)).toContain(roleName);
expect(loads.count(), 'окно создания сервера открылось без полной загрузки документа').toBe(
0,
);
} finally {
for (const session of sessions) {
await session.context.close();
}
await stepUp(ownerApi, OWNER_LOGIN.password).catch(() => undefined);
for (const guildId of [createdGuildId, sourceGuildId]) {
if (guildId !== '') {
await repeat429(() => ownerApi.delete(`${API}/guilds/${guildId}`)).catch(() => undefined);
}
}
if (templateId !== '') {
await repeat429(() => ownerApi.delete(`${API}/guild-templates/${templateId}`)).catch(
() => undefined,
);
}
await ownerApi.dispose();
}
});
});
/** Открывает раздел «Друзья» и переходит в беседу по ссылке сайдбара. */
async function openGroupConversation(page: Page, channelId: string): Promise<void> {
await page.getByRole('link', { name: 'Друзья' }).click();
const link = page.getByTestId(`dm-link-${channelId}`);
await expect(link).toBeVisible({ timeout: 30_000 });
await link.click();
await expect(page.getByTestId('dm-group-header')).toBeVisible({ timeout: 30_000 });
}
/** Беседа в списке `/users/@me/channels`: для уборки нужны её участники. */
interface DmChannelRef {
id: string;
name?: string;
is_group?: boolean;
member_ids?: string[];
recipients?: { user_id: string; is_self?: boolean }[];
}
/**
* dropGroup убирает тестовую группу: иконку снимаем до роспуска (иначе файл
* останется сиротой), участников удаляет владелец, а беседа без участников
* удаляется на сервере целиком.
*/
async function dropGroup(
api: APIRequestContext,
channel: DmChannelRef,
ownerId: string,
): Promise<void> {
await repeat429(() => api.delete(`${API}/channels/${channel.id}/icon`)).catch(() => undefined);
const others =
channel.member_ids ??
(channel.recipients ?? [])
.map((recipient) => recipient.user_id)
.filter((userId) => userId !== ownerId);
for (const userId of others) {
if (userId === ownerId) {
continue;
}
await repeat429(() => api.delete(`${API}/channels/${channel.id}/recipients/${userId}`)).catch(
() => undefined,
);
}
await repeat429(() => api.delete(`${API}/channels/${channel.id}/recipients/${ownerId}`)).catch(
() => undefined,
);
}
/** dropStaleGroups убирает группы прошлых прогонов: их имена начинаются с префикса. */
async function dropStaleGroups(
api: APIRequestContext,
prefix: string,
ownerId: string,
): Promise<void> {
const response = await repeat429(() => api.get(`${API}/users/@me/channels`));
if (!response.ok()) {
return;
}
const { channels } = (await response.json()) as { channels: DmChannelRef[] };
for (const channel of channels) {
if (channel.is_group === true && (channel.name ?? '').startsWith(prefix)) {
await dropGroup(api, channel, ownerId);
}
}
}
/** dropStaleTemplates убирает личные шаблоны прошлых прогонов (лимит — 20). */
async function dropStaleTemplates(api: APIRequestContext, prefix: string): Promise<void> {
const response = await repeat429(() => api.get(`${API}/guild-templates/mine`));
if (!response.ok()) {
return;
}
const { templates } = (await response.json()) as { templates: { id: string; name: string }[] };
for (const template of templates) {
if (template.name.startsWith(prefix)) {
await repeat429(() => api.delete(`${API}/guild-templates/${template.id}`)).catch(
() => undefined,
);
}
}
}
@@ -12,6 +12,7 @@ import { useAuthorDirectory } from '@/components/chat/useAuthorDirectory';
import { DmCallHistoryPanel } from '@/components/friends/DmCallHistoryPanel';
import { DmCallMenu } from '@/components/friends/DmCallMenu';
import { DmCallPanel } from '@/components/friends/DmCallPanel';
import { GroupRenameDialog } from '@/components/friends/GroupRenameDialog';
import { Avatar } from '@/components/ui/Avatar';
import { ErrorNotice } from '@/components/ui/ErrorNotice';
import { Menu, MenuItem } from '@/components/ui/Menu';
@@ -40,18 +41,19 @@ export function DirectMessageView({ channelId }: { channelId: string }) {
const upsertDmChannel = useSessionStore((state) => state.upsertDmChannel);
const currentUser = useCurrentUser();
// Иконку групповой беседы меняет только владелец (D-070): у 1:1 своей
// иконки нет — там показывается аватар собеседника.
// Иконку и имя групповой беседы меняет только владелец (D-070, D-075): у 1:1
// своей иконки нет — там показывается аватар собеседника.
const isOwner = channel?.is_group === true && channel.owner_id === currentUser.data?.id;
const [iconMenuOpen, setIconMenuOpen] = useState(false);
const [ownerMenuOpen, setOwnerMenuOpen] = useState(false);
const [renameOpen, setRenameOpen] = useState(false);
const [iconError, setIconError] = useState<unknown>(null);
const iconMenuRef = useRef<HTMLButtonElement | null>(null);
const ownerMenuRef = useRef<HTMLButtonElement | null>(null);
const iconInput = useRef<HTMLInputElement | null>(null);
// Ответ сервера кладём в стор бесед: иконка появляется сразу и у владельца,
// а остальным участникам приходит DM_CHANNEL_UPDATE (AGENT.md 11.6). Список
// перечитываем ещё и запросом: снятая иконка приходит пустым полем, а
// слияние в сторе пустое поле не затирает.
// Ответ сервера кладём в стор бесед: новое имя или иконка появляются сразу и
// у владельца, а остальным участникам приходит DM_CHANNEL_UPDATE (AGENT.md
// 11.6). Список перечитываем ещё и запросом: снятая иконка приходит пустым
// полем, а слияние в сторе пустое поле не затирает.
const applyUpdated = (updated: Channel): void => {
setIconError(null);
upsertDmChannel(updated);
@@ -178,17 +180,17 @@ export function DirectMessageView({ channelId }: { channelId: string }) {
{isOwner ? (
<>
<button
ref={iconMenuRef}
ref={ownerMenuRef}
type="button"
aria-haspopup="menu"
aria-expanded={iconMenuOpen}
aria-label={t('friends.group.icon.menu')}
title={t('friends.group.icon.menu')}
data-testid="dm-icon-menu"
aria-expanded={ownerMenuOpen}
aria-label={t('friends.group.menu')}
title={t('friends.group.menu')}
data-testid="dm-owner-menu"
className="rounded-[var(--radius-sm)] px-1.5 text-base leading-none text-fg-muted transition-[color,background-color,transform] duration-[var(--duration-fast)] hover:bg-surface-3 hover:text-fg active:scale-95"
onClick={() => setIconMenuOpen((open) => !open)}
onClick={() => setOwnerMenuOpen((open) => !open)}
>
🖼
⋯
</button>
<input
ref={iconInput}
@@ -231,19 +233,29 @@ export function DirectMessageView({ channelId }: { channelId: string }) {
{isOwner ? (
<Menu
open={iconMenuOpen}
label={t('friends.group.icon.menu')}
anchorRef={iconMenuRef}
open={ownerMenuOpen}
label={t('friends.group.menu')}
anchorRef={ownerMenuRef}
placement="bottom"
align="right"
onClose={() => setIconMenuOpen(false)}
onClose={() => setOwnerMenuOpen(false)}
>
<MenuItem
icon="✏️"
data-testid="dm-group-rename"
onClick={() => {
setOwnerMenuOpen(false);
setRenameOpen(true);
}}
>
{t('friends.group.rename')}
</MenuItem>
<MenuItem
icon="🖼"
data-testid="dm-icon-upload"
disabled={uploadIcon.isPending}
onClick={() => {
setIconMenuOpen(false);
setOwnerMenuOpen(false);
iconInput.current?.click();
}}
>
@@ -256,7 +268,7 @@ export function DirectMessageView({ channelId }: { channelId: string }) {
data-testid="dm-icon-remove"
disabled={removeIcon.isPending}
onClick={() => {
setIconMenuOpen(false);
setOwnerMenuOpen(false);
removeIcon.mutate();
}}
>
@@ -266,6 +278,15 @@ export function DirectMessageView({ channelId }: { channelId: string }) {
</Menu>
) : null}
{isOwner ? (
<GroupRenameDialog
channel={channel}
open={renameOpen}
onClose={() => setRenameOpen(false)}
onRenamed={applyUpdated}
/>
) : null}
{iconError === null ? null : (
<div className="px-4 pt-2">
<ErrorNotice error={iconError} />
@@ -0,0 +1,111 @@
import { useEffect, useRef, useState } from 'react';
import { useMutation } from '@tanstack/react-query';
import { useTranslation } from 'react-i18next';
import { renameGroupDirectChannel } from '@/api/friends';
import type { Channel } from '@/api/types';
import { ErrorNotice } from '@/components/ui/ErrorNotice';
import { Field } from '@/components/ui/Field';
import { Modal } from '@/components/ui/Modal';
import { Button } from '@/components/ui/primitives';
/**
* Переименование групповой беседы (AGENT.md 7.8, Фаза 7): имя задаёт владелец
* через `PATCH /channels/{id}`, остальным участникам его приносит
* `DM_CHANNEL_UPDATE`. Сервер требует 1–64 символа, поэтому пустое имя
* отправить нельзя — кнопка сохранения заблокирована.
*/
export function GroupRenameDialog({
channel,
open,
onClose,
onRenamed,
}: {
channel: Channel;
open: boolean;
onClose: () => void;
/** Ответ сервера: беседа с новым именем — кладём в стор и список бесед. */
onRenamed: (updated: Channel) => void;
}) {
const { t } = useTranslation();
const [name, setName] = useState(channel.name);
const formRef = useRef<HTMLFormElement | null>(null);
const rename = useMutation({
mutationFn: (value: string) => renameGroupDirectChannel(channel.id, value),
onSuccess: (updated) => {
onRenamed(updated);
onClose();
},
});
const { reset } = rename;
// Каждое открытие начинается с текущего имени: подстройка состояния под
// изменившийся проп во время рендера (тот же приём, что в `Modal`), эффект
// дал бы лишний кадр со старым черновиком.
const [wasOpen, setWasOpen] = useState(open);
if (wasOpen !== open) {
setWasOpen(open);
if (open) {
setName(channel.name);
}
}
// Фокус ставим в поле: окно открывают ровно за тем, чтобы набрать имя.
useEffect(() => {
if (open) {
formRef.current?.querySelector('input')?.focus();
}
}, [open]);
const trimmed = name.trim();
/** Закрытие гасит отказ прошлой попытки: он не должен встречать при открытии. */
const close = (): void => {
reset();
onClose();
};
const submit = (): void => {
if (trimmed !== '' && !rename.isPending) {
rename.mutate(trimmed);
}
};
return (
<Modal
open={open}
title={t('friends.group.renameTitle')}
onClose={close}
footer={
<>
<Button variant="ghost" onClick={close}>
{t('common.cancel')}
</Button>
<Button onClick={submit} disabled={trimmed === '' || rename.isPending}>
{t(rename.isPending ? 'common.saving' : 'common.save')}
</Button>
</>
}
>
<form
ref={formRef}
data-testid="dm-rename-form"
onSubmit={(event) => {
event.preventDefault();
submit();
}}
>
<Field
label={t('friends.group.name')}
name="group_name"
value={name}
maxLength={64}
placeholder={t('friends.group.namePlaceholder')}
onChange={(event) => setName(event.target.value)}
/>
{rename.isError ? <ErrorNotice className="mt-3" error={rename.error} /> : null}
</form>
</Modal>
);
}
+3 -1
View File
@@ -445,8 +445,10 @@
"create": "Create",
"members": "Members: {{count}}",
"open": "Open the group “{{name}}”, members: {{count}}",
"menu": "Conversation menu",
"rename": "Rename",
"renameTitle": "Rename conversation",
"icon": {
"menu": "Conversation icon",
"upload": "Upload icon",
"remove": "Remove icon"
}
+3 -1
View File
@@ -445,8 +445,10 @@
"create": "Создать",
"members": "Участников: {{count}}",
"open": "Открыть группу «{{name}}», участников: {{count}}",
"menu": "Меню беседы",
"rename": "Переименовать",
"renameTitle": "Переименовать беседу",
"icon": {
"menu": "Иконка беседы",
"upload": "Загрузить иконку",
"remove": "Удалить иконку"
}
+6 -6
View File
@@ -154,25 +154,25 @@ describe('иконка групповой беседы', () => {
);
});
it('меню иконки есть у владельца и отсутствует у участника', async () => {
it('меню владельца с иконкой есть у владельца и отсутствует у участника', async () => {
const channels = [groupChannel({ iconFileId: '777' })];
installFetch(dmRoutes(() => channels));
await openGroup();
const menu = await screen.findByTestId('dm-icon-menu');
const menu = await screen.findByTestId('dm-owner-menu');
await userEvent.click(menu);
expect(await screen.findByTestId('dm-icon-upload')).toBeVisible();
// Иконка уже есть: пункт удаления тоже доступен владельцу.
expect(screen.getByTestId('dm-icon-remove')).toBeVisible();
});
it('участник-не-владелец не видит меню иконки, но видит саму иконку', async () => {
it('участник-не-владелец не видит меню владельца, но видит саму иконку', async () => {
// Владелец беседы — Боб, текущий пользователь (Alice) только участник.
const channels = [groupChannel({ ownerId: 'user-2', iconFileId: '777' })];
installFetch(dmRoutes(() => channels));
await openGroup();
expect(screen.queryByTestId('dm-icon-menu')).toBeNull();
expect(screen.queryByTestId('dm-owner-menu')).toBeNull();
expect(screen.getByTestId('dm-group-header').querySelector('img')).toHaveAttribute(
'src',
'/files/777',
@@ -203,7 +203,7 @@ describe('иконка групповой беседы', () => {
const link = await screen.findByTestId('dm-link-dm-group-1');
expect(link.querySelector('img')).toBeNull();
await userEvent.click(await screen.findByTestId('dm-icon-menu'));
await userEvent.click(await screen.findByTestId('dm-owner-menu'));
await userEvent.click(screen.getByTestId('dm-icon-upload'));
// Диалог выбора файла в jsdom не открывается: файл кладём в поле напрямую.
await userEvent.upload(screen.getByTestId('dm-icon-input'), imageFile());
@@ -255,7 +255,7 @@ describe('иконка групповой беседы', () => {
expect(link.querySelector('img')).toHaveAttribute('src', '/files/777');
});
await userEvent.click(await screen.findByTestId('dm-icon-menu'));
await userEvent.click(await screen.findByTestId('dm-owner-menu'));
await userEvent.click(screen.getByTestId('dm-icon-remove'));
await waitFor(() => {
+254
View File
@@ -0,0 +1,254 @@
import { describe, expect, it } from 'vitest';
import { screen, waitFor, within } from '@testing-library/react';
import userEvent from '@testing-library/user-event';
import type { Relationships } from '@/api/friends';
import {
apiError,
bodyOf,
findRequest,
installFetch,
installGatewaySocket,
json,
makeReadySnapshot,
makeUser,
recordedRequests,
renderApp,
type FetchRoute,
} from './helpers';
/**
* Переименование групповой беседы (AGENT.md 7.8, D-075): пункт «Переименовать»
* в меню владельца, диалог с текущим именем и `PATCH /channels/{id}`. У
* остальных участников имя обновляется событием `DM_CHANNEL_UPDATE` — без
* перезагрузки и повторного входа (AGENT.md 11.6).
*/
const user = makeUser();
const instance = {
instance: {
name: 'glchat-test',
registration_enabled: true,
allow_guild_creation: true,
max_guilds_per_user: 5,
main_guild_id: 'g-main',
},
};
const friends: Relationships = { friends: [], incoming: [], outgoing: [], blocked: [] };
/** Групповая беседа в REST-форме: владелец настраивается. */
function groupChannel(options: { ownerId?: string; name?: string } = {}) {
return {
id: 'dm-group-1',
type: 'dm',
is_group: true,
name: options.name ?? 'Игровая',
member_count: 3,
owner_id: options.ownerId ?? user.id,
can_send: true,
can_view: true,
recipients: [
{
user_id: user.id,
username: 'alice',
display_name: 'Alice',
status: 'online',
visible_status: 'online',
is_owner: options.ownerId === undefined,
is_self: true,
},
{
user_id: 'user-2',
username: 'bob',
display_name: 'Боб',
status: 'online',
visible_status: 'online',
is_owner: false,
is_self: false,
},
{
user_id: 'user-3',
username: 'carol',
display_name: 'Кэрол',
status: 'offline',
visible_status: 'offline',
is_owner: false,
is_self: false,
},
],
};
}
/**
* Маршруты приложения: список бесед отдаётся из изменяемого состояния — так
* тест видит, что сервер уже принял новое имя.
*/
function dmRoutes(current: () => unknown[], extra: FetchRoute[] = []): FetchRoute[] {
return [
...extra,
{ match: '/api/v1/users/@me/relationships', response: () => json(friends) },
{ match: '/api/v1/users/@me/guilds', response: () => json({ guilds: [] }) },
{ match: '/api/v1/channels/dm-group-1/messages', response: () => json({ messages: [] }) },
{ match: '/api/v1/channels/dm-group-1/pins', response: () => json({ messages: [] }) },
{
match: '/api/v1/channels/dm-group-1/ack',
method: 'POST',
response: () => json({ ok: true }),
},
{ match: '/api/v1/users/@me/channels', response: () => json({ channels: current() }) },
{ match: '/api/v1/users/@me', response: () => json({ user }) },
{ match: '/api/v1/instance', response: () => json(instance) },
];
}
/** Открывает беседу и ждёт её отрисовки в сайдбаре. */
async function openGroup() {
renderApp('/app/friends/dm-group-1');
const gateway = installGatewaySocket(makeReadySnapshot());
await gateway.greet();
await screen.findByTestId('dm-group-header');
return gateway;
}
/** Открывает диалог переименования из меню владельца. */
async function openRenameDialog() {
await userEvent.click(await screen.findByTestId('dm-owner-menu'));
await userEvent.click(await screen.findByTestId('dm-group-rename'));
return screen.findByRole('dialog', { name: 'Переименовать беседу' });
}
describe('переименование групповой беседы', () => {
it('владелец переименовывает беседу: имя меняется в шапке и в списке', async () => {
let channels = [groupChannel()];
const fetchMock = installFetch(
dmRoutes(
() => channels,
[
{
match: '/api/v1/channels/dm-group-1',
method: 'PATCH',
response: () => {
const updated = groupChannel({ name: 'Кино по пятницам' });
channels = [updated];
return json({ channel: updated });
},
},
],
),
);
await openGroup();
const dialog = await openRenameDialog();
// Поле уже заполнено текущим именем: переименование начинается с правки.
const field = within(dialog).getByLabelText('Название группы');
expect(field).toHaveValue('Игровая');
await userEvent.clear(field);
await userEvent.type(field, 'Кино по пятницам');
await userEvent.click(within(dialog).getByRole('button', { name: 'Сохранить' }));
await waitFor(() => {
expect(
bodyOf(findRequest(fetchMock, { url: '/channels/dm-group-1', method: 'PATCH' })),
).toEqual({ name: 'Кино по пятницам' });
});
// Диалог закрылся, а новое имя видно и в шапке, и в списке бесед.
await waitFor(() => {
expect(screen.queryByRole('dialog', { name: 'Переименовать беседу' })).toBeNull();
});
expect(screen.getByTestId('dm-group-header')).toHaveTextContent('Кино по пятницам');
expect(await screen.findByTestId('dm-link-dm-group-1')).toHaveTextContent('Кино по пятницам');
});
it('пустое имя отправить нельзя: кнопка сохранения заблокирована', async () => {
const fetchMock = installFetch(dmRoutes(() => [groupChannel()]));
await openGroup();
const dialog = await openRenameDialog();
await userEvent.clear(within(dialog).getByLabelText('Название группы'));
expect(within(dialog).getByRole('button', { name: 'Сохранить' })).toBeDisabled();
// Пробелы именем не считаются: сервер обрезает их и отвечает отказом.
await userEvent.type(within(dialog).getByLabelText('Название группы'), ' ');
expect(within(dialog).getByRole('button', { name: 'Сохранить' })).toBeDisabled();
expect(
findRequest(fetchMock, { url: '/channels/dm-group-1', method: 'PATCH' }),
).toBeUndefined();
});
it('отмена закрывает диалог без запроса и не запоминает черновик', async () => {
const fetchMock = installFetch(dmRoutes(() => [groupChannel()]));
await openGroup();
const dialog = await openRenameDialog();
await userEvent.clear(within(dialog).getByLabelText('Название группы'));
await userEvent.type(within(dialog).getByLabelText('Название группы'), 'Черновик');
await userEvent.click(within(dialog).getByRole('button', { name: 'Отмена' }));
await waitFor(() => {
expect(screen.queryByRole('dialog', { name: 'Переименовать беседу' })).toBeNull();
});
expect(
findRequest(fetchMock, { url: '/channels/dm-group-1', method: 'PATCH' }),
).toBeUndefined();
expect(screen.getByTestId('dm-group-header')).toHaveTextContent('Игровая');
// Второе открытие снова показывает текущее имя сервера, а не черновик.
const reopened = await openRenameDialog();
expect(within(reopened).getByLabelText('Название группы')).toHaveValue('Игровая');
});
it('отказ сервера показывается в диалоге, имя остаётся прежним', async () => {
installFetch(
dmRoutes(
() => [groupChannel()],
[
{
match: '/api/v1/channels/dm-group-1',
method: 'PATCH',
response: () =>
apiError('validation.failed', 422, 'group name must be 1-64 characters'),
},
],
),
);
await openGroup();
const dialog = await openRenameDialog();
const field = within(dialog).getByLabelText('Название группы');
await userEvent.clear(field);
await userEvent.type(field, 'Слишком длинное имя');
await userEvent.click(within(dialog).getByRole('button', { name: 'Сохранить' }));
// Окно не закрывается: пользователь видит причину и может исправить имя.
expect(await within(dialog).findByRole('alert')).toBeVisible();
expect(screen.getByTestId('dm-group-header')).toHaveTextContent('Игровая');
});
it('участнику-не-владельцу пункт переименования недоступен', async () => {
installFetch(dmRoutes(() => [groupChannel({ ownerId: 'user-2' })]));
await openGroup();
expect(screen.queryByTestId('dm-group-rename')).toBeNull();
expect(screen.queryByTestId('dm-owner-menu')).toBeNull();
});
it('событие DM_CHANNEL_UPDATE доносит новое имя без перезагрузки', async () => {
let channels = [groupChannel()];
const fetchMock = installFetch(dmRoutes(() => channels));
const gateway = await openGroup();
// Беседу переименовал владелец (возможно, с другого устройства): сервер
// сообщает об изменении события, а список перечитывается по REST.
channels = [groupChannel({ name: 'Новое имя' })];
gateway.dispatch('DM_CHANNEL_UPDATE', { channel_id: 'dm-group-1' });
await waitFor(() => {
expect(screen.getByTestId('dm-group-header')).toHaveTextContent('Новое имя');
});
expect(await screen.findByTestId('dm-link-dm-group-1')).toHaveTextContent('Новое имя');
// Имя пришло событием: своих PATCH-запросов тест не делал.
expect(recordedRequests(fetchMock).filter((request) => request.method === 'PATCH')).toEqual([]);
});
});