Widget and golden tests for ClaimStatusAuditTimeline
epic-expense-approval-workflow-core-logic-task-016 — Write flutter_test widget tests for ClaimStatusAuditTimeline covering: rendering with a full multi-event history, rendering the empty state when no events exist, loading state display, error state display, correct Norwegian timezone conversion for a known UTC timestamp, screen reader semantics verification for actor/role/timestamp fields, and golden image tests for the standard multi-event layout.
Acceptance Criteria
Technical Requirements
Execution Context
Tier 8 - 48 tasks
Can start after Tier 7 completes
Implementation Notes
Organize the test file in three sections with comments: `// --- Pure Widget Tests ---`, `// --- Riverpod Integration Tests ---`, `// --- Golden Tests ---`. Create a `_buildFixtures()` helper that returns a deterministic list of `ClaimEvent` objects with fixed, known timestamps to make timezone assertions reproducible. For semantics tests, use `tester.getSemantics(find.byType(AccessibleTimestamp))` and assert on the `SemanticsData.label` property. For golden tests, set an explicit `Size` using `tester.binding.setSurfaceSize(const Size(390, 844))` and reset in `tearDown` to avoid cross-test contamination.
If the golden test is flaky due to font rendering differences between platforms, use `fontFamily: 'Ahem'` in the test theme to neutralize font rendering variance. Ensure the golden PNG files are tracked in git (not in .gitignore).
Testing Requirements
All tests are in a single file `test/widgets/claim_status_audit_timeline_test.dart`. Each test uses `testWidgets()`. For Riverpod-dependent tests (loading, error), wrap with `ProviderScope(overrides: [...])`. For pure widget tests (multi-event, empty), call `ClaimStatusAuditTimeline(events: fixtures)` directly without Riverpod.
Use a `TestWidgetsFlutterBinding.ensureInitialized()` call to initialize the timezone and nb_NO locale in `setUpAll()`. Golden tests use `matchesGoldenFile()` matcher. Run golden update with `flutter test --update-goldens` and commit the resulting PNG files. The golden images must be regenerated whenever the design tokens or layout change.
The ThresholdEvaluationService is described as shared Dart logic used both client-side and in the Edge Function. Supabase Edge Functions run Deno/TypeScript, not Dart, meaning the threshold logic must be maintained in two languages and can diverge, causing the server to reject legitimate client submissions.
Mitigation & Contingency
Mitigation: Implement the threshold logic as a single TypeScript module in the Edge Function and call it via a thin Dart HTTP client wrapper for client-side preview feedback only. The server is always authoritative; the client version is purely for UX (showing the user whether their claim will auto-approve before they submit).
Contingency: If dual-language maintenance is unavoidable, create a shared golden test file (JSON fixtures with inputs and expected outputs) that is run against both implementations in CI to detect divergence immediately.
A peer mentor could double-tap the submit button or a network retry could trigger a duplicate submission, causing the ApprovalWorkflowService to attempt two concurrent state transitions from draftβsubmitted for the same claim, potentially resulting in two audit events or conflicting statuses.
Mitigation & Contingency
Mitigation: Implement idempotency in the ApprovalWorkflowService using a database-level unique constraint on (claim_id, from_status, to_status) per transition, combined with a UI-level submission lock (disable button after first tap until response returns).
Contingency: Add a deduplication check at the start of every state transition method that returns the existing state if an identical transition is already in progress or completed within the last 10 seconds.
Claims with multiple expense lines (e.g., mileage + parking) must have their combined total evaluated against the threshold. If individual lines are added asynchronously or the evaluation runs before all lines are persisted, the auto-approval decision may be computed on an incomplete set of expense lines.
Mitigation & Contingency
Mitigation: The Edge Function always fetches all expense lines from the database (not from the client payload) before computing the threshold decision. Define a clear claim submission contract that requires all expense lines to be persisted before the submit action is called.
Contingency: Add a validation step in ApprovalWorkflowService that counts expected vs. persisted expense lines before allowing the transition, returning a validation error if lines are missing.