Some checks are pending
Bidi Control Character Guard / bidi-control-guard (push) Waiting to run
Circular Dependency Check / Check for new circular dependencies (push) Waiting to run
Citus Migration Smoke / Combined migrations on single-node Citus (push) Waiting to run
E2E Fresh Install Tests / fresh-install-e2e (push) Waiting to run
ext-v2 guardrails / Run ext-v2 guard and ESLint (push) Waiting to run
Integration Tests / Check for relevant changes (push) Waiting to run
Integration Tests / ${{ (github.event_name == 'schedule' || github.event.inputs.suite == 'full') && 'Full integration suite' || 'Tier-1 integration subset' }} (push) Blocked by required conditions
Mobile checks / Mobile lint + typecheck (push) Waiting to run
Mobile checks / Mobile unit tests (push) Waiting to run
Mobile checks / Mobile dependency audit (report) (push) Waiting to run
Mobile checks / Mobile reproducibility checks (push) Waiting to run
Secrets guard (env backups) / Ensure no tracked env backup files (push) Waiting to run
Temporal Readiness / fast-readiness (push) Waiting to run
Temporal Readiness / docker-parity (push) Waiting to run
TypeScript Type Check / Nx affected typecheck (push) Waiting to run
Unit Tests / Skipped-test budget (push) Waiting to run
Unit Tests / Nx affected unit tests (push) Waiting to run
Unit Tests / Server unit coverage (informational) (push) Waiting to run
Validate Tenant Management Schema / Check for relevant changes (push) Waiting to run
Validate Tenant Management Schema / Validate Tenant Management Schema (push) Blocked by required conditions
EE Workflows Build Guard / ee-workflows-build-guard (push) Waiting to run
Excluded: .git, node_modules, secrets/, compose.env, assemblyscript tgz Source: /opt/alga-psa on psa.joliet.tech
378 lines
13 KiB
TypeScript
378 lines
13 KiB
TypeScript
// @vitest-environment jsdom
|
|
import * as React from 'react';
|
|
import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
|
|
import { flushSync } from 'react-dom';
|
|
import { createRoot, Root } from 'react-dom/client';
|
|
|
|
// useFormatters requires an I18nProvider; these tests render the component
|
|
// standalone, so substitute locale-stable formatters.
|
|
vi.mock('@alga-psa/ui/lib/i18n/client', async (importOriginal) => {
|
|
const actual = await importOriginal<Record<string, unknown>>();
|
|
return {
|
|
...actual,
|
|
useFormatters: () => ({
|
|
formatDate: (date: Date | string, options?: Intl.DateTimeFormatOptions) =>
|
|
new Intl.DateTimeFormat('en-US', options).format(typeof date === 'string' ? new Date(date) : date),
|
|
formatNumber: (value: number, options?: Intl.NumberFormatOptions) =>
|
|
new Intl.NumberFormat('en-US', options).format(value),
|
|
formatCurrency: (value: number, currency: string, options?: Intl.NumberFormatOptions) =>
|
|
new Intl.NumberFormat('en-US', { style: 'currency', currency, ...options }).format(value),
|
|
formatRelativeTime: (date: Date | string) => String(date),
|
|
}),
|
|
};
|
|
});
|
|
|
|
vi.mock('@alga-psa/ui/components/Badge', () => ({
|
|
Badge: ({ children }: { children: React.ReactNode }) => React.createElement('span', null, children),
|
|
}), { virtual: true });
|
|
|
|
vi.mock('@alga-psa/ui/components/Button', () => ({
|
|
Button: ({ children, title, onClick, disabled }: { children: React.ReactNode; title?: string; onClick?: () => void; disabled?: boolean }) =>
|
|
React.createElement('button', { type: 'button', title, onClick, disabled }, children),
|
|
}), { virtual: true });
|
|
|
|
vi.mock('@alga-psa/ui/components/Card', () => ({
|
|
Card: ({ children }: { children: React.ReactNode }) => React.createElement('div', null, children),
|
|
CardHeader: ({ children }: { children: React.ReactNode }) => React.createElement('div', null, children),
|
|
CardTitle: ({ children }: { children: React.ReactNode }) => React.createElement('h3', null, children),
|
|
CardContent: ({ children }: { children: React.ReactNode }) => React.createElement('div', null, children),
|
|
}), { virtual: true });
|
|
|
|
vi.mock('@alga-psa/ui/components/TextArea', () => ({
|
|
TextArea: ({
|
|
value,
|
|
onChange,
|
|
placeholder,
|
|
}: {
|
|
value?: string;
|
|
onChange?: (event: { target: { value: string } }) => void;
|
|
placeholder?: string;
|
|
}) => React.createElement('textarea', {
|
|
value,
|
|
placeholder,
|
|
onChange: (event: React.ChangeEvent<HTMLTextAreaElement>) => onChange?.({ target: { value: event.target.value } }),
|
|
}),
|
|
}), { virtual: true });
|
|
|
|
const addCommentToTimeSheet = vi.fn();
|
|
const fetchTimeSheetComments = vi.fn();
|
|
const requestChangesForTimeSheet = vi.fn();
|
|
vi.mock('../src/actions/timeSheetActions', () => ({
|
|
addCommentToTimeSheet,
|
|
fetchTimeSheetComments,
|
|
requestChangesForTimeSheet,
|
|
}));
|
|
|
|
const fetchWorkItemsForTimeSheet = vi.fn();
|
|
const updateTimeEntryApprovalStatus = vi.fn();
|
|
vi.mock('../src/actions/timeEntryActions', () => ({
|
|
fetchWorkItemsForTimeSheet,
|
|
updateTimeEntryApprovalStatus,
|
|
}));
|
|
|
|
const { TimeSheetApproval } = await import('../src/components/time-management/approvals/TimeSheetApproval');
|
|
|
|
describe('TimeSheetApproval', () => {
|
|
let container: HTMLDivElement;
|
|
let root: Root;
|
|
|
|
beforeEach(() => {
|
|
container = document.createElement('div');
|
|
document.body.appendChild(container);
|
|
root = createRoot(container);
|
|
|
|
addCommentToTimeSheet.mockReset();
|
|
fetchTimeSheetComments.mockReset();
|
|
fetchWorkItemsForTimeSheet.mockReset();
|
|
updateTimeEntryApprovalStatus.mockReset();
|
|
requestChangesForTimeSheet.mockReset();
|
|
|
|
fetchTimeSheetComments.mockResolvedValue([]);
|
|
fetchWorkItemsForTimeSheet.mockResolvedValue([
|
|
{
|
|
work_item_id: 'ticket-1',
|
|
type: 'ticket',
|
|
name: 'Ticket 1',
|
|
description: 'Ticket description',
|
|
},
|
|
]);
|
|
updateTimeEntryApprovalStatus.mockResolvedValue(undefined);
|
|
requestChangesForTimeSheet.mockResolvedValue(undefined);
|
|
});
|
|
|
|
afterEach(() => {
|
|
root.unmount();
|
|
container.remove();
|
|
});
|
|
|
|
async function flushUi() {
|
|
await new Promise(resolve => setTimeout(resolve, 0));
|
|
}
|
|
|
|
function renderApprovalDrawer(
|
|
timeEntriesOverride?: Array<Record<string, unknown>>,
|
|
) {
|
|
flushSync(() => {
|
|
root.render(React.createElement(TimeSheetApproval, {
|
|
timeSheet: {
|
|
id: 'sheet-1',
|
|
period_id: 'period-1',
|
|
user_id: 'user-1',
|
|
approval_status: 'SUBMITTED',
|
|
employee_name: 'Ada Lovelace',
|
|
employee_email: 'ada@example.com',
|
|
comments: [],
|
|
tenant: 'tenant-1',
|
|
time_period: {
|
|
period_id: 'period-1',
|
|
start_date: '2026-03-01',
|
|
end_date: '2026-03-07',
|
|
tenant: 'tenant-1',
|
|
},
|
|
},
|
|
timeEntries: (timeEntriesOverride as any) ?? [
|
|
{
|
|
entry_id: 'entry-1',
|
|
work_item_id: 'ticket-1',
|
|
work_item_type: 'ticket',
|
|
start_time: '2026-03-02T09:00:00.000Z',
|
|
end_time: '2026-03-02T10:00:00.000Z',
|
|
created_at: '2026-03-02T10:00:00.000Z',
|
|
updated_at: '2026-03-02T10:00:00.000Z',
|
|
work_date: '2026-03-02',
|
|
billable_duration: 60,
|
|
notes: 'Needs follow-up',
|
|
user_id: 'user-1',
|
|
time_sheet_id: 'sheet-1',
|
|
approval_status: 'SUBMITTED',
|
|
tenant: 'tenant-1',
|
|
},
|
|
],
|
|
currentUser: {
|
|
user_id: 'manager-1',
|
|
first_name: 'Grace',
|
|
last_name: 'Hopper',
|
|
email: 'grace@example.com',
|
|
username: 'ghopper',
|
|
is_inactive: false,
|
|
tenant: 'tenant-1',
|
|
user_type: 'internal',
|
|
},
|
|
onApprove: vi.fn(),
|
|
onRequestChanges: vi.fn(),
|
|
}));
|
|
});
|
|
}
|
|
|
|
it('T001: renders a dedicated entry-level change suggestion input', async () => {
|
|
renderApprovalDrawer();
|
|
|
|
await flushUi();
|
|
|
|
const toggleButton = container.querySelector('button[title="Show Details"]');
|
|
if (!toggleButton) {
|
|
throw new Error('Show Details button not found');
|
|
}
|
|
toggleButton.dispatchEvent(new MouseEvent('click', { bubbles: true }));
|
|
await flushUi();
|
|
|
|
expect(container.textContent).toContain('Entry Change Suggestion');
|
|
expect(container.querySelector('textarea')?.getAttribute('placeholder')).toContain('Tell the employee exactly what to fix');
|
|
});
|
|
|
|
it('updates entry approval status without requiring service_id in the approval drawer flow', async () => {
|
|
renderApprovalDrawer();
|
|
|
|
await flushUi();
|
|
expect(fetchWorkItemsForTimeSheet).toHaveBeenCalledWith('sheet-1');
|
|
|
|
const toggleButton = container.querySelector('button[title="Show Details"]');
|
|
if (!toggleButton) {
|
|
throw new Error('Show Details button not found');
|
|
}
|
|
toggleButton.dispatchEvent(new MouseEvent('click', { bubbles: true }));
|
|
await flushUi();
|
|
|
|
const requestChangesButton = Array.from(container.querySelectorAll('button')).find(
|
|
node => node.textContent?.includes('Request Changes'),
|
|
);
|
|
if (!requestChangesButton) {
|
|
throw new Error('Request Changes button not found');
|
|
}
|
|
requestChangesButton.dispatchEvent(new MouseEvent('click', { bubbles: true }));
|
|
await flushUi();
|
|
|
|
expect(updateTimeEntryApprovalStatus).toHaveBeenCalledWith({
|
|
entryId: 'entry-1',
|
|
approvalStatus: 'CHANGES_REQUESTED',
|
|
changeRequestComment: undefined,
|
|
});
|
|
expect(requestChangesForTimeSheet).not.toHaveBeenCalled();
|
|
});
|
|
|
|
it('T002/T033: submits the entry id together with the optional suggestion and still supports an empty suggestion', async () => {
|
|
renderApprovalDrawer();
|
|
|
|
await flushUi();
|
|
|
|
const toggleButton = container.querySelector('button[title="Show Details"]');
|
|
if (!toggleButton) {
|
|
throw new Error('Show Details button not found');
|
|
}
|
|
toggleButton.dispatchEvent(new MouseEvent('click', { bubbles: true }));
|
|
await flushUi();
|
|
|
|
const suggestionInput = container.querySelector('textarea');
|
|
if (!suggestionInput) {
|
|
throw new Error('Suggestion input not found');
|
|
}
|
|
|
|
const valueSetter = Object.getOwnPropertyDescriptor(
|
|
HTMLTextAreaElement.prototype,
|
|
'value',
|
|
)?.set;
|
|
|
|
valueSetter?.call(suggestionInput, 'Please break out travel time separately.');
|
|
suggestionInput.dispatchEvent(new Event('input', { bubbles: true }));
|
|
suggestionInput.dispatchEvent(new Event('change', { bubbles: true }));
|
|
await flushUi();
|
|
|
|
let requestChangesButton = Array.from(container.querySelectorAll('button')).find(
|
|
node => node.textContent?.includes('Request Changes'),
|
|
);
|
|
if (!requestChangesButton) {
|
|
throw new Error('Request Changes button not found');
|
|
}
|
|
requestChangesButton.dispatchEvent(new MouseEvent('click', { bubbles: true }));
|
|
await flushUi();
|
|
|
|
expect(updateTimeEntryApprovalStatus).toHaveBeenLastCalledWith({
|
|
entryId: 'entry-1',
|
|
approvalStatus: 'CHANGES_REQUESTED',
|
|
changeRequestComment: 'Please break out travel time separately.',
|
|
});
|
|
|
|
updateTimeEntryApprovalStatus.mockClear();
|
|
|
|
root.unmount();
|
|
container.remove();
|
|
container = document.createElement('div');
|
|
document.body.appendChild(container);
|
|
root = createRoot(container);
|
|
|
|
renderApprovalDrawer();
|
|
await flushUi();
|
|
|
|
const freshToggleButton = container.querySelector('button[title="Show Details"]');
|
|
if (!freshToggleButton) {
|
|
throw new Error('Show Details button not found');
|
|
}
|
|
freshToggleButton.dispatchEvent(new MouseEvent('click', { bubbles: true }));
|
|
await flushUi();
|
|
|
|
requestChangesButton = Array.from(container.querySelectorAll('button')).find(
|
|
node => node.textContent?.includes('Request Changes'),
|
|
);
|
|
if (!requestChangesButton) {
|
|
throw new Error('Request Changes button not found');
|
|
}
|
|
requestChangesButton.dispatchEvent(new MouseEvent('click', { bubbles: true }));
|
|
await flushUi();
|
|
|
|
expect(updateTimeEntryApprovalStatus).toHaveBeenLastCalledWith({
|
|
entryId: 'entry-1',
|
|
approvalStatus: 'CHANGES_REQUESTED',
|
|
changeRequestComment: undefined,
|
|
});
|
|
});
|
|
|
|
it('shows existing per-entry feedback in the manager approval drawer', async () => {
|
|
renderApprovalDrawer([
|
|
{
|
|
entry_id: 'entry-1',
|
|
work_item_id: 'ticket-1',
|
|
work_item_type: 'ticket',
|
|
start_time: '2026-03-02T09:00:00.000Z',
|
|
end_time: '2026-03-02T10:00:00.000Z',
|
|
created_at: '2026-03-02T10:00:00.000Z',
|
|
updated_at: '2026-03-02T10:00:00.000Z',
|
|
work_date: '2026-03-02',
|
|
billable_duration: 60,
|
|
notes: 'Needs follow-up',
|
|
user_id: 'user-1',
|
|
time_sheet_id: 'sheet-1',
|
|
approval_status: 'CHANGES_REQUESTED',
|
|
tenant: 'tenant-1',
|
|
change_request_state: 'unresolved',
|
|
change_requests: [
|
|
{
|
|
change_request_id: 'cr-1',
|
|
time_entry_id: 'entry-1',
|
|
time_sheet_id: 'sheet-1',
|
|
comment: 'Please split travel time from labor.',
|
|
created_at: '2026-03-02T11:00:00.000Z',
|
|
created_by: 'manager-1',
|
|
created_by_name: 'Grace Hopper',
|
|
tenant: 'tenant-1',
|
|
},
|
|
],
|
|
},
|
|
]);
|
|
|
|
await flushUi();
|
|
|
|
expect(container.querySelector('[data-feedback-state="unresolved"]')).not.toBeNull();
|
|
|
|
const toggleButton = container.querySelector('button[title="Show Details"]');
|
|
if (!toggleButton) {
|
|
throw new Error('Show Details button not found');
|
|
}
|
|
toggleButton.dispatchEvent(new MouseEvent('click', { bubbles: true }));
|
|
await flushUi();
|
|
|
|
expect(container.textContent).toContain('Approver feedback');
|
|
expect(container.textContent).toContain('Please split travel time from labor.');
|
|
expect(container.textContent).toContain('Change requested');
|
|
});
|
|
|
|
it('shows newly requested feedback immediately after the manager submits it', async () => {
|
|
renderApprovalDrawer();
|
|
|
|
await flushUi();
|
|
|
|
const toggleButton = container.querySelector('button[title="Show Details"]');
|
|
if (!toggleButton) {
|
|
throw new Error('Show Details button not found');
|
|
}
|
|
toggleButton.dispatchEvent(new MouseEvent('click', { bubbles: true }));
|
|
await flushUi();
|
|
|
|
const suggestionInput = container.querySelector('textarea');
|
|
if (!suggestionInput) {
|
|
throw new Error('Suggestion input not found');
|
|
}
|
|
|
|
const valueSetter = Object.getOwnPropertyDescriptor(
|
|
HTMLTextAreaElement.prototype,
|
|
'value',
|
|
)?.set;
|
|
|
|
valueSetter?.call(suggestionInput, 'Please correct the service classification.');
|
|
suggestionInput.dispatchEvent(new Event('input', { bubbles: true }));
|
|
suggestionInput.dispatchEvent(new Event('change', { bubbles: true }));
|
|
await flushUi();
|
|
|
|
const requestChangesButton = Array.from(container.querySelectorAll('button')).find(
|
|
node => node.textContent?.includes('Request Changes'),
|
|
);
|
|
if (!requestChangesButton) {
|
|
throw new Error('Request Changes button not found');
|
|
}
|
|
requestChangesButton.dispatchEvent(new MouseEvent('click', { bubbles: true }));
|
|
await flushUi();
|
|
|
|
expect(container.textContent).toContain('Please correct the service classification.');
|
|
expect(container.querySelector('[data-feedback-state="unresolved"]')).not.toBeNull();
|
|
});
|
|
});
|