- Notifications
You must be signed in to change notification settings - Fork 234
test: add unit tests for app workers #489
Copy link
Copy link
Open
Labels
good first issueGood for newcomersGood for newcomershelp wantedExtra attention is neededExtra attention is neededtestUnit or integration test improvementsUnit or integration test improvements
Description
Metadata
Metadata
Assignees
Labels
good first issueGood for newcomersGood for newcomershelp wantedExtra attention is neededExtra attention is neededtestUnit or integration test improvementsUnit or integration test improvements
Summary
The app-level workers have very low or zero coverage.
static-mirroring-worker.tshas 0% coverage entirely.Files to cover
src/app/static-mirroring-worker.ts(0%)src/app/app.ts(20% statements, 0% branches)src/app/worker.ts(52% statements, 9% branches)src/app/maintenance-worker.ts(43% statements, 13% branches)Out of scope
src/services/maintenance-service.ts— already at 100%src/services/payments-service.ts— covered by test: add unit tests for MaintenanceWorker and PaymentsService #440src/app/Acceptance criteria
.spec.tsundertest/unit/app/npm run cover:unitpasses without regressions