Hacktoberfest 2026: die Issues, die Maintainer für den Oktober markiert haben – offen und einsteigerfreundlich. Hacktoberfest-Issues durchsuchen

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

Offen
  • Stimmt etwas nicht?
#905 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Maintainer antworten meist innerhalb von 1 Tag

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Anfängerfreundlichkeit
72/100
Issue-Typ
Bug
Klarheit
Klar beschrieben
Aktivitätsstatus
Aktiv
Tech-Stack
typescript
Bereich
database, testing

Rechercherichtung

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.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

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
Vorherrschende Sprache
TypeScript
Sterne
829
Forks
234
Ø Merge
18 Std. 18 Min.
Gemergte PRs (30 T.)
69

Entwicklungsumgebung

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus cameri/nostream

Alle Issues in cameri/nostream

Ähnliche Issues

Weitere Issues zu TypeScript

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.