Three repositories have no unit tests (notification-outbox, notification-delivery-log, nip05-verification)
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 72/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- typescript
Línea de trabajo
Start with the three named repository files and compare their patterns with the existing user, invoice, event, and invite-code repository specs. Then inspect the notification-outbox service spec to understand its current stubs. Add unit coverage for the listed SQL behavior and state transitions; done when each repository has tests for the specified cases, including admin filters and delivery-log outcomes.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Gap
Three repositories have no unit tests — no spec file imports them:
src/repositories/notification-outbox-repository.ts(162 lines)src/repositories/nip05-verification-repository.ts(140 lines)src/repositories/notification-delivery-log-repository.ts(96 lines)
Note the trap: test/unit/services/notification-outbox-service.spec.ts exists, but it stubs the repository (claimBatch, markDelivered, markFailed are all sinon stubs) — so the 162 lines of actual SQL in the repository are untested. The user, invoice, event, and invite-code repositories all have specs; these three don't.
How measured
Import graph over the suite: 175 spec files vs 325 src/**/*.ts files — zero spec files reference these three modules (the service spec references the repository only as a stubbed interface).
What's worth covering
- Outbox
claimBatchsemantics: batch claiming without double-delivery across workers markDelivered/markFailedstate transitions and failure-count accountingnip05-verification-repository.listForAdminfilters (domain, verified, stale, minFailures, pagination) — new in #895 and already serving the admin API untested- Delivery-log writes: what gets recorded on success vs failure
- Lenguaje dominante
- TypeScript
- Estrellas
- 829
- Forks
- 234
- Merge medio
- 18 h 18 min
- PR fusionados (30 d)
- 69
Preparar el entorno
- Incluye un Dockerfile o un archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de cameri/nostream
-
rust.sh shares one CARGO_TARGET_DIR across worktrees, so a scratch-copy build poisons the next runAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Los mantenedores suelen responder en 1 día
-
Website
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
Los mantenedores suelen responder en 1 día
-
feat(payments): subscription lifecycle, renewal invoices and plan-based write accessPosiblemente ocupada @Priyanshubhartistm la tomó hoy. Abiertoenhancement
cameri/nostream#908 · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
Dificultad 3/5 1-2 días Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día
-
Dificultad 3/5 1-2 días Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día
Todos los issues de cameri/nostream
Issues similares
-
RED ladder stopped @ 4db4ed91Abiertoarea:workflow bug ready-for-human
Dificultad 1/5 Menos de una hora Aptitud para principiantes 78/100
fil-donadoni/tolaria#6108 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
apache/rocketmq-dashboard#7234 ·
Los mantenedores suelen responder en 3 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
vitest-dev/vitest#11656 ·
Los mantenedores suelen responder en 1 día
-
e2e-failure ready-to-code
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
redhat-developer/rhdh-plugin-export-overlays#4460 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
ehmpathy/rhachet-roles-bhrain#655 ·
Los mantenedores suelen responder en 1 día