No unit tests for four of six payment processors (nodeless, lnurl, opennode, zebedee)
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 72/100
- Tipo di issue
- Funzionalità
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Attiva
- Stack tecnologico
- typescript
Direzione di ricerca
Read the four processor implementations in src/payments-processors/ and the existing specs for nwc and lnbits in test/unit/payments-processors/. Add unit specs covering createInvoice request mapping, getInvoice status mapping, and provider error or malformed response behavior. Done means all four processors have focused specs and the relevant unit tests pass.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Gap
Four of the six payment processors have no unit tests at all — no spec file imports them:
src/payments-processors/nodeless-payments-processor.ts(74 lines)src/payments-processors/lnurl-payments-processor.ts(68 lines)src/payments-processors/opennode-payments-processor.ts(64 lines)src/payments-processors/zebedee-payments-processor.ts(61 lines)
Each implements getInvoice / createInvoice against a third-party API. The nwc and lnbits processors do have specs (test/unit/payments-processors/), and the zebedee/opennode/nodeless callback schemas are tested — so this is an inconsistency, not a project-wide absence. The money-moving code paths for four providers are the ones without coverage.
How measured
Import graph over the suite: 175 spec files vs 325 src/**/*.ts files — zero spec files reference these four modules.
What's worth covering
createInvoicerequest mapping per provider (amounts, units, metadata) — the shape bugs live heregetInvoicestatus mapping (paid/pending/expired) — misreading a provider's status is a funds bug- Error paths: provider outage, malformed responses — what the caller sees and whether anything is retried or lost
- Lingua principale
- TypeScript
- Stelle
- 829
- Fork
- 234
- Merge medio
- 18h 18m
- PR unite (30g)
- 69
Preparare l'ambiente
- Include un Dockerfile o un file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di cameri/nostream
-
rust.sh shares one CARGO_TARGET_DIR across worktrees, so a scratch-copy build poisons the next runAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
I maintainer di solito rispondono entro 1 giorno
-
Website
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
I maintainer di solito rispondono entro 1 giorno
-
feat(payments): subscription lifecycle, renewal invoices and plan-based write accessForse già presa @Priyanshubhartistm l’ha presa oggi. Apertaenhancement
cameri/nostream#908 · 1 assegnatario ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 72/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 72/100
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di cameri/nostream
Issue simili
-
engine-rs P3 review-deferral wasm
Difficoltà 2/5 1-3 ore Idoneità per principianti 77/100
maniator/verticopolis#1154 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
I maintainer di solito rispondono entro 1 giorno
-
Bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
telerik/kendo-react#3861 ·
I maintainer di solito rispondono entro 1 giorno
-
[bug]: use-mobile.ts uses pixel logic, causing UI state de-sync with Tailwind CSS rem breakpointsApertabug
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
I maintainer di solito rispondono entro 1 giorno
-
task
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100