@micha.bigler/ui-core-micha 3.14.3 → 4.0.0

package/tests/messagingComposerChunk5.test.jsx

@@ -1,113 +0,0 @@
-// @vitest-environment jsdom
-import React from 'react';
-import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
-import { cleanup, fireEvent, render, screen, waitFor } from '@testing-library/react';
-
-vi.mock('react-i18next', () => ({ useTranslation: () => ({ t: (key, options) => options?.name ? `${key}:${options.name}` : key }) }));
-vi.mock('../src/notifications/realtime', () => ({ useRealtime: () => ({ subscribe: () => () => {}, onReconnect: () => () => {} }) }));
-
-import { Composer } from '../src/messaging/Composer';
-import { MessagingProvider } from '../src/messaging/MessagingProvider';
-
-function makeApi() {
-  return {
-    listConversations: vi.fn().mockResolvedValue({ results: [] }), listMessages: vi.fn().mockResolvedValue({ results: [] }), getUnreadCount: vi.fn().mockResolvedValue({ unread_count: 0, by_conversation: {} }),
-    createMessage: vi.fn().mockResolvedValue({ id: 1, conversation_id: 7 }), uploadAttachments: vi.fn().mockResolvedValue({ id: 2, conversation_id: 7 }), getAttachment: vi.fn(), getAttachmentThumbnail: vi.fn(),
-  };
-}
-function renderComposer(api = makeApi(), conversationId = 7) {
-  return render(<MessagingProvider api={api} active={false}><Composer conversationId={conversationId} /></MessagingProvider>);
-}
-function attachmentInput() { return document.querySelector('input[type=file]'); }
-function messageInput() { return screen.getByLabelText('MessagingComposer.MESSAGE'); }
-function imageFile(name = 'photo.png') { return new File(['image'], name, { type: 'image/png' }); }
-
-beforeEach(() => {
-  URL.createObjectURL = vi.fn((file) => `blob:${file.name}`);
-  URL.revokeObjectURL = vi.fn();
-});
-afterEach(() => { cleanup(); vi.restoreAllMocks(); });
-
-describe('Composer chunk 5 behavior', () => {
-  it('sends on Enter and leaves Shift+Enter available for a newline', async () => {
-    const api = makeApi(); renderComposer(api);
-    fireEvent.change(messageInput(), { target: { value: 'first' } });
-    fireEvent.keyDown(messageInput(), { key: 'Enter', code: 'Enter' });
-    await waitFor(() => expect(api.createMessage).toHaveBeenCalledTimes(1));
-
-    fireEvent.change(messageInput(), { target: { value: 'first' } });
-    fireEvent.keyDown(messageInput(), { key: 'Enter', code: 'Enter', shiftKey: true });
-    fireEvent.change(messageInput(), { target: { value: 'first\nsecond' } });
-    expect(api.createMessage).toHaveBeenCalledTimes(1);
-    expect(messageInput().value).toBe('first\nsecond');
-  });
-
-  it('shows staged image previews and removes only the chosen file before upload', async () => {
-    const api = makeApi(); renderComposer(api);
-    const first = imageFile('first.png'); const second = imageFile('second.png');
-    fireEvent.change(attachmentInput(), { target: { files: [first, second] } });
-    expect(screen.getByRole('img', { name: 'first.png' })).toBeTruthy();
-    expect(screen.getByRole('img', { name: 'second.png' })).toBeTruthy();
-    fireEvent.click(screen.getByRole('button', { name: 'MessagingComposer.REMOVE_ATTACHMENT:first.png' }));
-    expect(screen.queryByRole('img', { name: 'first.png' })).toBeNull();
-    fireEvent.click(screen.getByRole('button', { name: 'MessagingComposer.SEND' }));
-    await waitFor(() => expect(api.uploadAttachments).toHaveBeenCalledTimes(1));
-    const sentFiles = Array.from(api.uploadAttachments.mock.calls[0][1].getAll('files[]'));
-    expect(sentFiles).toHaveLength(1);
-    expect(sentFiles[0].name).toBe('second.png');
-  });
-
-  it('shows real byte-percentage upload progress while an attachment upload is in flight', async () => {
-    let resolveUpload; const api = makeApi(); api.uploadAttachments.mockReturnValue(new Promise((resolve) => { resolveUpload = resolve; }));
-    renderComposer(api); fireEvent.change(attachmentInput(), { target: { files: [imageFile()] } });
-    fireEvent.click(screen.getByRole('button', { name: 'MessagingComposer.SEND' }));
-    expect(await screen.findByRole('status')).toBeTruthy();
-    const { onUploadProgress } = api.uploadAttachments.mock.calls[0][2];
-    onUploadProgress({ loaded: 25, total: 100 });
-    await waitFor(() => expect(screen.getAllByRole('progressbar')[0].getAttribute('aria-valuenow')).toBe('25'));
-    resolveUpload({ id: 2, conversation_id: 7 });
-    await waitFor(() => expect(screen.queryByRole('status')).toBeNull());
-  });
-
-  it('compresses oversized images before adding them to FormData and falls back silently if compression fails', async () => {
-    const largeImage = new File([new Uint8Array(2 * 1024 * 1024 + 1)], 'large.png', { type: 'image/png' });
-    const api = makeApi();
-    class SuccessfulImage { naturalWidth = 5120; naturalHeight = 2560; set src(_) { this.onload(); } }
-    vi.stubGlobal('Image', SuccessfulImage);
-    vi.spyOn(HTMLCanvasElement.prototype, 'getContext').mockReturnValue({ drawImage: vi.fn() });
-    vi.spyOn(HTMLCanvasElement.prototype, 'toBlob').mockImplementation((callback) => callback(new Blob(['compressed'], { type: 'image/jpeg' })));
-    renderComposer(api); fireEvent.change(attachmentInput(), { target: { files: [largeImage] } }); fireEvent.click(screen.getByRole('button', { name: 'MessagingComposer.SEND' }));
-    await waitFor(() => expect(api.uploadAttachments).toHaveBeenCalledTimes(1));
-    const compressed = api.uploadAttachments.mock.calls[0][1].get('files[]');
-    expect(compressed.name).toBe('large.jpg'); expect(compressed.type).toBe('image/jpeg');
-
-    cleanup(); const fallbackApi = makeApi(); class FailingImage { set src(_) { this.onerror(); } }
-    vi.stubGlobal('Image', FailingImage);
-    renderComposer(fallbackApi); fireEvent.change(attachmentInput(), { target: { files: [largeImage] } }); fireEvent.click(screen.getByRole('button', { name: 'MessagingComposer.SEND' }));
-    await waitFor(() => expect(fallbackApi.uploadAttachments).toHaveBeenCalledTimes(1));
-    const fallback = fallbackApi.uploadAttachments.mock.calls[0][1].get('files[]');
-    expect(fallback.name).toBe('large.png'); expect(fallback.type).toBe('image/png');
-  });
-
-  it('inserts the selected emoji at the current cursor position', async () => {
-    // MSG-6i: the Composer's own picker is now a wider curated grid
-    // (IconButton per emoji), decoupled from ReactionBar's 5-entry
-    // QUICK_EMOJIS -- role is "button", not "menuitem".
-    renderComposer(); fireEvent.change(messageInput(), { target: { value: 'abCD' } });
-    messageInput().setSelectionRange(2, 2);
-    fireEvent.click(screen.getByRole('button', { name: 'MessagingComposer.ADD_EMOJI' }));
-    fireEvent.click(await screen.findByRole('button', { name: '👍' }));
-    await waitFor(() => expect(messageInput().value).toBe('ab👍CD'));
-  });
-
-  it('disables poll composing while an image is staged and resets drafts for a new conversation', async () => {
-    const api = makeApi(); const view = renderComposer(api, 7);
-    fireEvent.change(attachmentInput(), { target: { files: [imageFile()] } });
-    expect(screen.getByRole('button', { name: 'MessagingPoll.CREATE' }).disabled).toBe(true);
-    fireEvent.click(screen.getByRole('button', { name: 'MessagingComposer.REMOVE_ATTACHMENT:photo.png' }));
-    expect(screen.getByRole('button', { name: 'MessagingPoll.CREATE' }).disabled).toBe(false);
-    fireEvent.change(messageInput(), { target: { value: 'unsent draft' } });
-    view.rerender(<MessagingProvider api={api} active={false}><Composer conversationId={8} /></MessagingProvider>);
-    await waitFor(() => expect(messageInput().value).toBe(''));
-  });
-});

package/tests/messagingContractConformance.test.js

@@ -1,69 +0,0 @@
-import fs from 'node:fs';
-import path from 'node:path';
-import { describe, expect, it } from 'vitest';
-
-const root = path.resolve(import.meta.dirname, '..');
-const messagingRoot = path.join(root, 'src', 'messaging');
-
-// These are deliberately named and time-bounded. Chunks 3 and 4 must remove
-// their entries when they wire their existing adapters; the final three are
-// retained only if their stated product rationale still holds at WO close.
-const API_EXEMPTIONS = {
-  createManagedConversation: 'No managed-conversation launcher is in the current public surface.',
-  createObjectThreadConversation: 'Object-thread creation is host-owned routing/scope work.',
-  getMessage: 'Timeline data is loaded through listMessages and listThread.',
-};
-const FRAME_TYPES = ['conversation_upsert', 'conversation_archived', 'message', 'message_edited', 'message_deleted', 'attachment_ready', 'reaction', 'poll_updated', 'delivered', 'read_state', 'thread_read_state', 'participant_changed'];
-const FRAME_EXEMPTIONS = {
-  attachment_ready: 'Reserved by the design and deliberately unemitted.',
-  delivered: 'Receipt synchronisation is outside MSG-3b.',
-  read_state: 'Receipt synchronisation is outside MSG-3b.',
-  thread_read_state: 'Receipt synchronisation is outside MSG-3b.',
-};
-
-const BLOCKED_LINE = /\*\*BLOCKED\*\*/;
-const VERSION_PIN = /dcm \d+\.\d+\.\d+/;
-
-// A BLOCKED row without a version pin is exactly how MSG-3b's seven blocked
-// rows went stale unnoticed: dcm shipped what they needed mid-flight and the
-// WO-end review had no cheap way to tell. This check can't know whether dcm
-// has since shipped — it only guarantees the pin exists, which is what turns
-// the re-check into a two-minute job instead of an archaeology exercise.
-function unpinnedBlockedLines(markdown) {
-  return markdown.split('\n').filter((line) => BLOCKED_LINE.test(line) && !VERSION_PIN.test(line));
-}
-
-describe('messaging deviation-doc BLOCKED entries carry a dependency version pin', () => {
-  it('every BLOCKED line in docs/messaging-deviations.md names the dcm version it was blocked at', () => {
-    const markdown = fs.readFileSync(path.join(root, 'docs', 'messaging-deviations.md'), 'utf8');
-    expect(unpinnedBlockedLines(markdown)).toEqual([]);
-  });
-
-  // Required negative case (the WO is explicit: prove the check can fail,
-  // not only that the real file happens to pass).
-  it('flags a BLOCKED line that omits the version pin', () => {
-    const unpinned = '99. **BLOCKED** — some capability with no dependency version named.';
-    const pinned = '100. **BLOCKED** (blocked as of dcm 2.37.0) — some other capability.';
-    expect(unpinnedBlockedLines(`${unpinned}\n${pinned}`)).toEqual([unpinned]);
-  });
-});
-
-describe('messaging adapter and realtime contract conformance', () => {
-  it('has a caller for every API export unless it has an explicit rationale', () => {
-    const apiSource = fs.readFileSync(path.join(messagingRoot, 'api.js'), 'utf8');
-    const source = fs.readdirSync(messagingRoot).filter((file) => file.endsWith('.jsx')).map((file) => fs.readFileSync(path.join(messagingRoot, file), 'utf8')).join('\n');
-    const exports = [...apiSource.matchAll(/export function (\w+)\(/g)].map((match) => match[1]);
-    // Attachment readers are intentionally passed through the provider to the
-    // standalone AttachmentList, where they are called. All other adapters
-    // are invoked by the provider/component that owns the action.
-    const unwired = exports.filter((name) => !new RegExp(`api\\.${name}\\(|${name}: api\\.${name}`).test(source) && !API_EXEMPTIONS[name]);
-    expect(unwired).toEqual([]);
-  });
-
-  it('handles only design-listed frames and accounts for intentionally unhandled frames', () => {
-    const provider = fs.readFileSync(path.join(messagingRoot, 'MessagingProvider.jsx'), 'utf8');
-    const handled = [...provider.matchAll(/frame\.type === '([^']+)'/g)].map((match) => match[1]);
-    expect(handled.every((type) => FRAME_TYPES.includes(type))).toBe(true);
-    expect(FRAME_TYPES.filter((type) => !handled.includes(type) && !FRAME_EXEMPTIONS[type])).toEqual([]);
-  });
-});

package/tests/messagingDirectMessageLauncher.test.jsx

@@ -1,151 +0,0 @@
-// @vitest-environment jsdom
-import React from 'react';
-import { afterEach, describe, expect, it, vi } from 'vitest';
-import { cleanup, fireEvent, render, screen, waitFor } from '@testing-library/react';
-
-vi.mock('react-i18next', () => ({ useTranslation: () => ({ t: (key) => key }) }));
-vi.mock('../src/notifications/realtime', () => ({ useRealtime: () => ({ subscribe: () => () => {}, onReconnect: () => () => {} }) }));
-
-import { DirectMessageLauncher } from '../src/index';
-import { MessagingProvider } from '../src/messaging/MessagingProvider';
-
-const candidates = [{ id: 2, display_name: 'Alex' }, { id: 3, display_name: 'Sam' }];
-
-function renderLauncher({ api = { createDirectConversation: vi.fn().mockResolvedValue({ id: 41, kind: 'direct' }) }, ...props } = {}) {
-  const onOpen = props.onOpen || vi.fn();
-  render(<MessagingProvider api={api} active={false}><DirectMessageLauncher candidates={candidates} onOpen={onOpen} {...props} /></MessagingProvider>);
-  return { api, onOpen };
-}
-function openPicker() { fireEvent.click(screen.getByRole('button', { name: 'MessagingDirect.LAUNCH' })); }
-function selectCandidate(name) {
-  const input = screen.getByLabelText('MessagingDirect.CANDIDATES');
-  fireEvent.mouseDown(input);
-  fireEvent.click(screen.getByRole('option', { name }));
-}
-
-afterEach(() => {
-  cleanup();
-  window.matchMedia = undefined;
-});
-
-describe('DirectMessageLauncher', () => {
-  it('opens a picker with a searchable candidate autocomplete and renders a distinct empty state', () => {
-    renderLauncher();
-    openPicker();
-    expect(screen.getByRole('dialog')).toBeTruthy();
-    const input = screen.getByLabelText('MessagingDirect.CANDIDATES');
-    expect(input.getAttribute('placeholder')).toBe('MessagingDirect.SEARCH_PLACEHOLDER');
-    fireEvent.mouseDown(input);
-    expect(screen.getByRole('option', { name: 'Alex' })).toBeTruthy();
-    expect(screen.getByRole('option', { name: 'Sam' })).toBeTruthy();
-
-    cleanup();
-    renderLauncher({ candidates: [] });
-    openPicker();
-    expect(screen.getByText('MessagingDirect.EMPTY')).toBeTruthy();
-    expect(screen.queryByLabelText('MessagingDirect.CANDIDATES')).toBeNull();
-  });
-
-  it('filters candidates as the query is typed', () => {
-    renderLauncher();
-    openPicker();
-    const input = screen.getByLabelText('MessagingDirect.CANDIDATES');
-    fireEvent.mouseDown(input);
-    // MUI's Autocomplete only reacts to a typed value once the input is genuinely focused --
-    // mouseDown alone opens the listbox but doesn't focus the input in jsdom.
-    input.focus();
-    fireEvent.change(input, { target: { value: 'Sa' } });
-    expect(screen.getByRole('option', { name: 'Sam' })).toBeTruthy();
-    expect(screen.queryByRole('option', { name: 'Alex' })).toBeNull();
-  });
-
-  it('shows a localized message when a typed query matches nothing (ui_reviewer finding U1)', () => {
-    renderLauncher();
-    openPicker();
-    const input = screen.getByLabelText('MessagingDirect.CANDIDATES');
-    fireEvent.mouseDown(input);
-    input.focus();
-    fireEvent.change(input, { target: { value: 'zzz-no-such-person' } });
-    expect(screen.getByText('MessagingDirect.NO_MATCHES')).toBeTruthy();
-  });
-
-  it('requires a recipient, disables while starting, then opens the created conversation with scope', async () => {
-    let resolveRequest;
-    const api = { createDirectConversation: vi.fn(() => new Promise((resolve) => { resolveRequest = resolve; })) };
-    const { onOpen } = renderLauncher({ api, scope: { kind: 'event', id: 8 } });
-    openPicker();
-    const start = screen.getByRole('button', { name: 'MessagingDirect.START' });
-    expect(start.disabled).toBe(true);
-    selectCandidate('Alex');
-    expect(start.disabled).toBe(false);
-    fireEvent.click(start);
-    expect(api.createDirectConversation).toHaveBeenCalledWith({ target_user_id: 2, scope: { kind: 'event', id: 8 } });
-    expect(start.disabled).toBe(true);
-    expect(screen.getByRole('status')).toBeTruthy();
-    resolveRequest({ id: 41, kind: 'direct' });
-    await waitFor(() => expect(onOpen).toHaveBeenCalledWith({ id: 41, kind: 'direct' }));
-    await waitFor(() => expect(screen.queryByRole('dialog')).toBeNull());
-  });
-
-  it('uses the selected candidate scope over the launcher scope', () => {
-    const api = { createDirectConversation: vi.fn().mockResolvedValue({ id: 41, kind: 'direct' }) };
-    renderLauncher({
-      api,
-      candidates: [
-        { id: 2, display_name: 'Alex', scope: 'scope-a' },
-        { id: 3, display_name: 'Sam', scope: 'scope-b' },
-      ],
-      scope: 'scope-prop',
-    });
-    openPicker();
-    selectCandidate('Sam');
-    fireEvent.click(screen.getByRole('button', { name: 'MessagingDirect.START' }));
-    expect(api.createDirectConversation).toHaveBeenCalledWith({ target_user_id: 3, scope: 'scope-b' });
-  });
-
-  it('falls back to the launcher scope when the selected candidate has none', () => {
-    const api = { createDirectConversation: vi.fn().mockResolvedValue({ id: 41, kind: 'direct' }) };
-    renderLauncher({
-      api,
-      candidates: [
-        { id: 2, display_name: 'Alex' },
-        { id: 3, display_name: 'Sam', scope: 'scope-b' },
-      ],
-      scope: 'scope-prop',
-    });
-    openPicker();
-    selectCandidate('Alex');
-    fireEvent.click(screen.getByRole('button', { name: 'MessagingDirect.START' }));
-    expect(api.createDirectConversation).toHaveBeenCalledWith({ target_user_id: 2, scope: 'scope-prop' });
-  });
-
-  it('omits scope when neither the candidate nor launcher provides one', () => {
-    const api = { createDirectConversation: vi.fn().mockResolvedValue({ id: 41, kind: 'direct' }) };
-    renderLauncher({ api });
-    openPicker();
-    selectCandidate('Alex');
-    fireEvent.click(screen.getByRole('button', { name: 'MessagingDirect.START' }));
-    expect(Object.keys(api.createDirectConversation.mock.calls[0][0])).toEqual(['target_user_id']);
-  });
-
-  it('surfaces a policy rejection without closing the picker', async () => {
-    const api = { createDirectConversation: vi.fn().mockRejectedValue({ response: { data: { detail: 'Direct messages are restricted.' } } }) };
-    renderLauncher({ api });
-    openPicker();
-    selectCandidate('Alex');
-    fireEvent.click(screen.getByRole('button', { name: 'MessagingDirect.START' }));
-    expect((await screen.findByRole('alert')).textContent).toContain('Direct messages are restricted.');
-    expect(screen.getByRole('dialog')).toBeTruthy();
-  });
-
-  it('uses a fullscreen dialog at the small-viewport media query', () => {
-    window.matchMedia = vi.fn().mockImplementation(() => ({
-      matches: true,
-      addEventListener: vi.fn(),
-      removeEventListener: vi.fn(),
-    }));
-    renderLauncher();
-    openPicker();
-    expect(document.querySelector('.MuiDialog-paperFullScreen')).not.toBeNull();
-  });
-});

package/tests/messagingExports.test.js

@@ -1,21 +0,0 @@
-import { describe, expect, it } from 'vitest';
-import { AttachmentList, Composer, ConversationLaunchers, ConversationList, DirectMessageLauncher, MessageBubble, MessagingProvider, MessagingScopeConfig, messagingTranslations, PollCard, ReactionBar, ReadTicks, Thread, useMessaging } from '../src/index';
-
-describe('messaging barrel exports', () => {
-  it('exports the provider, hook, standalone list and launchers', () => {
-    expect(MessagingProvider).toBeDefined();
-    expect(useMessaging).toBeDefined();
-    expect(ConversationList).toBeDefined();
-    expect(ConversationLaunchers).toBeDefined();
-    expect(DirectMessageLauncher).toBeDefined();
-    expect(Thread).toBeDefined();
-    expect(MessageBubble).toBeDefined();
-    expect(ReadTicks).toBeDefined();
-    expect(Composer).toBeDefined();
-    expect(AttachmentList).toBeDefined();
-    expect(ReactionBar).toBeDefined();
-    expect(PollCard).toBeDefined();
-    expect(MessagingScopeConfig).toBeDefined();
-    expect(messagingTranslations).toBeDefined();
-  });
-});

package/tests/messagingInteractions.test.jsx

@@ -1,136 +0,0 @@
-// @vitest-environment jsdom
-import React from 'react';
-import { afterEach, describe, expect, it, vi } from 'vitest';
-import { cleanup, fireEvent, render, screen, waitFor } from '@testing-library/react';
-
-vi.mock('react-i18next', () => ({ useTranslation: () => ({ t: (key, options) => options?.message ? `${key}:${options.message}` : key }) }));
-vi.mock('../src/notifications/realtime', () => ({ useRealtime: () => ({ subscribe: () => () => {}, onReconnect: () => () => {} }) }));
-
-import { Composer } from '../src/messaging/Composer';
-import { MessagingProvider, useMessaging } from '../src/messaging/MessagingProvider';
-import { PollCard } from '../src/messaging/PollCard';
-import { ReactionBar } from '../src/messaging/ReactionBar';
-import { MessagingScopeConfig } from '../src/messaging/MessagingScopeConfig';
-
-function CacheState() { const { cache } = useMessaging(); return <output data-testid="cache">{JSON.stringify(cache)}</output>; }
-
-const baseApi = () => ({ listConversations: vi.fn().mockResolvedValue({ results: [] }), listMessages: vi.fn().mockResolvedValue({ results: [] }), getUnreadCount: vi.fn().mockResolvedValue({ unread_count: 0, by_conversation: {} }), addReaction: vi.fn(), removeReaction: vi.fn(), votePoll: vi.fn(), closePoll: vi.fn(), createPoll: vi.fn() });
-afterEach(cleanup);
-
-describe('messaging reactions and polls', () => {
-  it('round-trips add and remove reactions through the cache-backed API', async () => {
-    // serialize_reactions (dcm) never returns a per-viewer `reacted` flag —
-    // {emoji, count} only, in every context.
-    const api = baseApi(); api.addReaction.mockResolvedValue({ reactions: [{ emoji: '👍', count: 1 }] }); api.removeReaction.mockResolvedValue({ reactions: [] });
-    const { rerender } = render(<MessagingProvider api={api} active={false}><ReactionBar message={{ id: 7, reactions: [{ emoji: '👍', count: 1, reacted: false }] }} /></MessagingProvider>);
-    fireEvent.click(screen.getByRole('button', { name: 'MessagingReactions.TOGGLE' })); await waitFor(() => expect(api.addReaction).toHaveBeenCalledWith(7, '👍'));
-    rerender(<MessagingProvider api={api} active={false}><ReactionBar message={{ id: 7, reactions: [{ emoji: '👍', count: 1, reacted: true }] }} /></MessagingProvider>);
-    fireEvent.click(screen.getByRole('button', { name: 'MessagingReactions.TOGGLE' })); await waitFor(() => expect(api.removeReaction).toHaveBeenCalledWith(7, '👍'));
-  });
-
-  it('taps vote immediately (no submit step): single-choice uses radios and only ever sends the last-tapped option, multi-select uses checkboxes and accumulates', async () => {
-    // MSG-6g: tap-to-vote, no separate "Abstimmen" button.
-    const single = baseApi(); single.votePoll.mockResolvedValue({ poll: { id: 1, options: [] } });
-    const { unmount } = render(<MessagingProvider api={single} active={false}><PollCard message={{ id: 7, poll: { id: 1, question: 'One', options: [{ id: 1, text: 'A' }, { id: 2, text: 'B' }], allow_multiple: false }}} /></MessagingProvider>);
-    const radios = screen.getAllByRole('radio');
-    fireEvent.click(radios[0]); await waitFor(() => expect(single.votePoll).toHaveBeenCalledWith(1, [1]));
-    fireEvent.click(radios[1]); await waitFor(() => expect(single.votePoll).toHaveBeenCalledWith(1, [2]));
-    expect(single.votePoll).toHaveBeenCalledTimes(2);
-    unmount();
-    const multiple = baseApi(); multiple.votePoll.mockResolvedValue({ poll: { id: 2, options: [] } }); render(<MessagingProvider api={multiple} active={false}><PollCard message={{ id: 8, poll: { id: 2, question: 'Many', options: [{ id: 1, text: 'A' }, { id: 2, text: 'B' }], allow_multiple: true }}} /></MessagingProvider>);
-    const multiChoices = screen.getAllByRole('checkbox');
-    fireEvent.click(multiChoices[0]); await waitFor(() => expect(multiple.votePoll).toHaveBeenCalledWith(2, [1]));
-    fireEvent.click(multiChoices[1]); await waitFor(() => expect(multiple.votePoll).toHaveBeenCalledWith(2, [1, 2]));
-  });
-
-  it('only exposes close to the creator/capability and surfaces a forbidden close', async () => {
-    const api = baseApi(); api.closePoll.mockRejectedValue({ response: { data: { detail: 'Not permitted' } } });
-    const { rerender } = render(<MessagingProvider api={api} active={false}><PollCard message={{ id: 7, poll: { id: 1, question: 'Q', options: [], created_by: 2 }}} /></MessagingProvider>);
-    expect(screen.queryByRole('button', { name: 'MessagingPoll.CLOSE' })).toBeNull();
-    rerender(<MessagingProvider api={api} active={false}><PollCard canClose message={{ id: 7, poll: { id: 1, question: 'Q', options: [] }}} /></MessagingProvider>);
-    fireEvent.click(screen.getByRole('button', { name: 'MessagingPoll.CLOSE' })); await screen.findByRole('alert');
-  });
-
-  it('creates polls with an idempotency key', async () => {
-    const api = baseApi(); api.createPoll.mockResolvedValue({ id: 70, conversation_id: 4, poll: { id: 1 } });
-    render(<MessagingProvider api={api} active={false}><Composer conversationId={4} /></MessagingProvider>);
-    fireEvent.click(screen.getByRole('button', { name: 'MessagingPoll.CREATE' })); fireEvent.change(screen.getByLabelText('MessagingPoll.QUESTION'), { target: { value: 'Where?' } });
-    const options = screen.getAllByLabelText(/MessagingPoll.OPTION/); fireEvent.change(options[0], { target: { value: 'Here' } }); fireEvent.change(options[1], { target: { value: 'There' } });
-    fireEvent.click(screen.getAllByRole('button', { name: 'MessagingPoll.CREATE' })[0]); await waitFor(() => expect(api.createPoll).toHaveBeenCalled());
-    expect(api.createPoll.mock.calls[0][2].idempotencyKey).toBeTruthy();
-  });
-
-  it('mounts the scope config as an independent provider collaborator', async () => {
-    const api = baseApi(); api.getConversationConfig = vi.fn().mockResolvedValue({ dm_policy: 'all', group_chat_enabled: true, everyone_can_post: false });
-    render(<MessagingProvider api={api} active={false}><MessagingScopeConfig conversationId={4} /></MessagingProvider>);
-    await screen.findByLabelText('MessagingConfig.LABEL');
-    expect(api.getConversationConfig).toHaveBeenCalledWith(4);
-  });
-
-  it('creates an announcement with a title, body and host-supplied link target', async () => {
-    const api = baseApi(); api.createMessage = vi.fn().mockResolvedValue({ id: 90, conversation_id: 4, kind: 'announcement', title: 'Heads up' });
-    render(<MessagingProvider api={api} active={false}><Composer conversationId={4} allowAnnouncement linkTarget="/events/4/info" /></MessagingProvider>);
-    fireEvent.click(screen.getByRole('button', { name: 'MessagingAnnouncement.CREATE' }));
-    fireEvent.change(screen.getByLabelText('MessagingAnnouncement.TITLE'), { target: { value: 'Heads up' } });
-    fireEvent.change(screen.getByLabelText('MessagingAnnouncement.BODY'), { target: { value: 'Schedule changed.' } });
-    fireEvent.click(screen.getAllByRole('button', { name: 'MessagingAnnouncement.SEND' })[0]);
-    await waitFor(() => expect(api.createMessage).toHaveBeenCalled());
-    const [, payload] = api.createMessage.mock.calls[0];
-    expect(payload).toMatchObject({ kind: 'announcement', title: 'Heads up', body: 'Schedule changed.', link_target: '/events/4/info' });
-  });
-
-  it('carries title and link_target into the optimistic row, not just the final REST payload', async () => {
-    const api = baseApi(); api.createMessage = vi.fn().mockReturnValue(new Promise(() => {}));
-    render(<MessagingProvider api={api} active={false}><Composer conversationId={4} allowAnnouncement linkTarget="/events/4/info" /><CacheState /></MessagingProvider>);
-    fireEvent.click(screen.getByRole('button', { name: 'MessagingAnnouncement.CREATE' }));
-    fireEvent.change(screen.getByLabelText('MessagingAnnouncement.TITLE'), { target: { value: 'Heads up' } });
-    fireEvent.click(screen.getAllByRole('button', { name: 'MessagingAnnouncement.SEND' })[0]);
-    await waitFor(() => {
-      const [optimistic] = Object.values(JSON.parse(screen.getByTestId('cache').textContent).messages);
-      expect(optimistic.status).toBe('pending');
-      expect(optimistic.title).toBe('Heads up');
-      expect(optimistic.link_target).toBe('/events/4/info');
-    });
-  });
-
-  it('does not offer announcement composing unless the host opts in', () => {
-    const api = baseApi();
-    render(<MessagingProvider api={api} active={false}><Composer conversationId={4} /></MessagingProvider>);
-    expect(screen.queryByRole('button', { name: 'MessagingAnnouncement.CREATE' })).toBeNull();
-  });
-
-  it('never offers announcement composing in a direct conversation, even when the host allows it', () => {
-    // A host's allowAnnouncement is typically an event-manager permission check
-    // with no notion of which conversation is currently open — broadcasting to
-    // a 1:1 direct chat is never meaningful regardless of that permission.
-    const api = baseApi();
-    render(<MessagingProvider api={api} active={false}><Composer conversationId={4} allowAnnouncement conversation={{ kind: 'direct' }} /></MessagingProvider>);
-    expect(screen.queryByRole('button', { name: 'MessagingAnnouncement.CREATE' })).toBeNull();
-  });
-
-  it('still offers announcement composing for a non-direct conversation when the host allows it', () => {
-    const api = baseApi();
-    render(<MessagingProvider api={api} active={false}><Composer conversationId={4} allowAnnouncement conversation={{ kind: 'group' }} /></MessagingProvider>);
-    expect(screen.getByRole('button', { name: 'MessagingAnnouncement.CREATE' })).toBeTruthy();
-  });
-
-  it('disables the chip while pending, then rolls back and re-enables it on API failure', async () => {
-    let rejectAdd; const api = baseApi(); api.addReaction.mockReturnValue(new Promise((_, reject) => { rejectAdd = reject; }));
-    render(<MessagingProvider api={api} active={false}><ReactionBar message={{ id: 7, reactions: [{ emoji: '👍', count: 1, reacted: false }] }} /></MessagingProvider>);
-    const chip = screen.getByRole('button', { name: 'MessagingReactions.TOGGLE' });
-    fireEvent.click(chip);
-    await waitFor(() => expect(chip.className.includes('Mui-disabled')).toBe(true));
-    rejectAdd({ response: { data: { detail: 'Reactions disabled' } } });
-    await screen.findByRole('alert');
-    expect(chip.className.includes('Mui-disabled')).toBe(false);
-  });
-
-  it('ignores a second click on the same reaction while the first toggle is still in flight', async () => {
-    let resolveAdd; const api = baseApi(); api.addReaction.mockReturnValue(new Promise((resolve) => { resolveAdd = resolve; }));
-    render(<MessagingProvider api={api} active={false}><ReactionBar message={{ id: 7, reactions: [{ emoji: '👍', count: 1, reacted: false }] }} /></MessagingProvider>);
-    const chip = screen.getByRole('button', { name: 'MessagingReactions.TOGGLE' });
-    fireEvent.click(chip); fireEvent.click(chip);
-    resolveAdd({ reactions: [{ emoji: '👍', count: 2, reacted: true }] });
-    await waitFor(() => expect(api.addReaction).toHaveBeenCalledTimes(1));
-  });
-});

package/tests/messagingLastMessagePreview.test.jsx

@@ -1,93 +0,0 @@
-// @vitest-environment jsdom
-import React from 'react';
-import { afterEach, describe, expect, it, vi } from 'vitest';
-import { cleanup, render, screen, waitFor } from '@testing-library/react';
-
-vi.mock('react-i18next', () => ({ useTranslation: () => ({ t: (key) => key }) }));
-
-const realtimeSubscribers = new Map();
-vi.mock('../src/notifications/realtime', () => ({
-  useRealtime: () => ({
-    subscribe: (envelope, handler) => { realtimeSubscribers.set(envelope, handler); return () => realtimeSubscribers.delete(envelope); },
-    onReconnect: () => () => {},
-  }),
-}));
-function dispatchMessagingFrame(frame) { realtimeSubscribers.get('messaging')?.(frame); }
-
-import { ConversationList } from '../src/messaging/ConversationList';
-import { applyFrame, MessagingProvider, useMessaging } from '../src/messaging/MessagingProvider';
-
-function cache() { return { conversations: {}, messages: {}, threads: {}, polls: {}, reactions: {}, receipts: {}, unread: { unread_count: 0, by_conversation: {} }, cursors: { conversations: null, messages: {}, threads: {} } }; }
-
-afterEach(() => { cleanup(); realtimeSubscribers.clear(); });
-
-describe('last-message preview and conversation reorder', () => {
-  it('moves a conversation to the top and updates its preview from a server-shaped conversation_upsert frame', async () => {
-    const api = {
-      listConversations: vi.fn().mockResolvedValue({
-        results: [
-          { id: 1, title: 'Older', last_message_at: '2026-07-31T09:00:00Z', last_message: { excerpt: 'Old preview' } },
-          { id: 2, title: 'Newer', last_message_at: '2026-07-31T10:00:00Z', last_message: { excerpt: 'Newer preview' } },
-        ],
-        next_cursor: null,
-      }),
-      getUnreadCount: vi.fn().mockResolvedValue({ unread_count: 0, by_conversation: {} }),
-      listMessages: vi.fn().mockResolvedValue({ results: [] }),
-    };
-    render(<MessagingProvider api={api}><ConversationList /></MessagingProvider>);
-    await screen.findByText('Newer preview');
-    // Server-shaped conversation_upsert payload: serialize_conversation_core's
-    // real keys (id, last_message_at, last_message: {excerpt}), not a
-    // client-invented shape.
-    dispatchMessagingFrame({
-      envelope: 'messaging', type: 'conversation_upsert', event_id: 'evt-1', conversation_id: 1,
-      id: '1', last_message_at: '2026-07-31T12:00:00Z', last_message: { excerpt: 'Fresh reply' },
-    });
-    await screen.findByText('Fresh reply');
-    const rows = screen.getAllByRole('button').map((el) => el.textContent);
-    const olderIndex = rows.findIndex((text) => text.includes('Older'));
-    const newerIndex = rows.findIndex((text) => text.includes('Newer'));
-    expect(olderIndex).toBeGreaterThanOrEqual(0);
-    expect(olderIndex).toBeLessThan(newerIndex);
-  });
-
-  it('reflects an edited or deleted last message through a REST refresh', async () => {
-    const api = {
-      listConversations: vi.fn()
-        .mockResolvedValueOnce({ results: [{ id: 1, title: 'Chat', last_message: { excerpt: 'Original text' } }], next_cursor: null })
-        .mockResolvedValueOnce({ results: [{ id: 1, title: 'Chat', last_message: { excerpt: '' } }], next_cursor: null }),
-      getUnreadCount: vi.fn().mockResolvedValue({ unread_count: 0, by_conversation: {} }),
-      listMessages: vi.fn().mockResolvedValue({ results: [] }),
-    };
-    function Refresher() {
-      const { refreshConversations } = useMessaging();
-      return <button onClick={() => refreshConversations()}>refresh</button>;
-    }
-    render(<MessagingProvider api={api}><ConversationList /><Refresher /></MessagingProvider>);
-    await screen.findByText('Original text');
-    screen.getByText('refresh').click();
-    await waitFor(() => expect(screen.queryByText('Original text')).toBeNull());
-    await screen.findByText('MessagingList.NO_MESSAGES');
-  });
-
-  it('applies a real message_edited frame (dcm always attaches frame.message) and updates the edited body', () => {
-    const state = { ...cache(), messages: { 'msg-1': { id: 'msg-1', conversation_id: 'conv-1', body: 'Before' } } };
-    // dcm's publish_messaging_event (realtime.py) always attaches a full
-    // frame.message = serialize_message(message) for message_edited, never
-    // just {message_id} alone — verified against django_core_micha's actual
-    // realtime.py. This is the real wire shape, not a client-invented one.
-    const next = applyFrame(state, {
-      envelope: 'messaging', type: 'message_edited', event_id: 'evt-2', conversation_id: 'conv-1', message_id: 'msg-1',
-      message: { id: 'msg-1', conversation_id: 'conv-1', body: 'After edit', edited_at: '2026-07-31T12:00:00Z' },
-    }, null);
-    expect(next.messages['msg-1'].body).toBe('After edit');
-    expect(next.messages['msg-1'].edited_at).toBe('2026-07-31T12:00:00Z');
-  });
-
-  it('keys a message_edited frame by message_id defensively, even if a producer ever omitted frame.message', () => {
-    const state = { ...cache(), messages: { 'msg-1': { id: 'msg-1', conversation_id: 'conv-1', body: 'Before' } } };
-    const next = applyFrame(state, { envelope: 'messaging', type: 'message_edited', event_id: 'evt-3', conversation_id: 'conv-1', message_id: 'msg-1' }, null);
-    expect(next.messages['msg-1']).toBeTruthy();
-    expect(next.messages['conv-1']).toBeUndefined();
-  });
-});

package/tests/messagingLinkify.test.jsx

@@ -1,51 +0,0 @@
-// @vitest-environment jsdom
-import React from 'react';
-import { afterEach, describe, expect, it, vi } from 'vitest';
-import { cleanup, render, screen } from '@testing-library/react';
-
-vi.mock('react-i18next', () => ({ useTranslation: () => ({ t: (key) => key, i18n: { language: 'en' } }) }));
-vi.mock('../src/notifications/realtime', () => ({ useRealtime: () => ({ subscribe: () => () => {}, onReconnect: () => () => {} }) }));
-
-import { MessageBubble } from '../src/messaging/MessageBubble';
-import { MessagingProvider } from '../src/messaging/MessagingProvider';
-
-function makeApi() {
-  return {
-    getReadStatus: vi.fn().mockResolvedValue({ all_read: false }),
-    getAttachment: vi.fn(), getAttachmentThumbnail: vi.fn(), addReaction: vi.fn(), removeReaction: vi.fn(),
-    votePoll: vi.fn(), closePoll: vi.fn(),
-  };
-}
-function renderMessage(body) {
-  return render(<MessagingProvider api={makeApi()} active={false}><MessageBubble message={{ id: 1, body, sender: { display_name: 'Ava' } }} conversation={{ kind: 'group' }} /></MessagingProvider>);
-}
-
-afterEach(cleanup);
-
-describe('message URL linkification', () => {
-  it('renders http(s) URLs as safe external links', () => {
-    renderMessage('Open https://example.com/path for details.');
-    const link = screen.getByRole('link', { name: 'https://example.com/path' });
-    expect(link.getAttribute('href')).toBe('https://example.com/path');
-    expect(link.getAttribute('target')).toBe('_blank');
-    expect(link.getAttribute('rel')).toBe('noopener noreferrer');
-  });
-
-  it('does not turn javascript URLs into links', () => {
-    renderMessage('javascript:alert(1)');
-    expect(screen.queryByRole('link')).toBeNull();
-    expect(screen.getByText('javascript:alert(1)')).toBeTruthy();
-  });
-
-  it('leaves surrounding prose outside the link and trims sentence punctuation', () => {
-    const { container } = renderMessage('See https://example.com/path, then continue reading.');
-    const link = screen.getByRole('link');
-    expect(link.textContent).toBe('https://example.com/path');
-    // MUI's `Link` itself carries a `MuiTypography-root` class (it is a
-    // Typography variant), so `.closest('.MuiTypography-root')` would match
-    // the link, not its containing paragraph -- `.closest('p')` skips past it
-    // to the actual message-body Typography (MUI's default root tag for the
-    // unvaried `body1` variant used here).
-    expect(link.closest('p').textContent).toBe('See https://example.com/path, then continue reading.');
-  });
-});