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
117 lines
3.4 KiB
TypeScript
117 lines
3.4 KiB
TypeScript
import { beforeEach, describe, expect, it, vi } from 'vitest';
|
|
|
|
const state = {
|
|
invoice: {
|
|
invoice_id: 'invoice-1',
|
|
client_id: 'client-1',
|
|
credit_applied: 0,
|
|
status: 'draft',
|
|
} as Record<string, any> | null,
|
|
hasCanonicalRecurringDetails: true,
|
|
queriedTables: [] as string[],
|
|
};
|
|
|
|
function createQueryBuilder(table: string) {
|
|
const builder: any = {};
|
|
builder.join = vi.fn(() => builder);
|
|
builder.where = vi.fn(() => builder);
|
|
builder.andWhere = vi.fn(() => builder);
|
|
builder.whereNotNull = vi.fn(() => builder);
|
|
builder.first = vi.fn(async () => {
|
|
if (table === 'invoices') {
|
|
return state.invoice;
|
|
}
|
|
|
|
if (table === 'invoice_charge_details as iid') {
|
|
return state.hasCanonicalRecurringDetails ? { item_detail_id: 'detail-1' } : undefined;
|
|
}
|
|
|
|
return undefined;
|
|
});
|
|
return builder;
|
|
}
|
|
|
|
function createMockTrx() {
|
|
return ((table: string) => {
|
|
state.queriedTables.push(table);
|
|
return createQueryBuilder(table);
|
|
}) as any;
|
|
}
|
|
|
|
vi.mock('@alga-psa/auth', () => ({
|
|
withAuth: (fn: any) => (...args: any[]) =>
|
|
fn({ user_id: 'user-1' }, { tenant: 'tenant-1' }, ...args),
|
|
getSession: vi.fn(async () => ({ user: { id: 'user-1' } })),
|
|
}));
|
|
|
|
vi.mock('@alga-psa/auth/rbac', () => ({
|
|
hasPermission: vi.fn().mockResolvedValue(true),
|
|
}));
|
|
|
|
vi.mock('@alga-psa/db', async () => {
|
|
const actual = await vi.importActual<any>('@alga-psa/db');
|
|
return {
|
|
...actual,
|
|
// knex must be callable: hardDeleteInvoice queries
|
|
// tenant_external_entity_mappings directly before opening the transaction.
|
|
createTenantKnex: vi.fn(async () => ({ knex: createMockTrx() })),
|
|
withTransaction: vi.fn(async (_knex: any, callback: any) => callback(createMockTrx())),
|
|
};
|
|
});
|
|
|
|
vi.mock('../src/models/clientContractLine', () => ({
|
|
default: {
|
|
updateClientCredit: vi.fn(async () => undefined),
|
|
},
|
|
}));
|
|
|
|
vi.mock('../src/actions/creditActions', () => ({
|
|
applyCreditToInvoice: vi.fn(),
|
|
}));
|
|
|
|
vi.mock('../src/lib/billing/billingEngine', () => ({
|
|
BillingEngine: class BillingEngine {},
|
|
}));
|
|
|
|
vi.mock('../src/services/invoiceService', () => ({
|
|
persistInvoiceCharges: vi.fn(),
|
|
persistManualInvoiceCharges: vi.fn(),
|
|
}));
|
|
|
|
vi.mock('@alga-psa/billing/models/invoice', () => ({
|
|
default: {},
|
|
}));
|
|
|
|
vi.mock('@alga-psa/event-bus/publishers', () => ({
|
|
publishWorkflowEvent: vi.fn(async () => undefined),
|
|
}));
|
|
|
|
vi.mock('../src/actions/taxSourceActions', () => ({
|
|
validateInvoiceFinalization: vi.fn(async () => ({ canFinalize: true })),
|
|
}));
|
|
|
|
describe('hardDeleteInvoice recurring detail safeguards', () => {
|
|
beforeEach(() => {
|
|
state.invoice = {
|
|
invoice_id: 'invoice-1',
|
|
client_id: 'client-1',
|
|
credit_applied: 0,
|
|
status: 'draft',
|
|
};
|
|
state.hasCanonicalRecurringDetails = true;
|
|
state.queriedTables = [];
|
|
});
|
|
|
|
it('T205: blocks hard deletion once canonical recurring detail periods exist on the invoice', async () => {
|
|
const { hardDeleteInvoice } = await import('../src/actions/invoiceModification.ts');
|
|
|
|
await expect(hardDeleteInvoice('invoice-1')).rejects.toThrow(
|
|
'Cannot delete invoice invoice-1: canonical recurring detail periods already exist. Cancel the invoice instead of deleting it.'
|
|
);
|
|
|
|
expect(state.queriedTables).toContain('invoices');
|
|
expect(state.queriedTables).toContain('invoice_charge_details as iid');
|
|
expect(state.queriedTables).not.toContain('transactions');
|
|
});
|
|
});
|