Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

Three repositories have no unit tests (notification-outbox, notification-delivery-log, nip05-verification)

Aperta
#905 0 commenti 0 reazioni 1 assegnatario Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

@Priyanshubhartistm ci sta già lavorando.

Dal 20/10/2026.

  • #918 di @Priyanshubhartistm — aperta

Valutazione

Difficoltà
4/5
Tempo stimato
3-5 giorni
Idoneità per principianti
72/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
typescript
Ambito
database, testing

Direzione di ricerca

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.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

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 claimBatch semantics: batch claiming without double-delivery across workers
  • markDelivered / markFailed state transitions and failure-count accounting
  • nip05-verification-repository.listForAdmin filters (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
Lingua principale
TypeScript
Stelle
829
Fork
234
Merge medio
17h 46m
PR unite (30g)
73

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di cameri/nostream

Tutte le issue di cameri/nostream

Issue simili

Altre issue su TypeScript

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.