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

package/tests/messagingMsg6h.test.jsx

@@ -1,136 +0,0 @@
-// @vitest-environment jsdom
-import React from 'react';
-import { afterEach, describe, expect, it, vi } from 'vitest';
-import { act, cleanup, fireEvent, render, screen, waitFor } from '@testing-library/react';
-
-vi.mock('react-i18next', () => ({ useTranslation: () => ({ t: (key, options) => options && Object.keys(options).length ? `${key}:${JSON.stringify(options)}` : key }) }));
-vi.mock('../src/notifications/realtime', () => ({ useRealtime: () => ({ subscribe: () => () => {}, onReconnect: () => () => {} }) }));
-
-import { AttachmentList } from '../src/messaging/AttachmentList';
-import { AuthContext } from '../src/auth/AuthContext';
-import { MessageBubble } from '../src/messaging/MessageBubble';
-import { MessagingProvider } from '../src/messaging/MessagingProvider';
-
-const imageAttachment = { id: 'att-1', filename: 'vacation-photo.jpg', content_type: 'image/jpeg' };
-const fileAttachment = { id: 'att-2', filename: 'Report.pdf', content_type: 'application/pdf' };
-const fallbackAttachment = { id: 'att-3', content_type: 'application/pdf' }; // no filename -- pre-MSG-12 dcm pin
-
-function makeApi(overrides = {}) {
-  return {
-    listConversations: vi.fn().mockResolvedValue({ results: [] }), listMessages: vi.fn().mockResolvedValue({ results: [] }),
-    getUnreadCount: vi.fn().mockResolvedValue({ unread_count: 0, by_conversation: {} }),
-    getAttachment: vi.fn().mockResolvedValue(new Blob(['full-bytes'], { type: 'image/jpeg' })),
-    getAttachmentThumbnail: vi.fn().mockResolvedValue(new Blob(['thumb-bytes'], { type: 'image/jpeg' })),
-    ...overrides,
-  };
-}
-
-afterEach(cleanup);
-
-describe('MSG-6h: attachment gallery (thumbnail/lightbox/right-click download)', () => {
-  it('1. an image attachment renders a thumbnail; clicking it opens a lightbox showing the full image', async () => {
-    const api = makeApi();
-    render(<MessagingProvider api={api} active={false}><AttachmentList attachments={[imageAttachment]} /></MessagingProvider>);
-    const thumbnail = await screen.findByLabelText(/MessagingAttachments\.PREVIEW/);
-    await waitFor(() => expect(api.getAttachmentThumbnail).toHaveBeenCalledWith('att-1'));
-
-    fireEvent.click(thumbnail);
-    await waitFor(() => expect(api.getAttachment).toHaveBeenCalledWith('att-1'));
-    const lightboxImage = await screen.findByAltText('vacation-photo.jpg');
-    expect(lightboxImage.tagName).toBe('IMG');
-  });
-
-  it('2. right-clicking an image attachment opens a context menu with Download; selecting it downloads', async () => {
-    const api = makeApi();
-    const clickSpy = vi.spyOn(HTMLAnchorElement.prototype, 'click').mockImplementation(() => {});
-    render(<MessagingProvider api={api} active={false}><AttachmentList attachments={[imageAttachment]} /></MessagingProvider>);
-    const thumbnail = await screen.findByLabelText(/MessagingAttachments\.PREVIEW/);
-
-    fireEvent.contextMenu(thumbnail);
-    const downloadItem = await screen.findByText('MessagingAttachments.DOWNLOAD_ACTION');
-    fireEvent.click(downloadItem);
-    await waitFor(() => expect(api.getAttachment).toHaveBeenCalledWith('att-1'));
-    expect(clickSpy).toHaveBeenCalled();
-    clickSpy.mockRestore();
-  });
-
-  it('3. a non-image attachment renders a file indicator, not a broken image; clicking it downloads directly', async () => {
-    const api = makeApi();
-    const clickSpy = vi.spyOn(HTMLAnchorElement.prototype, 'click').mockImplementation(() => {});
-    render(<MessagingProvider api={api} active={false}><AttachmentList attachments={[fileAttachment]} /></MessagingProvider>);
-    const target = screen.getByLabelText('MessagingAttachments.DOWNLOAD:{"name":"Report.pdf"}');
-    expect(target.querySelector('img')).toBeNull();
-    expect(screen.getByText('Report.pdf')).toBeTruthy();
-
-    fireEvent.click(target);
-    await waitFor(() => expect(api.getAttachment).toHaveBeenCalledWith('att-2'));
-    expect(clickSpy).toHaveBeenCalled();
-    clickSpy.mockRestore();
-    // No lightbox for a non-image -- nothing to preview.
-    expect(screen.queryByRole('dialog')).toBeNull();
-  });
-
-  it('4. nameOf prefers attachment.filename when present, falls back to attachment.id when absent (pre-MSG-12 dcm interop)', () => {
-    render(<MessagingProvider api={makeApi()} active={false}><AttachmentList attachments={[fileAttachment, fallbackAttachment]} /></MessagingProvider>);
-    expect(screen.getByText('Report.pdf')).toBeTruthy();
-    expect(screen.getByText('att-3')).toBeTruthy();
-  });
-
-  it('6. the lightbox itself has a Download action -- reachable without a right-click, closing an accessibility regression (touch/keyboard users could not download an image otherwise)', async () => {
-    const api = makeApi();
-    const clickSpy = vi.spyOn(HTMLAnchorElement.prototype, 'click').mockImplementation(() => {});
-    render(<MessagingProvider api={api} active={false}><AttachmentList attachments={[imageAttachment]} /></MessagingProvider>);
-    const thumbnail = await screen.findByLabelText(/MessagingAttachments\.PREVIEW/);
-    fireEvent.click(thumbnail);
-    await screen.findByAltText('vacation-photo.jpg');
-    api.getAttachment.mockClear();
-
-    const downloadButton = screen.getByLabelText('MessagingAttachments.DOWNLOAD_ACTION');
-    fireEvent.click(downloadButton);
-    await waitFor(() => expect(api.getAttachment).toHaveBeenCalledWith('att-1'));
-    expect(clickSpy).toHaveBeenCalled();
-    clickSpy.mockRestore();
-  });
-
-  it('7. the lightbox shows a loading state while the full image fetches, and an error state if the fetch fails', async () => {
-    let resolveFetch;
-    const api = makeApi({ getAttachment: vi.fn(() => new Promise((resolve) => { resolveFetch = resolve; })) });
-    render(<MessagingProvider api={api} active={false}><AttachmentList attachments={[imageAttachment]} /></MessagingProvider>);
-    const thumbnail = await screen.findByLabelText(/MessagingAttachments\.PREVIEW/);
-
-    fireEvent.click(thumbnail);
-    await screen.findByLabelText('MessagingAttachments.LOADING_PREVIEW');
-    expect(screen.queryByAltText('vacation-photo.jpg')).toBeNull();
-
-    resolveFetch(new Blob(['bytes'], { type: 'image/jpeg' }));
-    await screen.findByAltText('vacation-photo.jpg');
-    expect(screen.queryByLabelText('MessagingAttachments.LOADING_PREVIEW')).toBeNull();
-  });
-
-  it('8. a failed full-image fetch shows a visible error, not a silent no-op', async () => {
-    const api = makeApi({ getAttachment: vi.fn().mockRejectedValue(new Error('network error')) });
-    render(<MessagingProvider api={api} active={false}><AttachmentList attachments={[imageAttachment]} /></MessagingProvider>);
-    const thumbnail = await screen.findByLabelText(/MessagingAttachments\.PREVIEW/);
-
-    fireEvent.click(thumbnail);
-    await screen.findByRole('alert');
-    expect(screen.getByRole('alert').textContent).toBe('MessagingAttachments.PREVIEW_ERROR');
-  });
-
-  it('5. the attachment context menu does not also trigger the message-level action menu', async () => {
-    const api = makeApi();
-    render(
-      <AuthContext.Provider value={{ user: { id: 1 } }}>
-        <MessagingProvider api={api} active={false}>
-          <MessageBubble message={{ id: 'm1', body: 'see attached', sender: { id: 1 }, attachments: [imageAttachment] }} conversation={{ kind: 'group' }} canModerateMessages />
-        </MessagingProvider>
-      </AuthContext.Provider>,
-    );
-    const thumbnail = await screen.findByLabelText(/MessagingAttachments\.PREVIEW/);
-
-    fireEvent.contextMenu(thumbnail);
-    await screen.findByText('MessagingAttachments.DOWNLOAD_ACTION');
-    // The message-level menu's own items (e.g. Reply) must not also be open.
-    expect(screen.queryByText('MessagingThread.REPLY')).toBeNull();
-  });
-});

package/tests/messagingMsg6i.test.jsx

@@ -1,129 +0,0 @@
-// @vitest-environment jsdom
-import React from 'react';
-import { afterEach, describe, expect, it, vi } from 'vitest';
-import { cleanup, fireEvent, render, screen, waitFor, within } from '@testing-library/react';
-
-vi.mock('react-i18next', () => ({ useTranslation: () => ({ t: (key, options) => options && Object.keys(options).length ? `${key}:${JSON.stringify(options)}` : key }) }));
-vi.mock('../src/notifications/realtime', () => ({ useRealtime: () => ({ subscribe: () => () => {}, onReconnect: () => () => {} }) }));
-
-import { AttachmentList } from '../src/messaging/AttachmentList';
-import { AuthContext } from '../src/auth/AuthContext';
-import { Composer } from '../src/messaging/Composer';
-import { MessagingProvider } from '../src/messaging/MessagingProvider';
-import { PollCard } from '../src/messaging/PollCard';
-import { QUICK_EMOJIS } from '../src/messaging/ReactionBar';
-import { Thread } from '../src/messaging/Thread';
-
-function makeApi(overrides = {}) {
-  return {
-    listConversations: vi.fn().mockResolvedValue({ results: [] }), listMessages: vi.fn().mockResolvedValue({ results: [] }),
-    getUnreadCount: vi.fn().mockResolvedValue({ unread_count: 0, by_conversation: {} }),
-    getAttachment: vi.fn().mockResolvedValue(new Blob(['bytes'], { type: 'image/jpeg' })),
-    getAttachmentThumbnail: vi.fn().mockResolvedValue(new Blob(['thumb'], { type: 'image/jpeg' })),
-    createMessage: vi.fn(), uploadAttachments: vi.fn(),
-    ...overrides,
-  };
-}
-
-afterEach(cleanup);
-
-describe('MSG-6i: attachment tile size, poll spacing/denominator, Composer emoji picker', () => {
-  it('1. an attachment tile renders at 120x120 (image and non-image)', async () => {
-    const image = { id: 'a1', filename: 'photo.jpg', content_type: 'image/jpeg' };
-    const file = { id: 'a2', filename: 'doc.pdf', content_type: 'application/pdf' };
-    render(<MessagingProvider api={makeApi()} active={false}><AttachmentList attachments={[image, file]} /></MessagingProvider>);
-    const tiles = await screen.findAllByRole('button');
-    const gallery = tiles.filter((tile) => tile.getAttribute('title') === 'photo.jpg' || tile.getAttribute('title') === 'doc.pdf');
-    expect(gallery).toHaveLength(2);
-    gallery.forEach((tile) => {
-      // MUI applies `sx={{ width, height }}` via a generated CSS class, not
-      // an inline style attribute -- read the resolved value instead.
-      expect(getComputedStyle(tile).width).toBe('120px');
-      expect(getComputedStyle(tile).height).toBe('120px');
-    });
-  });
-
-  it('2. poll percentage denominator is the count of distinct respondents, not sum(vote_count) -- must fail against the old sum(vote_count) code for a multi-select poll where one voter picked two options', () => {
-    // One respondent (voter-1) voted for both options. sum(vote_count) = 2,
-    // respondent count = 1. Old code: 1/2 = 50% each. New code: 1/1 = 100% each.
-    const poll = {
-      id: 'poll-1', question: 'Q', allow_multiple: true, closed_at: null,
-      options: [
-        { id: 'opt-a', text: 'A', order: 0, vote_count: 1, voters: ['voter-1'] },
-        { id: 'opt-b', text: 'B', order: 1, vote_count: 1, voters: ['voter-1'] },
-      ],
-    };
-    render(<MessagingProvider api={makeApi()} active={false}><PollCard message={{ id: 'msg-1', poll }} /></MessagingProvider>);
-    const results = screen.getAllByText(/MessagingPoll\.OPTION_RESULT/);
-    expect(results).toHaveLength(2);
-    results.forEach((node) => {
-      expect(node.textContent).toMatch(/"count":1/);
-      expect(node.textContent).toMatch(/"percent":100/);
-    });
-  });
-
-  it('3. poll percentage denominator: zero respondents still renders 0%, never NaN (no regression)', () => {
-    const poll = {
-      id: 'poll-2', question: 'Q', allow_multiple: false, closed_at: null,
-      options: [
-        { id: 'opt-a', text: 'A', order: 0, vote_count: 0, voters: [] },
-        { id: 'opt-b', text: 'B', order: 1, vote_count: 0, voters: [] },
-      ],
-    };
-    render(<MessagingProvider api={makeApi()} active={false}><PollCard message={{ id: 'msg-2', poll }} /></MessagingProvider>);
-    const results = screen.getAllByText(/MessagingPoll\.OPTION_RESULT/);
-    results.forEach((node) => {
-      expect(node.textContent).toMatch(/"count":0/);
-      expect(node.textContent).toMatch(/"percent":0/);
-      expect(node.textContent).not.toMatch(/NaN/);
-    });
-  });
-
-  it('4. the Composer emoji picker shows more than 5 options (the wider curated grid, not ReactionBar\'s QUICK_EMOJIS)', async () => {
-    const api = makeApi();
-    render(<MessagingProvider api={api} active={false}><Composer conversationId={4} /></MessagingProvider>);
-    fireEvent.click(screen.getByRole('button', { name: 'MessagingComposer.ADD_EMOJI' }));
-    const grid = await screen.findByRole('group', { name: 'MessagingComposer.EMOJI_PICKER' });
-    const emojiButtons = grid.querySelectorAll('button');
-    expect(emojiButtons.length).toBeGreaterThan(5);
-  });
-
-  it('4b. selecting an emoji from the Composer grid inserts it at the cursor position, not appended', async () => {
-    const api = makeApi();
-    render(<MessagingProvider api={api} active={false}><Composer conversationId={4} /></MessagingProvider>);
-    const input = screen.getByLabelText('MessagingComposer.MESSAGE');
-    fireEvent.change(input, { target: { value: 'abCD' } });
-    input.setSelectionRange(2, 2);
-    fireEvent.click(screen.getByRole('button', { name: 'MessagingComposer.ADD_EMOJI' }));
-    const grid = await screen.findByRole('group', { name: 'MessagingComposer.EMOJI_PICKER' });
-    fireEvent.click(within(grid).getByRole('button', { name: '😀' }));
-    await waitFor(() => expect(input.value).toBe('ab😀CD'));
-  });
-
-  it('5. ReactionBar\'s own QUICK_EMOJIS stays exactly 5, unaffected by the wider Composer picker', () => {
-    expect(QUICK_EMOJIS).toHaveLength(5);
-  });
-
-  it('6. Thread\'s root Stack and scroll Box carry flex:1/minHeight:0 (MSG-18 companion fix), so it can shrink inside a host\'s flex-column layout', async () => {
-    const api = makeApi({
-      listConversations: vi.fn().mockResolvedValue({ results: [{ id: 1, kind: 'group' }] }),
-      listMessages: vi.fn().mockResolvedValue({ results: [{ id: 2, conversation_id: 1, body: 'hi' }], next_cursor: null }),
-      listThread: vi.fn().mockResolvedValue({ results: [] }),
-      getReadStatus: vi.fn().mockResolvedValue({ all_read: true }),
-    });
-    render(
-      <AuthContext.Provider value={{ user: { id: 9 } }}>
-        <MessagingProvider api={api}><Thread conversationId={1} /></MessagingProvider>
-      </AuthContext.Provider>,
-    );
-    const timeline = await screen.findByLabelText('MessagingThread.TIMELINE');
-    // The scroll Box is a direct child of Thread's root Stack.
-    const rootStack = timeline.parentElement;
-    [timeline, rootStack].forEach((el) => {
-      expect(getComputedStyle(el).flexGrow).toBe('1');
-      expect(getComputedStyle(el).minHeight).toBe('0px');
-    });
-    // maxHeight stays as the fallback cap for a host that doesn't flex-wrap Thread.
-    expect(getComputedStyle(timeline).maxHeight).toBe('560px');
-  });
-});

package/tests/messagingPollRendering.test.jsx

@@ -1,142 +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 && Object.keys(options).length ? `${key}:${JSON.stringify(options)}` : 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 { AuthContext } from '../src/auth/AuthContext';
-import { PollCard } from '../src/messaging/PollCard';
-import { MessagingProvider, useMessaging } from '../src/messaging/MessagingProvider';
-
-function baseApi(overrides = {}) {
-  return {
-    listConversations: vi.fn().mockResolvedValue({ results: [] }), listMessages: vi.fn().mockResolvedValue({ results: [] }),
-    getUnreadCount: vi.fn().mockResolvedValue({ unread_count: 0, by_conversation: {} }),
-    votePoll: vi.fn(), closePoll: vi.fn(),
-    ...overrides,
-  };
-}
-
-// A genuinely server-shaped serialize_poll payload (django_core_micha
-// serializers.py:52-64): id/question/allow_multiple/closed_at/created_by_id,
-// options carry {id,text,order,vote_count,voters} — never selected/voted
-// flags, never created_by as an object. voted_option_ids is added ONLY on
-// the vote/create/close REST responses (views.py _poll_response), never here.
-function serverPoll(overrides = {}) {
-  return {
-    id: 'poll-1', question: 'Where should we meet?', allow_multiple: false, closed_at: null, created_by_id: 'user-9',
-    options: [
-      { id: 'opt-a', text: 'Library', order: 0, vote_count: 2, voters: ['user-1', 'user-2'] },
-      { id: 'opt-b', text: 'Cafe', order: 1, vote_count: 1, voters: ['user-3'] },
-    ],
-    ...overrides,
-  };
-}
-function State() { const { cache } = useMessaging(); return <output data-testid="cache">{JSON.stringify(cache)}</output>; }
-// PollCard is a "dumb" prop-driven component (matches Thread.jsx's usage,
-// which passes down cache.messages[id] directly) — it does not subscribe to
-// the cache itself. A live-frame test must read the message back out of the
-// cache, not hold a static prop, or it can never observe a frame update.
-function ConnectedPollCard({ messageId }) {
-  const { cache } = useMessaging();
-  return <PollCard message={cache.messages[messageId]} />;
-}
-
-afterEach(() => { cleanup(); realtimeSubscribers.clear(); });
-
-describe('poll rendering against the real server contract', () => {
-  it('renders question, options and proportional vote counts from a server-shaped payload', () => {
-    // totalVotes = 2 + 1 = 3 -> Library 67%, Cafe 33%.
-    render(<MessagingProvider api={baseApi()} active={false}><PollCard message={{ id: 'msg-1', poll: serverPoll() }} /></MessagingProvider>);
-    expect(screen.getByText('Where should we meet?')).toBeTruthy();
-    expect(screen.getByText('Library')).toBeTruthy();
-    expect(screen.getByText(/MessagingPoll\.OPTION_RESULT.*"count":2.*"percent":67/)).toBeTruthy();
-    expect(screen.getByText('Cafe')).toBeTruthy();
-    expect(screen.getByText(/MessagingPoll\.OPTION_RESULT.*"count":1.*"percent":33/)).toBeTruthy();
-  });
-
-  it('derives selection from voted_option_ids (REST-only) and marks the viewer\'s own vote', () => {
-    // Single-choice (allow_multiple: false) renders radios, not checkboxes --
-    // native radio semantics match the WO's "tapping a different option moves
-    // the vote, tapping the same one is a no-op" design directly.
-    const poll = serverPoll({ voted_option_ids: ['opt-a'] });
-    render(<MessagingProvider api={baseApi()} active={false}><PollCard message={{ id: 'msg-1', poll }} /></MessagingProvider>);
-    const radios = screen.getAllByRole('radio');
-    expect(radios[0].checked).toBe(true);
-    expect(radios[1].checked).toBe(false);
-    // Count/percent and the viewer's own vote both live in the accessible
-    // name, not conveyed by the bar's width alone (WO accessibility risk).
-    expect(radios[0].getAttribute('aria-label')).toMatch(/Library.*MessagingPoll\.OPTION_RESULT.*MessagingPoll\.YOU_VOTED/);
-    expect(radios[1].getAttribute('aria-label')).not.toMatch(/YOU_VOTED/);
-  });
-
-  it('does not clear or invert the viewer\'s own vote when a poll_updated frame carries no voted_option_ids', async () => {
-    const api = baseApi();
-    const votedPoll = serverPoll({ voted_option_ids: ['opt-a'] });
-    render(<MessagingProvider api={api} active={false}><ConnectedPollCard messageId="msg-1" /><State /></MessagingProvider>);
-    // Seed the cache the way a real poll message arrives: a `message` frame
-    // (kind: 'poll') carrying the just-voted poll, exactly as this viewer's
-    // own castPollVote reconciliation would leave it — not a static prop the
-    // component can never observe a later frame update against.
-    dispatchMessagingFrame({ envelope: 'messaging', type: 'message', event_id: 'seed', conversation_id: 'conv-1', message: { id: 'msg-1', conversation_id: 'conv-1', kind: 'poll', poll: votedPoll } });
-    await waitFor(() => expect(screen.getAllByRole('radio')[0].checked).toBe(true));
-    // The real poll_updated frame shape: payload.poll is a bare serialize_poll
-    // result (verified against services.py's _poll_updated_payload) — never
-    // carries voted_option_ids. A vote_count bump from another viewer must
-    // not touch this viewer's own selection.
-    dispatchMessagingFrame({
-      envelope: 'messaging', type: 'poll_updated', event_id: 'evt-1', conversation_id: 'conv-1', message_id: 'msg-1', poll_id: 'poll-1',
-      poll: { ...serverPoll(), options: [{ id: 'opt-a', text: 'Library', order: 0, vote_count: 3, voters: ['user-1', 'user-2', 'user-4'] }, { id: 'opt-b', text: 'Cafe', order: 1, vote_count: 1, voters: ['user-3'] }] },
-    });
-    await waitFor(() => expect(screen.getByText(/MessagingPoll\.OPTION_RESULT.*"count":3/)).toBeTruthy());
-    // The frame carries no voted_option_ids at all — the viewer's REST-known
-    // vote must be carried forward, not wiped, by this update.
-    expect(screen.getAllByRole('radio')[0].checked).toBe(true);
-  });
-
-  it('grants close only to the poll creator (created_by_id) or a host override, never a guess from an object shape', () => {
-    const poll = serverPoll({ created_by_id: 'user-9' });
-    const { rerender } = render(<AuthContext.Provider value={{ user: { id: 'user-9' } }}><MessagingProvider api={baseApi()} active={false}><PollCard message={{ id: 'msg-1', poll }} /></MessagingProvider></AuthContext.Provider>);
-    expect(screen.getByRole('button', { name: 'MessagingPoll.CLOSE' })).toBeTruthy();
-    rerender(<AuthContext.Provider value={{ user: { id: 'user-2' } }}><MessagingProvider api={baseApi()} active={false}><PollCard message={{ id: 'msg-1', poll }} /></MessagingProvider></AuthContext.Provider>);
-    expect(screen.queryByRole('button', { name: 'MessagingPoll.CLOSE' })).toBeNull();
-    rerender(<AuthContext.Provider value={{ user: { id: 'user-2' } }}><MessagingProvider api={baseApi()} active={false}><PollCard canClose message={{ id: 'msg-1', poll }} /></MessagingProvider></AuthContext.Provider>);
-    expect(screen.getByRole('button', { name: 'MessagingPoll.CLOSE' })).toBeTruthy();
-  });
-
-  it('disables voting once the poll is closed and shows a closed indicator, not merely a missing button', () => {
-    // With no separate submit button (MSG-6g), "closed" can no longer be
-    // inferred from its absence -- must be an explicit, visible indicator.
-    const poll = serverPoll({ closed_at: '2026-07-31T10:00:00Z' });
-    render(<MessagingProvider api={baseApi()} active={false}><PollCard message={{ id: 'msg-1', poll }} /></MessagingProvider>);
-    screen.getAllByRole('radio').forEach((radio) => expect(radio.disabled).toBe(true));
-    expect(screen.getByText('MessagingPoll.CLOSED')).toBeTruthy();
-  });
-
-  it('gives the card a minimum width so a short question/option set is not squeezed narrower than comfortable (MSG-6j)', () => {
-    render(<MessagingProvider api={baseApi()} active={false}><PollCard message={{ id: 'msg-1', poll: serverPoll() }} /></MessagingProvider>);
-    const card = screen.getByLabelText('MessagingPoll.LABEL');
-    expect(getComputedStyle(card).minWidth).toBe('260px');
-  });
-
-  it('keeps a real gap between an option label and its vote-count/percent text, not merely space-between with nothing to distribute (MSG-6j)', () => {
-    // Regression test for the reported bug: `justifyContent: 'space-between'` alone only
-    // distributes LEFTOVER width, and this row sits inside a `width: fit-content` bubble
-    // sized around this exact content -- there is never leftover width, so space-between
-    // reserved a zero-width gap and the option text ran directly into "2 (100 %)". Must
-    // fail if the row's own `spacing` is removed.
-    render(<MessagingProvider api={baseApi()} active={false}><PollCard message={{ id: 'msg-1', poll: serverPoll() }} /></MessagingProvider>);
-    const resultText = screen.getByText(/MessagingPoll\.OPTION_RESULT.*"count":2.*"percent":67/);
-    expect(getComputedStyle(resultText).marginLeft).not.toBe('0px');
-  });
-});

package/tests/messagingReactionMerge.test.jsx

@@ -1,77 +0,0 @@
-// @vitest-environment jsdom
-import React from 'react';
-import { afterEach, describe, expect, it, vi } from 'vitest';
-import { act, cleanup, fireEvent, render, screen, waitFor } from '@testing-library/react';
-
-vi.mock('react-i18next', () => ({ useTranslation: () => ({ t: (key, options) => options?.emoji ? `${key}:${options.emoji}:${options.count}` : 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 { ReactionBar } from '../src/messaging/ReactionBar';
-import { MessagingProvider, useMessaging } from '../src/messaging/MessagingProvider';
-
-function baseApi(overrides = {}) {
-  return {
-    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(),
-    ...overrides,
-  };
-}
-// ReactionBar is prop-driven (matches every other messaging leaf component) —
-// a live-frame test must read the message back out of the cache.
-function ConnectedReactionBar({ messageId }) {
-  const { cache } = useMessaging();
-  return <ReactionBar message={cache.messages[messageId]} expanded />;
-}
-
-afterEach(() => { cleanup(); realtimeSubscribers.clear(); });
-
-describe('reaction frame/REST merge against the real aggregate-only contract', () => {
-  it('an unrelated reaction frame updates its own emoji without touching the viewer\'s known reacted state on other emoji', async () => {
-    const api = baseApi();
-    render(<MessagingProvider api={api} active={false}><ConnectedReactionBar messageId="msg-1" /></MessagingProvider>);
-    // Seed: the viewer has already reacted with 👍 (locally known — e.g. from
-    // an earlier REST confirmation), the server aggregate never carries that.
-    dispatchMessagingFrame({ envelope: 'messaging', type: 'message', event_id: 'seed', conversation_id: 'conv-1', message: { id: 'msg-1', conversation_id: 'conv-1', body: 'Hi', reactions: [{ emoji: '👍', count: 1, reacted: true }] } });
-    await waitFor(() => expect(screen.getByRole('button', { name: 'MessagingReactions.TOGGLE:👍:1' })).toBeTruthy());
-    // A different viewer reacts with 🎉 — dcm's real reaction frame shape
-    // (serialize_reactions): {emoji, count} only, never reacted, for the
-    // WHOLE aggregate list including 👍.
-    dispatchMessagingFrame({ envelope: 'messaging', type: 'reaction', event_id: 'evt-1', conversation_id: 'conv-1', message_id: 'msg-1', reactions: [{ emoji: '👍', count: 1 }, { emoji: '🎉', count: 1 }] });
-    await waitFor(() => expect(screen.getByRole('button', { name: 'MessagingReactions.TOGGLE:🎉:1' })).toBeTruthy());
-    const thumbsUp = screen.getByRole('button', { name: 'MessagingReactions.TOGGLE:👍:1' });
-    expect(thumbsUp.className.includes('MuiChip-colorPrimary')).toBe(true); // still marked as the viewer's own reaction
-  });
-
-  it('the viewer\'s own optimistic reacted flag survives its own REST confirmation, which returns an aggregate-only shape', async () => {
-    const api = baseApi({ addReaction: vi.fn().mockResolvedValue({ reactions: [{ emoji: '👍', count: 1 }] }) });
-    render(<MessagingProvider api={api} active={false}><ConnectedReactionBar messageId="msg-1" /></MessagingProvider>);
-    act(() => { dispatchMessagingFrame({ envelope: 'messaging', type: 'message', event_id: 'seed', conversation_id: 'conv-1', message: { id: 'msg-1', conversation_id: 'conv-1', body: 'Hi', reactions: [] } }); });
-    fireEvent.click(screen.getByText('👍'));
-    await waitFor(() => expect(api.addReaction).toHaveBeenCalledWith('msg-1', '👍'));
-    await waitFor(() => {
-      const chip = screen.getByRole('button', { name: 'MessagingReactions.TOGGLE:👍:1' });
-      expect(chip.className.includes('MuiChip-colorPrimary')).toBe(true);
-    });
-  });
-
-  it('a duplicate event_id changes nothing', async () => {
-    const api = baseApi();
-    render(<MessagingProvider api={api} active={false}><ConnectedReactionBar messageId="msg-1" /></MessagingProvider>);
-    dispatchMessagingFrame({ envelope: 'messaging', type: 'message', event_id: 'seed', conversation_id: 'conv-1', message: { id: 'msg-1', conversation_id: 'conv-1', body: 'Hi', reactions: [{ emoji: '👍', count: 1 }] } });
-    await waitFor(() => screen.getByRole('button', { name: 'MessagingReactions.TOGGLE:👍:1' }));
-    dispatchMessagingFrame({ envelope: 'messaging', type: 'reaction', event_id: 'dup-1', conversation_id: 'conv-1', message_id: 'msg-1', reactions: [{ emoji: '👍', count: 2 }] });
-    await waitFor(() => expect(screen.getByRole('button', { name: 'MessagingReactions.TOGGLE:👍:2' })).toBeTruthy());
-    dispatchMessagingFrame({ envelope: 'messaging', type: 'reaction', event_id: 'dup-1', conversation_id: 'conv-1', message_id: 'msg-1', reactions: [{ emoji: '👍', count: 99 }] });
-    expect(screen.queryByRole('button', { name: 'MessagingReactions.TOGGLE:👍:99' })).toBeNull();
-    expect(screen.getByRole('button', { name: 'MessagingReactions.TOGGLE:👍:2' })).toBeTruthy();
-  });
-});

package/tests/messagingReadTicksTooltip.test.jsx

@@ -1,36 +0,0 @@
-// @vitest-environment jsdom
-import React from 'react';
-import { afterEach, describe, expect, it, vi } from 'vitest';
-import { act, cleanup, render, screen, waitFor } from '@testing-library/react';
-
-vi.mock('react-i18next', () => ({ useTranslation: () => ({ t: (key, options) => options?.read != null ? `${key}:${options.read}/${options.total}` : key }) }));
-vi.mock('../src/notifications/realtime', () => ({ useRealtime: () => ({ subscribe: () => () => {}, onReconnect: () => () => {} }) }));
-
-import { MessagingProvider } from '../src/messaging/MessagingProvider';
-import { ReadTicks } from '../src/messaging/ReadTicks';
-
-function makeApi(status) {
-  return {
-    getReadStatus: vi.fn().mockResolvedValue(status),
-    getAttachment: vi.fn(), getAttachmentThumbnail: vi.fn(), addReaction: vi.fn(), removeReaction: vi.fn(),
-    votePoll: vi.fn(), closePoll: vi.fn(),
-  };
-}
-
-afterEach(cleanup);
-
-describe('message read ratio marker', () => {
-  it('gives the visible ratio an explanatory tooltip', async () => {
-    render(<MessagingProvider api={makeApi({ read_count: 2, recipient_count: 5 })} active={false}><ReadTicks messageId={7} conversation={{ kind: 'group' }} /></MessagingProvider>);
-    expect(await screen.findByTitle('MessagingReadTicks.READ_RATIO:2/5')).toBeTruthy();
-  });
-
-  it('hides a ratio when there are no recipients', async () => {
-    let resolveStatus;
-    const api = { ...makeApi({}), getReadStatus: vi.fn().mockReturnValue(new Promise((resolve) => { resolveStatus = resolve; })) };
-    const { container } = render(<MessagingProvider api={api} active={false}><ReadTicks messageId={7} conversation={{ kind: 'group' }} /></MessagingProvider>);
-    await waitFor(() => expect(api.getReadStatus).toHaveBeenCalled());
-    await act(async () => resolveStatus({ read_count: 0, recipient_count: 0 }));
-    expect(container.textContent).toBe('');
-  });
-});