Three repositories have no unit tests (notification-outbox, notification-delivery-log, nip05-verification)
Mantenedores costumam responder em até 1 dia
@Priyanshubhartistm já está trabalhando nisso.
Desde 20/10/2026.
- #918 de @Priyanshubhartistm — aberto
Avaliação
- Dificuldade
- 4/5
- Tempo estimado
- 3-5 dias
- Facilidade para iniciantes
- 72/100
- Tipo de issue
- Bug
- Clareza
- Claramente especificada
- Status de atividade
- Ativa
- Stack de tecnologia
- typescript
Direção de pesquisa
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.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
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
- Linguagem predominante
- TypeScript
- Estrelas
- 829
- Forks
- 234
- Merge médio
- 17h 46min
- PRs com merge (30d)
- 73
Preparar o ambiente
- Inclui um Dockerfile ou arquivo Docker Compose
- Tem um modelo de pull request
- Ler o guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de cameri/nostream
-
Website
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 84/100
Mantenedores costumam responder em até 1 dia
-
feat(payments): subscription lifecycle, renewal invoices and plan-based write accessTalvez já em andamento @Priyanshubhartistm assumiu há 1 dia. Abertaenhancement
cameri/nostream#908 · 1 responsável ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 72/100
Mantenedores costumam responder em até 1 dia
-
No unit tests for four of six payment processors (nodeless, lnurl, opennode, zebedee)Talvez já em andamento @Priyanshubhartistm assumiu hoje. Aberta
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 72/100
cameri/nostream#903 · 1 responsável ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 5/5 Mais de uma semana Facilidade para iniciantes 25/100
cameri/nostream#902 · 1 comentário ·
Mantenedores costumam responder em até 1 dia
Todas as issues de cameri/nostream
Issues semelhantes
-
skillopt: success-reflect prompt omits the edit schema, so every success-only edit is droppedAberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
Mantenedores costumam responder em até 1 dia
-
Workspace secrets/constants containing "$" are corrupted when resolved in data source optionsAbertabug
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 84/100
Mantenedores costumam responder em até 1 dia
-
bug
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 67/100
mobile-next/mobile-mcp#526 ·
Mantenedores costumam responder em até 1 dia
-
e2e-failure ready-to-code
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 73/100
redhat-developer/rhdh-plugin-export-overlays#4470 · 1 comentário ·
Mantenedores costumam responder em até 1 dia