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

package/tests/PasswordSetForm.test.jsx

@@ -1,70 +0,0 @@
-// @vitest-environment jsdom
-import React from 'react';
-import { cleanup, fireEvent, render, screen } from '@testing-library/react';
-import { afterEach, describe, expect, it, vi } from 'vitest';
-
-vi.mock('react-i18next', () => ({
-  useTranslation: () => ({ t: (key, fallback) => (typeof fallback === 'string' ? fallback : key) }),
-}));
-
-import { PasswordSetForm } from '../src/components/PasswordSetForm';
-
-afterEach(() => {
-  cleanup();
-});
-
-function fillAndSubmit(password1, password2) {
-  fireEvent.change(screen.getByLabelText('Auth.NEW_PASSWORD_LABEL'), { target: { value: password1 } });
-  fireEvent.change(screen.getByLabelText('Auth.PASSWORD_CONFIRM_LABEL'), { target: { value: password2 } });
-  fireEvent.click(screen.getByRole('button', { name: 'Auth.PASSWORD_SET_BUTTON' }));
-}
-
-describe('PasswordSetForm', () => {
-  it('renders the default password-rules helperText', () => {
-    render(<PasswordSetForm onSubmit={vi.fn()} />);
-    expect(screen.getAllByText('Auth.PASSWORD_RULES_HINT').length).toBeGreaterThan(0);
-  });
-
-  it('lets a caller override the helperText via passwordRulesHint', () => {
-    render(<PasswordSetForm onSubmit={vi.fn()} passwordRulesHint="Custom rule text" />);
-    expect(screen.getAllByText('Custom rule text').length).toBeGreaterThan(0);
-    expect(screen.queryByText('Auth.PASSWORD_RULES_HINT')).toBeNull();
-  });
-
-  it('blocks submit and shows the local error for a password shorter than 8 characters', () => {
-    const onSubmit = vi.fn();
-    render(<PasswordSetForm onSubmit={onSubmit} />);
-    fillAndSubmit('short1', 'short1');
-    expect(screen.getByText('Auth.PASSWORD_TOO_SHORT_LOCAL')).toBeTruthy();
-    expect(onSubmit).not.toHaveBeenCalled();
-  });
-
-  it('blocks submit and shows the local error for an entirely numeric password', () => {
-    const onSubmit = vi.fn();
-    render(<PasswordSetForm onSubmit={onSubmit} />);
-    fillAndSubmit('12345678', '12345678');
-    expect(screen.getByText('Auth.PASSWORD_NUMERIC_LOCAL')).toBeTruthy();
-    expect(onSubmit).not.toHaveBeenCalled();
-  });
-
-  it('calls onSubmit for a valid, non-numeric password of sufficient length', () => {
-    const onSubmit = vi.fn();
-    render(<PasswordSetForm onSubmit={onSubmit} />);
-    fillAndSubmit('correct-horse-1', 'correct-horse-1');
-    expect(onSubmit).toHaveBeenCalledWith('correct-horse-1');
-  });
-
-  it('honours a custom minPasswordLength for an app with a looser backend policy', () => {
-    const onSubmit = vi.fn();
-    render(<PasswordSetForm onSubmit={onSubmit} minPasswordLength={4} />);
-    fillAndSubmit('ab12', 'ab12');
-    expect(onSubmit).toHaveBeenCalledWith('ab12');
-  });
-
-  it('allows an entirely numeric password when allowNumericPassword is set', () => {
-    const onSubmit = vi.fn();
-    render(<PasswordSetForm onSubmit={onSubmit} allowNumericPassword />);
-    fillAndSubmit('12345678', '12345678');
-    expect(onSubmit).toHaveBeenCalledWith('12345678');
-  });
-});

package/tests/PopupSurface.test.jsx

@@ -1,142 +0,0 @@
-// @vitest-environment jsdom
-import React from 'react';
-import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
-import { act, cleanup, fireEvent, render, screen } from '@testing-library/react';
-
-vi.mock('react-i18next', () => ({
-  useTranslation: () => ({
-    t: (key, params) => (params && Object.keys(params).length ? `${key}:${JSON.stringify(params)}` : key),
-  }),
-}));
-
-const onboardingModule = vi.hoisted(() => ({ useOnboarding: vi.fn() }));
-vi.mock('../src/onboarding/OnboardingProvider', () => onboardingModule);
-
-const notificationsModule = vi.hoisted(() => ({ useNotifications: vi.fn() }));
-vi.mock('../src/notifications/NotificationsProvider', () => notificationsModule);
-
-const realtimeModule = vi.hoisted(() => ({ useRealtime: vi.fn(), DEFAULT_ENVELOPE: 'notification' }));
-vi.mock('../src/notifications/realtime', () => realtimeModule);
-
-import { PopupSurface } from '../src/notifications/PopupSurface';
-
-describe('PopupSurface (NOTIF-12 scope B/C)', () => {
-  let handler;
-  let subscribeFn;
-  let markSeen;
-  let markDismissed;
-
-  beforeEach(() => {
-    vi.clearAllMocks();
-    handler = null;
-    subscribeFn = (envelope, fn) => {
-      handler = fn;
-      return () => { handler = null; };
-    };
-    markSeen = vi.fn();
-    markDismissed = vi.fn();
-    notificationsModule.useNotifications.mockReturnValue({ markSeen, markDismissed, markDone: vi.fn() });
-    onboardingModule.useOnboarding.mockReturnValue({ activeSteps: [] });
-    realtimeModule.useRealtime.mockImplementation(() => ({ subscribe: subscribeFn }));
-  });
-
-  afterEach(() => {
-    cleanup();
-  });
-
-  function push(data) {
-    act(() => {
-      handler(data);
-    });
-  }
-
-  it('renders nothing when no popup-eligible notification is present', () => {
-    render(<PopupSurface />);
-
-    expect(screen.queryByRole('dialog')).toBeNull();
-  });
-
-  it('renders a popup-eligible notification once, marking seen exactly once on display, and shows the notification title (not the onboarding header)', () => {
-    render(<PopupSurface />);
-
-    push({ notification_id: 1, channel: 'popup', content: { title_key: 'T', body_key: 'B', params: {} } });
-
-    expect(screen.getByRole('dialog')).toBeTruthy();
-    expect(screen.getByText('T')).toBeTruthy();
-    expect(screen.queryByText('Onboarding.SETUP')).toBeNull();
-    expect(markSeen).toHaveBeenCalledTimes(1);
-    expect(markSeen).toHaveBeenCalledWith([1]);
-  });
-
-  it('marks using the wire recipient_id (feed/mark/ id-space), not the notification_id, when present', () => {
-    render(<PopupSurface />);
-
-    push({ notification_id: 10, recipient_id: 999, channel: 'popup', content: { title_key: 'T' } });
-
-    expect(markSeen).toHaveBeenCalledWith([999]);
-
-    fireEvent.click(screen.getByRole('button', { name: 'PopupSurface.CLOSE' }));
-
-    expect(markDismissed).toHaveBeenCalledWith([999]);
-  });
-
-  it('fires markDismissed, never markDone, when the popup is closed', () => {
-    const { markDone } = notificationsModule.useNotifications();
-    render(<PopupSurface />);
-    push({ notification_id: 1, channel: 'popup', content: { title_key: 'T' } });
-
-    fireEvent.click(screen.getByRole('button', { name: 'PopupSurface.CLOSE' }));
-
-    expect(markDismissed).toHaveBeenCalledWith([1]);
-    expect(markDone).not.toHaveBeenCalled();
-    expect(screen.queryByRole('dialog')).toBeNull();
-  });
-
-  it('ignores a chip-channel message and a channel-less legacy message — no popup storm', () => {
-    render(<PopupSurface />);
-
-    push({ notification_id: 2, channel: 'chip', content: { title_key: 'T' } });
-    push({ notification_id: 3, content: { title_key: 'T' } });
-
-    expect(screen.queryByRole('dialog')).toBeNull();
-    expect(markSeen).not.toHaveBeenCalled();
-  });
-
-  it('ignores a notification.status message regardless of its channel field', () => {
-    render(<PopupSurface />);
-
-    push({ type: 'notification.status', notification_id: 4, channel: 'popup', status: {} });
-
-    expect(screen.queryByRole('dialog')).toBeNull();
-  });
-
-  it('coexistence: a blocking onboarding step suppresses the popup; it appears once the step clears, never stacking two dialogs', () => {
-    onboardingModule.useOnboarding.mockReturnValue({ activeSteps: [{ id: 'cookie_consent', blocking: true }] });
-    const { rerender } = render(<PopupSurface />);
-
-    push({ notification_id: 5, channel: 'popup', content: { title_key: 'T' } });
-    expect(screen.queryByRole('dialog')).toBeNull();
-    expect(markSeen).not.toHaveBeenCalled();
-
-    onboardingModule.useOnboarding.mockReturnValue({ activeSteps: [] });
-    rerender(<PopupSurface />);
-
-    expect(screen.getAllByRole('dialog')).toHaveLength(1);
-    expect(markSeen).toHaveBeenCalledWith([5]);
-  });
-
-  it('coexistence: a non-blocking onboarding step also suppresses the popup, never stacking two dialogs', () => {
-    onboardingModule.useOnboarding.mockReturnValue({ activeSteps: [{ id: 'browser_push', blocking: false }] });
-    const { rerender } = render(<PopupSurface />);
-
-    push({ notification_id: 6, channel: 'popup', content: { title_key: 'T' } });
-    expect(screen.queryByRole('dialog')).toBeNull();
-    expect(markSeen).not.toHaveBeenCalled();
-
-    onboardingModule.useOnboarding.mockReturnValue({ activeSteps: [] });
-    rerender(<PopupSurface />);
-
-    expect(screen.getAllByRole('dialog')).toHaveLength(1);
-    expect(markSeen).toHaveBeenCalledWith([6]);
-  });
-});

package/tests/ProfileComponent.refresh.test.jsx

@@ -1,194 +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';
-import { ThemeProvider, createTheme } from '@mui/material/styles';
-
-vi.mock('react-i18next', () => ({
-  useTranslation: () => ({
-    t: (key, fallback) => ({
-      'Profile.USERNAME_LABEL': 'Username',
-      'Auth.EMAIL_LABEL': 'Email',
-      'Profile.FIRST_NAME_LABEL': 'First name',
-      'Profile.LAST_NAME_LABEL': 'Last name',
-      'Profile.ACCEPT_PRIVACY_LABEL': 'Accept privacy statement',
-      'Profile.ACCEPT_COOKIES_LABEL': 'Accept convenience cookies',
-      'Profile.SAVE_BUTTON': 'Save',
-    }[key] ?? fallback ?? key),
-    i18n: { exists: () => true },
-  }),
-}));
-
-vi.mock('../src/auth/authApi', () => ({
-  fetchCookieStatement: vi.fn(),
-  fetchPrivacyStatement: vi.fn(),
-}));
-
-import { AuthContext } from '../src/auth/AuthContext';
-import { ProfileComponent } from '../src/components/ProfileComponent';
-
-const theme = createTheme();
-
-function makeUser(overrides = {}) {
-  return {
-    id: 17,
-    username: 'pat',
-    email: 'pat@example.com',
-    first_name: 'Pat',
-    last_name: 'Example',
-    accepted_privacy_statement: false,
-    accepted_convenience_cookies: true,
-    ...overrides,
-  };
-}
-
-function profileTree(user, onSubmit) {
-  return (
-    <ThemeProvider theme={theme}>
-      <AuthContext.Provider value={{ user, loading: false }}>
-        <ProfileComponent onSubmit={onSubmit} showStatements={false} />
-      </AuthContext.Provider>
-    </ThemeProvider>
-  );
-}
-
-function renderProfile(user, onSubmit = vi.fn()) {
-  const view = render(profileTree(user, onSubmit));
-  return {
-    ...view,
-    onSubmit,
-    setUser: (nextUser) => view.rerender(profileTree(nextUser, onSubmit)),
-  };
-}
-
-afterEach(() => {
-  cleanup();
-  vi.clearAllMocks();
-});
-
-describe('ProfileComponent user refresh', () => {
-  it('refreshes untouched editable fields while retaining the independently edited field', () => {
-    const { setUser } = renderProfile(makeUser());
-    fireEvent.change(screen.getByRole('textbox', { name: 'First name' }), {
-      target: { value: 'Local first' },
-    });
-
-    setUser(makeUser({
-      first_name: 'Server first',
-      last_name: 'Server last',
-      accepted_privacy_statement: true,
-      accepted_convenience_cookies: false,
-    }));
-
-    expect(screen.getByRole('textbox', { name: 'First name' }).value).toBe('Local first');
-    expect(screen.getByRole('textbox', { name: 'Last name' }).value).toBe('Server last');
-    expect(screen.getByRole('checkbox', { name: 'Accept privacy statement' }).checked).toBe(true);
-    expect(screen.getByRole('checkbox', { name: 'Accept convenience cookies' }).checked).toBe(false);
-  });
-
-  it('preserves edits for the same identity and refreshes untouched fields', async () => {
-    const { onSubmit, setUser } = renderProfile(makeUser());
-    fireEvent.change(screen.getByRole('textbox', { name: 'First name' }), {
-      target: { value: 'Patricia' },
-    });
-    fireEvent.change(screen.getByRole('textbox', { name: 'Last name' }), {
-      target: { value: 'Newname' },
-    });
-    fireEvent.click(screen.getByRole('checkbox', { name: 'Accept privacy statement' }));
-    fireEvent.click(screen.getByRole('checkbox', { name: 'Accept convenience cookies' }));
-
-    setUser(makeUser({
-      username: 'pat-updated',
-      email: 'new@example.com',
-      first_name: 'Server first',
-      last_name: 'Server last',
-      accepted_privacy_statement: false,
-      accepted_convenience_cookies: true,
-    }));
-
-    await waitFor(() => {
-      expect(screen.getByRole('textbox', { name: 'Username' }).value).toBe('pat-updated');
-      expect(screen.getByRole('textbox', { name: 'Email' }).value).toBe('new@example.com');
-    });
-    expect(screen.getByRole('textbox', { name: 'First name' }).value).toBe('Patricia');
-    expect(screen.getByRole('textbox', { name: 'Last name' }).value).toBe('Newname');
-    expect(screen.getByRole('checkbox', { name: 'Accept privacy statement' }).checked).toBe(true);
-    expect(screen.getByRole('checkbox', { name: 'Accept convenience cookies' }).checked).toBe(false);
-
-    fireEvent.click(screen.getByRole('button', { name: 'Save' }));
-    await waitFor(() => {
-      expect(onSubmit).toHaveBeenCalledWith({
-        first_name: 'Patricia',
-        last_name: 'Newname',
-        accepted_privacy_statement: true,
-        accepted_convenience_cookies: false,
-      });
-    });
-  });
-
-  it('resets every field when the identity changes', async () => {
-    const { setUser } = renderProfile(makeUser());
-    fireEvent.change(screen.getByRole('textbox', { name: 'First name' }), {
-      target: { value: 'Unsaved first' },
-    });
-    fireEvent.change(screen.getByRole('textbox', { name: 'Last name' }), {
-      target: { value: 'Unsaved last' },
-    });
-    fireEvent.click(screen.getByRole('checkbox', { name: 'Accept privacy statement' }));
-    fireEvent.click(screen.getByRole('checkbox', { name: 'Accept convenience cookies' }));
-
-    setUser(makeUser({
-      id: 18,
-      username: 'lee',
-      email: 'lee@example.com',
-      first_name: 'Lee',
-      last_name: 'Newperson',
-      accepted_privacy_statement: false,
-      accepted_convenience_cookies: true,
-    }));
-
-    await waitFor(() => {
-      expect(screen.getByRole('textbox', { name: 'First name' }).value).toBe('Lee');
-    });
-    expect(screen.getByRole('textbox', { name: 'Username' }).value).toBe('lee');
-    expect(screen.getByRole('textbox', { name: 'Email' }).value).toBe('lee@example.com');
-    expect(screen.getByRole('textbox', { name: 'Last name' }).value).toBe('Newperson');
-    expect(screen.getByRole('checkbox', { name: 'Accept privacy statement' }).checked).toBe(false);
-    expect(screen.getByRole('checkbox', { name: 'Accept convenience cookies' }).checked).toBe(true);
-  });
-
-  it('clears edits on logout before initializing the next login', async () => {
-    const { setUser } = renderProfile(makeUser());
-    fireEvent.change(screen.getByRole('textbox', { name: 'First name' }), {
-      target: { value: 'Unsaved first' },
-    });
-    fireEvent.click(screen.getByRole('checkbox', { name: 'Accept privacy statement' }));
-    fireEvent.click(screen.getByRole('checkbox', { name: 'Accept convenience cookies' }));
-
-    setUser(null);
-    expect(await screen.findByText('User not logged in.')).toBeTruthy();
-    setUser(makeUser({
-      first_name: 'Pat after login',
-      accepted_privacy_statement: false,
-      accepted_convenience_cookies: true,
-    }));
-
-    await waitFor(() => {
-      expect(screen.getByRole('textbox', { name: 'First name' }).value).toBe('Pat after login');
-    });
-    expect(screen.getByRole('checkbox', { name: 'Accept privacy statement' }).checked).toBe(false);
-    expect(screen.getByRole('checkbox', { name: 'Accept convenience cookies' }).checked).toBe(true);
-  });
-
-  it('hydrates fields when the current user arrives asynchronously', async () => {
-    const { setUser } = renderProfile(null);
-    setUser(makeUser({ first_name: 'Hydrated' }));
-
-    await waitFor(() => {
-      expect(screen.getByRole('textbox', { name: 'First name' }).value).toBe('Hydrated');
-      expect(screen.getByRole('textbox', { name: 'Last name' }).value).toBe('Example');
-    });
-    expect(screen.getByRole('checkbox', { name: 'Accept privacy statement' }).checked).toBe(false);
-    expect(screen.getByRole('checkbox', { name: 'Accept convenience cookies' }).checked).toBe(true);
-  });
-});

package/tests/QrSignupManager.registrationContext.test.jsx

@@ -1,106 +0,0 @@
-// @vitest-environment jsdom
-
-import React from 'react';
-import { cleanup, fireEvent, render, screen, waitFor } from '@testing-library/react';
-import { afterEach, describe, expect, it, vi } from 'vitest';
-
-const mockState = vi.hoisted(() => ({
-  createSignupQr: vi.fn().mockResolvedValue({
-    signup_url: 'https://example.test/signup?rt=token',
-    expires_at: '2026-09-01T12:00:00Z',
-  }),
-}));
-
-vi.mock('../src/auth/authApi', () => ({
-  createSignupQr: mockState.createSignupQr,
-}));
-
-vi.mock('qrcode.react', () => ({
-  QRCodeSVG: () => null,
-}));
-
-// A stable `t` reference matters here: it's an effect dependency in QrSignupManager, and
-// the real react-i18next memoizes it across renders. A fresh function per render (a common
-// shorthand mock elsewhere in this repo) would make that effect re-fire on every unrelated
-// re-render (e.g. the setBusy(true) inside it), which is a mock artifact this test file must
-// not reintroduce given it asserts exact createSignupQr call counts.
-const stableT = (_key, fallback) => fallback;
-// resolveErrorText (AUTH-10) reads i18n.exists/i18n.t off the object
-// useTranslation() returns — kept module-scoped/stable for the same reason
-// `stableT` is, above.
-const stableI18n = { exists: () => false, t: stableT };
-vi.mock('react-i18next', () => ({
-  useTranslation: () => ({ t: stableT, i18n: stableI18n }),
-}));
-
-import { QrSignupManager } from '../src/components/QrSignupManager';
-
-describe('QrSignupManager registration context', () => {
-  afterEach(() => {
-    cleanup();
-  });
-
-  it('passes the department context and practical redemption default to the API', async () => {
-    mockState.createSignupQr.mockClear();
-
-    render(
-      <QrSignupManager
-        enabled
-        expiryDays={365}
-        registrationContext={{ group_ref: '42' }}
-        defaultMaxRedemptions={5000}
-      />,
-    );
-
-    await waitFor(() => expect(mockState.createSignupQr).toHaveBeenCalledWith({
-      expires_minutes: 365 * 24 * 60,
-      max_redemptions: 5000,
-      registration_context: { group_ref: '42' },
-    }));
-  });
-
-  it('behaves exactly as before when the new props are omitted (existing AccountPage-style call)', async () => {
-    mockState.createSignupQr.mockClear();
-
-    render(<QrSignupManager enabled expiryDays={90} />);
-
-    await waitFor(() => expect(mockState.createSignupQr).toHaveBeenCalledWith({
-      expires_minutes: 90 * 24 * 60,
-      max_redemptions: 1,
-    }));
-    expect(mockState.createSignupQr).toHaveBeenCalledTimes(1);
-  });
-
-  it('fires exactly one create call when the context changes after a custom redemption count was set', async () => {
-    mockState.createSignupQr.mockClear();
-
-    const { rerender } = render(
-      <QrSignupManager
-        enabled
-        expiryDays={90}
-        registrationContext={{ group_ref: '1' }}
-        defaultMaxRedemptions={5000}
-      />,
-    );
-    await waitFor(() => expect(mockState.createSignupQr).toHaveBeenCalledTimes(1));
-
-    fireEvent.change(screen.getByLabelText('Max redemptions'), { target: { value: '250' } });
-
-    mockState.createSignupQr.mockClear();
-    rerender(
-      <QrSignupManager
-        enabled
-        expiryDays={90}
-        registrationContext={{ group_ref: '2' }}
-        defaultMaxRedemptions={5000}
-      />,
-    );
-
-    await waitFor(() => expect(mockState.createSignupQr).toHaveBeenCalledWith({
-      expires_minutes: 90 * 24 * 60,
-      max_redemptions: 5000,
-      registration_context: { group_ref: '2' },
-    }));
-    expect(mockState.createSignupQr).toHaveBeenCalledTimes(1);
-  });
-});

package/tests/QrSignupManagerInfoBox.test.jsx

@@ -1,49 +0,0 @@
-// @vitest-environment jsdom
-
-import React from 'react';
-import { ThemeProvider } from '@mui/material/styles';
-import { cleanup, render, screen } from '@testing-library/react';
-import { afterEach, describe, expect, it, vi } from 'vitest';
-
-vi.mock('../src/auth/authApi', () => ({
-  createSignupQr: vi.fn().mockResolvedValue({
-    signup_url: 'https://example.test/signup?rt=token',
-    expires_at: '2026-09-01T12:00:00Z',
-  }),
-}));
-
-vi.mock('qrcode.react', () => ({
-  QRCodeSVG: () => null,
-}));
-
-const stableT = (_key, fallback) => fallback;
-const stableI18n = { exists: () => false, t: stableT };
-vi.mock('react-i18next', () => ({
-  useTranslation: () => ({ t: stableT, i18n: stableI18n }),
-}));
-
-import { QrSignupManager } from '../src/components/QrSignupManager';
-import { createAppTheme } from '../src/theme';
-
-// jsdom serialises the resolved colour as rgb(), so compare against the theme's own value in that form.
-function toRgb(hex) {
-  const [r, g, b] = [1, 3, 5].map((i) => parseInt(hex.slice(i, i + 2), 16));
-  return `rgb(${r}, ${g}, ${b})`;
-}
-
-describe('QrSignupManager info box', () => {
-  afterEach(cleanup);
-
-  it.each(['light', 'dark'])('paints background.subtle of the %s theme', async (mode) => {
-    const theme = createAppTheme({ palette: { mode, primary: { main: '#468AB2' } } });
-    render(
-      <ThemeProvider theme={theme}>
-        <QrSignupManager enabled expiryDays={90} />
-      </ThemeProvider>,
-    );
-
-    const title = await screen.findByText('Signup Access');
-    const box = title.parentElement;
-    expect(getComputedStyle(box).backgroundColor).toBe(toRgb(theme.palette.background.subtle));
-  });
-});