MichilisandClaude Opus 5
5dc284c5f2
The dashboard, the deadline engine, the three sweeps and the notification channels. Confirm a comprobante and the IVA and IRP figures move; every headline number opens the documents behind it. The dashboard does no arithmetic of its own: it picks inputs and calls packages/rules. The acceptance test asserts the API's numbers equal what computeF120 and computeF515 produce over the same rows, before and after the bandeja is confirmed, so a drift in either direction fails. Deadlines come from the calendario perpetuo, skipping any period earlier than the date the taxpayer took the obligation on. Without that guard a brand new account opens on a red overdue card for a period that predates it. Sweeps are idempotent by dedupe key rather than bookkeeping: a restarted poller, a second replica and a crash mid-sweep all converge on one run, and one reminder per user per period per milestone. They queue notifications rather than sending them, so a channel being down retries on the job schedule. The T-10 test derives the date from dueDateFor rather than restating the calendar. Auto-confirm only touches what the rules were confident about and never a decision the user already made. An unconfigured channel is absent rather than broken: the fan-out skips it, the UI hides it, and a frozen account receives nothing. Web: the dashboard's three zones with swipeable position cards and traceable numbers, the deadlines timeline, and the app shell with the tab bar and the persistent scan button FLOWS.md asks for. Two additions to the specs, both marked: hasDocuments on DashboardDto, without which a position of all zeros is indistinguishable from a real one and the first-run state never shows; and an insight_dismissals table, which FLOWS.md requires and SPEC.md has nowhere to put. Also: pnpm db:reset, because the e2e suite changes the seed it runs against and a suite that is not repeatable is not a suite. 280 vitest tests, 51 Playwright tests, rules coverage still 100%, typecheck and lint clean. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
65 lines
2.8 KiB
TypeScript
65 lines
2.8 KiB
TypeScript
import { es } from '@impuestos/i18n';
|
|
import { expect, test } from '@playwright/test';
|
|
|
|
/**
|
|
* Read-only bandeja coverage, so the most phone-centric screen in the product is checked
|
|
* on a phone viewport too. Nothing here confirms or rejects: the mutating flows live in
|
|
* ingestion.spec.ts and run in a single project to avoid two workers racing for a card.
|
|
*/
|
|
test.describe('bandeja', () => {
|
|
test.beforeEach(async ({ page }) => {
|
|
await page.goto('/es/login');
|
|
// Carlos, not Maria: the mutating ingestion flows work through Maria's bandeja, and
|
|
// two workers pulling cards from the same stack makes this inherently flaky.
|
|
await page.getByLabel(es['auth.register.email']).fill('carlos@demo.local');
|
|
await page.getByLabel(es['auth.login.password']).fill('demo-carlos-1');
|
|
await page.getByRole('button', { name: es['auth.login.submit'] }).click();
|
|
await expect(page).toHaveURL(/\/es\/inicio$/);
|
|
await page.goto('/es/bandeja');
|
|
});
|
|
|
|
test('shows a card with every gesture mirrored as a button', async ({ page }) => {
|
|
const card = page.getByTestId('bandeja-card');
|
|
await expect(card).toBeVisible();
|
|
|
|
// Every swipe has a button, which is what makes the screen usable without gestures.
|
|
await expect(page.getByRole('button', { name: es['bandeja.confirm'] })).toBeEnabled();
|
|
await expect(page.getByRole('button', { name: es['scan.result.changeCat'] })).toBeEnabled();
|
|
await expect(page.getByRole('button', { name: es['scan.result.discard'] })).toBeEnabled();
|
|
await expect(page.getByText(es['bandeja.swipeHint'])).toBeVisible();
|
|
});
|
|
|
|
test('opens the category picker with all eight categories', async ({ page }) => {
|
|
await page.getByRole('button', { name: es['scan.result.changeCat'] }).click();
|
|
|
|
for (const key of [
|
|
'categories.alimentacion',
|
|
'categories.salud',
|
|
'categories.educacion',
|
|
'categories.vivienda',
|
|
'categories.vestimenta',
|
|
'categories.esparcimiento',
|
|
'categories.vehiculo',
|
|
'categories.familiares',
|
|
] as const) {
|
|
await expect(page.getByRole('button', { name: es[key], exact: true })).toBeVisible();
|
|
}
|
|
|
|
await page.getByRole('button', { name: es['common.cancel'] }).click();
|
|
await expect(page.getByRole('button', { name: es['bandeja.confirm'] })).toBeVisible();
|
|
});
|
|
|
|
test('J and K move through the stack without changing anything', async ({ page }) => {
|
|
const first = await page.getByTestId('bandeja-card').locator('p').first().textContent();
|
|
|
|
await page.keyboard.press('j');
|
|
await expect(async () => {
|
|
const next = await page.getByTestId('bandeja-card').locator('p').first().textContent();
|
|
expect(next).not.toBe(first);
|
|
}).toPass({ timeout: 5_000 });
|
|
|
|
await page.keyboard.press('k');
|
|
await expect(page.getByTestId('bandeja-card').locator('p').first()).toHaveText(first ?? '');
|
|
});
|
|
});
|