@happyvertical/smrt-projects 0.52.0 → 0.53.2

package/dist/svelte/components/__tests__/RejectDialog.test.js

@@ -1,45 +0,0 @@
-// @vitest-environment jsdom
-/**
- * Component coverage for RejectDialog via the shared S11 harness (#1416).
- */
-import { expectNoA11yViolations, render, screen, userEvent, } from '@happyvertical/smrt-vitest/svelte';
-import { describe, expect, it, vi } from 'vitest';
-import RejectDialog from '../RejectDialog.svelte';
-const baseProps = (over = {}) => ({
-    open: true,
-    onconfirm: vi.fn(),
-    oncancel: vi.fn(),
-    ...over,
-});
-describe('RejectDialog', () => {
-    it('renders the dialog with a reason field when open', () => {
-        render(RejectDialog, { props: baseProps() });
-        expect(screen.getByRole('dialog')).toBeInTheDocument();
-        expect(screen.getByRole('textbox')).toBeInTheDocument();
-        expect(screen.getByRole('button', { name: 'Reject' })).toBeInTheDocument();
-    });
-    it('renders nothing when closed', () => {
-        render(RejectDialog, { props: baseProps({ open: false }) });
-        expect(screen.queryByRole('dialog')).toBeNull();
-    });
-    it('requires a reason before the confirm button is enabled', async () => {
-        const onconfirm = vi.fn();
-        render(RejectDialog, { props: baseProps({ onconfirm }) });
-        const confirm = screen.getByRole('button', { name: 'Reject' });
-        expect(confirm).toBeDisabled();
-        await userEvent.type(screen.getByRole('textbox'), 'Insufficient detail');
-        expect(confirm).toBeEnabled();
-        await userEvent.click(confirm);
-        expect(onconfirm).toHaveBeenCalledWith('Insufficient detail');
-    });
-    it('cancels via the cancel button', async () => {
-        const oncancel = vi.fn();
-        render(RejectDialog, { props: baseProps({ oncancel }) });
-        await userEvent.click(screen.getByRole('button', { name: 'Cancel' }));
-        expect(oncancel).toHaveBeenCalledTimes(1);
-    });
-    it('is axe-clean', async () => {
-        const { container } = render(RejectDialog, { props: baseProps() });
-        await expectNoA11yViolations(container);
-    });
-});

package/dist/svelte/components/__tests__/TimeEntryCard.test.js

@@ -1,45 +0,0 @@
-// @vitest-environment jsdom
-/**
- * Component coverage for TimeEntryCard via the shared S11 harness (#1416).
- */
-import { expectNoA11yViolations, render, screen, userEvent, } from '@happyvertical/smrt-vitest/svelte';
-import { describe, expect, it, vi } from 'vitest';
-import TimeEntryCard from '../TimeEntryCard.svelte';
-const entry = {
-    id: 'e1',
-    date: '2026-06-01',
-    description: 'Implement feature',
-    status: 'draft',
-    hours: 3,
-    hourlyRate: 10000, // $100.00/h in cents (#2401)
-    amount: 30000, // 3 h x $100.00
-    workerName: 'Ada Lovelace',
-};
-describe('TimeEntryCard', () => {
-    it('renders the entry description, status, and worker', () => {
-        render(TimeEntryCard, { props: { entry } });
-        expect(screen.getByText('Implement feature')).toBeInTheDocument();
-        expect(screen.getByText('DRAFT')).toBeInTheDocument();
-        expect(screen.getByText('Ada Lovelace')).toBeInTheDocument();
-    });
-    it('forwards checkbox selection through onselect', async () => {
-        const onselect = vi.fn();
-        render(TimeEntryCard, {
-            props: { entry, selectable: true, onselect },
-        });
-        await userEvent.click(screen.getByLabelText('Select time entry for Implement feature'));
-        expect(onselect).toHaveBeenCalledWith('e1', true);
-    });
-    it('invokes onclick when the card is activated', async () => {
-        const onclick = vi.fn();
-        render(TimeEntryCard, { props: { entry, onclick } });
-        await userEvent.click(screen.getByText('Implement feature'));
-        expect(onclick).toHaveBeenCalledTimes(1);
-    });
-    it('is axe-clean', async () => {
-        const { container } = render(TimeEntryCard, {
-            props: { entry, selectable: true, onselect: vi.fn() },
-        });
-        await expectNoA11yViolations(container);
-    });
-});

package/dist/svelte/components/__tests__/TimeEntryList.test.js

@@ -1,50 +0,0 @@
-// @vitest-environment jsdom
-/**
- * Component coverage for TimeEntryList via the shared S11 harness (#1416).
- */
-import { expectNoA11yViolations, render, screen, userEvent, } from '@happyvertical/smrt-vitest/svelte';
-import { describe, expect, it, vi } from 'vitest';
-import TimeEntryList from '../TimeEntryList.svelte';
-const entries = [
-    {
-        id: 'e1',
-        date: '2026-06-01',
-        description: 'First task',
-        status: 'draft',
-        hours: 2,
-    },
-    {
-        id: 'e2',
-        date: '2026-06-02',
-        description: 'Second task',
-        status: 'draft',
-        hours: 4,
-    },
-];
-describe('TimeEntryList', () => {
-    it('renders a card per entry', () => {
-        render(TimeEntryList, { props: { entries } });
-        expect(screen.getByText('First task')).toBeInTheDocument();
-        expect(screen.getByText('Second task')).toBeInTheDocument();
-    });
-    it('shows the empty message when there are no entries', () => {
-        render(TimeEntryList, {
-            props: { entries: [], emptyMessage: 'No time entries' },
-        });
-        expect(screen.getByText('No time entries')).toBeInTheDocument();
-    });
-    it('selects all entries via the header checkbox', async () => {
-        const onselectionchange = vi.fn();
-        render(TimeEntryList, {
-            props: { entries, selectable: true, onselectionchange },
-        });
-        await userEvent.click(screen.getByLabelText('Select all time entries'));
-        expect(onselectionchange).toHaveBeenCalledWith(['e1', 'e2']);
-    });
-    it('is axe-clean', async () => {
-        const { container } = render(TimeEntryList, {
-            props: { entries, selectable: true, onselectionchange: vi.fn() },
-        });
-        await expectNoA11yViolations(container);
-    });
-});

package/dist/svelte/components/__tests__/TimeSummary.test.js

@@ -1,39 +0,0 @@
-// @vitest-environment jsdom
-/**
- * Component coverage for TimeSummary via the shared S11 harness (#1416).
- */
-import { expectNoA11yViolations, render, screen, } from '@happyvertical/smrt-vitest/svelte';
-import { describe, expect, it } from 'vitest';
-import TimeSummary from '../TimeSummary.svelte';
-describe('TimeSummary', () => {
-    it('renders the total hours and value cards', () => {
-        render(TimeSummary, {
-            props: { totalHours: 12, totalAmount: 600, entryCount: 3 },
-        });
-        expect(screen.getByText('Total Hours')).toBeInTheDocument();
-        expect(screen.getByText('Total Value')).toBeInTheDocument();
-        expect(screen.getByText('3 entries')).toBeInTheDocument();
-    });
-    it('shows pending and approved cards when enabled and non-zero', () => {
-        render(TimeSummary, {
-            props: {
-                totalHours: 12,
-                totalAmount: 600,
-                pendingHours: 4,
-                pendingAmount: 200,
-                approvedHours: 8,
-                approvedAmount: 400,
-                showPending: true,
-                showApproved: true,
-            },
-        });
-        expect(screen.getByText('Pending Approval')).toBeInTheDocument();
-        expect(screen.getByText('Approved')).toBeInTheDocument();
-    });
-    it('is axe-clean', async () => {
-        const { container } = render(TimeSummary, {
-            props: { totalHours: 12, totalAmount: 600 },
-        });
-        await expectNoA11yViolations(container);
-    });
-});

package/dist/svelte/components/utils.d.ts

@@ -1,51 +0,0 @@
-/**
- * Shared utilities for time tracking components
- */
-export type Currency = 'CAD' | 'USD';
-export type TimeEntryStatus = 'draft' | 'submitted' | 'approved' | 'rejected';
-/** Alias for TimeEntryStatus for approval workflows */
-export type ApprovalStatus = TimeEntryStatus;
-/**
- * Time entry data structure
- */
-export interface TimeEntry {
-    id: string;
-    date: Date | string;
-    hours: number;
-    description: string;
-    status: TimeEntryStatus;
-    /** Integer minor units of the display currency — $19.99 is 1999 (#2401). */
-    amount?: number;
-    workerName?: string;
-    mileage?: number;
-    /** Minor units per hour, so `hours * hourlyRate` is minor units (#2401). */
-    hourlyRate?: number;
-}
-/**
- * Status color mapping using M3 design tokens
- */
-export declare const statusColors: Record<TimeEntryStatus, string>;
-/**
- * Format a date for display (e.g., "Jan 5")
- */
-export declare function formatDate(date: Date | string): string;
-/**
- * Format a minor-units amount for display.
- *
- * Money in this package is integer minor units (#2401) — `$19.99` is `1999` —
- * and `Intl.NumberFormat` expects major units, so the scale is undone here.
- * Without it a $19.99 charge renders as $1,999.00.
- *
- * The currency's own minor-unit exponent is used rather than a hard-coded 100,
- * so zero-decimal currencies (JPY, KRW) are not divided by anything.
- */
-export declare function formatCurrency(amount: number, currency?: Currency): string;
-/**
- * Format hours for display (e.g., "8.5h")
- */
-export declare function formatHours(hours: number): string;
-/**
- * Format hours in HH:MM format (e.g., "8:30")
- */
-export declare function formatHoursHHMM(hours: number): string;
-//# sourceMappingURL=utils.d.ts.map

package/dist/svelte/components/utils.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"utils.d.ts","sourceRoot":"","sources":["../../../src/svelte/components/utils.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH,MAAM,MAAM,QAAQ,GAAG,KAAK,GAAG,KAAK,CAAC;AAErC,MAAM,MAAM,eAAe,GAAG,OAAO,GAAG,WAAW,GAAG,UAAU,GAAG,UAAU,CAAC;AAE9E,uDAAuD;AACvD,MAAM,MAAM,cAAc,GAAG,eAAe,CAAC;AAE7C;;GAEG;AACH,MAAM,WAAW,SAAS;IACxB,EAAE,EAAE,MAAM,CAAC;IACX,IAAI,EAAE,IAAI,GAAG,MAAM,CAAC;IACpB,KAAK,EAAE,MAAM,CAAC;IACd,WAAW,EAAE,MAAM,CAAC;IACpB,MAAM,EAAE,eAAe,CAAC;IACxB,4EAA4E;IAC5E,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,4EAA4E;IAC5E,UAAU,CAAC,EAAE,MAAM,CAAC;CACrB;AAED;;GAEG;AACH,eAAO,MAAM,YAAY,EAAE,MAAM,CAAC,eAAe,EAAE,MAAM,CAKxD,CAAC;AAEF;;GAEG;AACH,wBAAgB,UAAU,CAAC,IAAI,EAAE,IAAI,GAAG,MAAM,GAAG,MAAM,CAGtD;AAED;;;;;;;;;GASG;AACH,wBAAgB,cAAc,CAC5B,MAAM,EAAE,MAAM,EACd,QAAQ,GAAE,QAAgB,GACzB,MAAM,CAcR;AAED;;GAEG;AACH,wBAAgB,WAAW,CAAC,KAAK,EAAE,MAAM,GAAG,MAAM,CAEjD;AAED;;GAEG;AACH,wBAAgB,eAAe,CAAC,KAAK,EAAE,MAAM,GAAG,MAAM,CAIrD"}

package/dist/svelte/components/utils.js

@@ -1,58 +0,0 @@
-/**
- * Shared utilities for time tracking components
- */
-/**
- * Status color mapping using M3 design tokens
- */
-export const statusColors = {
-    draft: 'var(--md-sys-color-outline)',
-    submitted: 'var(--md-sys-color-tertiary)',
-    approved: 'var(--md-sys-color-primary)',
-    rejected: 'var(--md-sys-color-error)',
-};
-/**
- * Format a date for display (e.g., "Jan 5")
- */
-export function formatDate(date) {
-    const d = typeof date === 'string' ? new Date(date) : date;
-    return d.toLocaleDateString('en-CA', { month: 'short', day: 'numeric' });
-}
-/**
- * Format a minor-units amount for display.
- *
- * Money in this package is integer minor units (#2401) — `$19.99` is `1999` —
- * and `Intl.NumberFormat` expects major units, so the scale is undone here.
- * Without it a $19.99 charge renders as $1,999.00.
- *
- * The currency's own minor-unit exponent is used rather than a hard-coded 100,
- * so zero-decimal currencies (JPY, KRW) are not divided by anything.
- */
-export function formatCurrency(amount, currency = 'CAD') {
-    // No fraction-digit override: the currency's own exponent is both the scale
-    // to undo and the precision to print. Forcing `minimumFractionDigits: 2`
-    // (as this did before minor units) would pin `maximumFractionDigits` to 2 as
-    // well, so a zero-decimal currency like JPY would report a scale of 2 — and
-    // then render `¥1,999.00` for a currency that has no fractional part at all.
-    // For CAD/USD the resolved default is already 2, so their output is
-    // unchanged.
-    const format = new Intl.NumberFormat('en-CA', {
-        style: 'currency',
-        currency,
-    });
-    const digits = format.resolvedOptions().maximumFractionDigits ?? 2;
-    return format.format(amount / 10 ** digits);
-}
-/**
- * Format hours for display (e.g., "8.5h")
- */
-export function formatHours(hours) {
-    return `${hours.toFixed(1)}h`;
-}
-/**
- * Format hours in HH:MM format (e.g., "8:30")
- */
-export function formatHoursHHMM(hours) {
-    const h = Math.floor(hours);
-    const m = Math.round((hours - h) * 60);
-    return `${h}:${m.toString().padStart(2, '0')}`;
-}

package/dist/svelte/utils.test.js

@@ -1,44 +0,0 @@
-/**
- * Currency formatting for integer minor units (#2401).
- *
- * `formatCurrency` receives what the database stores — integer minor units —
- * and `Intl.NumberFormat` formats major units, so the scale has to be undone
- * first. Which scale depends on the currency, which is the trap this file
- * guards: a hard-coded `/100`, or a `minimumFractionDigits: 2` override that
- * pins the resolved exponent to 2, both mis-render every currency that does not
- * have exactly two decimal places.
- */
-import { describe, expect, it } from 'vitest';
-import { formatCurrency as formatCurrencyFromComponents } from './components/utils.js';
-import { formatCurrency } from './utils.js';
-describe('formatCurrency (#2401)', () => {
-    it('renders a two-decimal currency by undoing the cent scale', () => {
-        // 19999 minor units is $199.99, not $19,999.00.
-        expect(formatCurrency(19999, 'CAD')).toContain('199.99');
-        expect(formatCurrency(19999, 'CAD')).not.toContain('19,999');
-    });
-    it('renders zero as the currency zero rather than nothing', () => {
-        expect(formatCurrency(0, 'CAD')).toContain('0.00');
-    });
-    it('does not rescale or add decimals to a zero-decimal currency', () => {
-        // JPY has no minor unit, so 1999 is ¥1,999 — dividing by 100 here would
-        // report ¥19.99, a hundredfold understatement, and printing two decimals
-        // would invent a fractional part the currency does not have.
-        const formatted = formatCurrency(1999, 'JPY');
-        expect(formatted).toContain('1,999');
-        expect(formatted).not.toContain('1,999.00');
-    });
-    it('uses a three-decimal currency exponent for a three-decimal currency', () => {
-        // BHD is 1000 fils to the dinar, so 1999 fils is BHD 1.999.
-        expect(formatCurrency(1999, 'BHD')).toContain('1.999');
-    });
-    it('is duplicated verbatim under components/, so both copies must agree', () => {
-        for (const [amount, currency] of [
-            [19999, 'CAD'],
-            [1999, 'JPY'],
-            [0, 'USD'],
-        ]) {
-            expect(formatCurrencyFromComponents(amount, currency)).toBe(formatCurrency(amount, currency));
-        }
-    });
-});