Three repositories have no unit tests (notification-outbox, notification-delivery-log, nip05-verification)
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 72/100
- Type d'issue
- Bug
- Clarté
- Clairement spécifiée
- Activité
- Active
- Stack technique
- typescript
Piste de recherche
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.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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
- Langage dominant
- TypeScript
- Étoiles
- 829
- Forks
- 234
- Merge moyen
- 18 h 18 min
- PR mergées (30 j)
- 69
Préparer son environnement
- Fournit un Dockerfile ou un fichier Docker Compose
- Propose un modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de cameri/nostream
-
rust.sh shares one CARGO_TARGET_DIR across worktrees, so a scratch-copy build poisons the next runOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
Les mainteneurs répondent en général sous 1 jour
-
Website
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
Les mainteneurs répondent en général sous 1 jour
-
feat(payments): subscription lifecycle, renewal invoices and plan-based write accessPeut-être pris @Priyanshubhartistm l’a pris aujourd’hui. Ouverteenhancement
cameri/nostream#908 · 1 personne assignée ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 3/5 1-2 jours Accessibilité débutants 72/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 3/5 1-2 jours Accessibilité débutants 72/100
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de cameri/nostream
Issues similaires
-
Edit: Travel Mix (RO)Ouvertecheck:failed streams:edit
Difficulté 2/5 1-3 heures Accessibilité débutants 64/100
iptv-org/iptv#55901 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
Les mainteneurs répondent en général sous 2 jours
-
package:ios ships XCFrameworks with absolute symlinks, which SwiftPM refuses to extractPeut-être pris @cusnirvladislav l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
callstackincubator/rock#740 ·
-
S: triage
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
Les mainteneurs répondent en général sous 2 jours
-
RED ladder stopped @ 4db4ed91Ouvertearea:workflow bug ready-for-human
Difficulté 1/5 Moins d'une heure Accessibilité débutants 78/100
fil-donadoni/tolaria#6108 ·
Les mainteneurs répondent en général sous 1 jour