Reduce redundant test coverage at consumer boundaries #41

Merged

eleith merged 1 commits from chore/test-boundary-cleanup into main 2026-11-06 22:56:58 +00:00

Conversation 0 Commits 1 Files Changed 45

+371 -1630

eleith-gitea-agent Bot commented 2026-11-06 22:49:33 +00:00

Copy link

Copy source

Summary

Reduce redundant tests at consumer boundaries while retaining application-owned financial, privacy, publication and lifecycle protection.

This is the independent test-cleanup extraction from the pending homepage work: 45 files, +370/−1629 lines. It changes no production code, dependencies, CI, browser journeys or financial format. Main's format 6 and existing UI remain intact; format 8, period counts, saved engine version and new pages are not included.

Review guide

• Config/jobs: keep date semantics at one owner and representative concrete-validator/spec bindings. Remove schema/constructor inventories and the generic SDK scheduling, migration-locking and drain cases.
• Native/storage: reduce malformed-type products and duplicated codec checks; keep every required stored container and meaningful consumed input protected. Add one complete bad-commodity fixture at its codec owner so another missing field cannot mask the guard.
• Web/worker: remove repeated waiter/enum/caller permutations and incidental platform assertions. Keep exact public projections, explicit work, safe failures and resource ownership.
• Jobs README: describe the retained application integration rather than the deleted synthetic SDK suites.

Real native interpretation, populated SDK migration, SQLite/HTTP disconnect, complete publication, service drain, and actual retention scheduling/deletion tests remain. Five mixed files contain only cleanup hunks; their future feature fixtures were deliberately excluded.

Validation

On an isolated worktree based on 1512cae, with its own dependencies and locally built shared packages:

• Offline frozen install and shared-package build passed.
• 755 tests / 49 files passed: affected tests plus retained owner/native, migration, waiter, preparation/drain and retention integrations.
• All six package/app type checks and lints passed; Svelte check reported 0 errors and 0 warnings.
• Independent review: no must-fix findings.
• Whitespace, exact patch inventory and unchanged production files verified.

No browser/container run or performance improvement is claimed for this test-only PR. No runtime changeset is added. The financial feature will follow separately after this cleanup is merged.

## Summary Reduce redundant tests at consumer boundaries while retaining application-owned financial, privacy, publication and lifecycle protection. This is the independent test-cleanup extraction from the pending homepage work: **45 files, +370/−1629 lines**. It changes no production code, dependencies, CI, browser journeys or financial format. Main's format 6 and existing UI remain intact; format 8, period counts, saved engine version and new pages are not included. ## Review guide - **Config/jobs:** keep date semantics at one owner and representative concrete-validator/spec bindings. Remove schema/constructor inventories and the generic SDK scheduling, migration-locking and drain cases. - **Native/storage:** reduce malformed-type products and duplicated codec checks; keep every required stored container and meaningful consumed input protected. Add one complete bad-commodity fixture at its codec owner so another missing field cannot mask the guard. - **Web/worker:** remove repeated waiter/enum/caller permutations and incidental platform assertions. Keep exact public projections, explicit work, safe failures and resource ownership. - **Jobs README:** describe the retained application integration rather than the deleted synthetic SDK suites. Real native interpretation, populated SDK migration, SQLite/HTTP disconnect, complete publication, service drain, and actual retention scheduling/deletion tests remain. Five mixed files contain only cleanup hunks; their future feature fixtures were deliberately excluded. ## Validation On an isolated worktree based on `1512cae`, with its own dependencies and locally built shared packages: - Offline frozen install and shared-package build passed. - **755 tests / 49 files passed**: affected tests plus retained owner/native, migration, waiter, preparation/drain and retention integrations. - All six package/app type checks and lints passed; Svelte check reported 0 errors and 0 warnings. - Independent review: no must-fix findings. - Whitespace, exact patch inventory and unchanged production files verified. No browser/container run or performance improvement is claimed for this test-only PR. No runtime changeset is added. The financial feature will follow separately after this cleanup is merged.

eleith-gitea-agent Bot added 1 commit 2026-11-06 22:49:33 +00:00

test: reduce redundant consumer coverage ... bf20865fe5

Keep detailed rules at their owning boundaries and retain real native, SQLite, publication and service-lifetime integrations. Production code and financial format remain unchanged.

eleith merged commit bf20865fe5 into main 2026-11-06 22:56:58 +00:00

eleith deleted branch chore/test-boundary-cleanup 2026-11-06 22:56:58 +00:00

eleith-gitea-agent Bot referenced this pull request 2026-11-06 23:25:39 +00:00

Present period-aware Overview and dedicated financial pages #42

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

No items

No labels

Milestone

No items

No Milestone

Assignees

Clear assignees

eleith-gitea-agent Bot eleith (eleith)

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: eleith/hledger-web-reports#41