Test depth on the money paths: raise the playerActions ratchet; integration tests against a real chain
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 35/100
- Tipo di issue
- Funzionalità
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Attiva
- Stack tecnologico
- blockchain, typescript
- Ambito
- blockchain, frontend, testing
Direzione di ricerca
Start with src/hooks/playerActions/, jest.config.js, and src/submit/ to map existing test coverage and the current 45/38/43/45 ratchet. Then inspect the available pokerchaind or pokerchain e2e job and how ignite chain serve could run in CI. Done means new tests cover failed execution, stale index, sequence mismatch, and forced-timeout races while the ratchet increases and no stub-only assumption remains.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Follow-ups split out of #619 when it closed (coverage in CI landed in #622; the lint job lives with #601).
- Raise the
src/hooks/playerActions/ratchet as tests are added. #622 pinned it at 45/38/43/45 (statements/branches/functions/lines) because 17 of 25 files there have no test pairing — it holds the line today and nothing more. Each new test file should move the ratchet injest.config.jsup behind it; never lower a ratchet to make a build pass. - Integration tests against a real chain + SDK, not the stub. The review's sharpest point: a faithful stub can still reproduce the client's own mistaken assumptions. The stub-based e2e suite passed while #609's confirmation bug was live, precisely because the stub behaved as the UI expected — it records actions under the names the UI sends and never rejects a broadcast at execution. A small suite that drives the UI (or at least
src/submit/+ the SDK signing client) against a localpokerchaind— a failed-execution tx (code ≠ 0), a stale index, a sequence mismatch, a forced timeout superseding a late call — would catch that class. Candidates:ignite chain servein CI with the embedded engine, or the pokerchain e2e job if one exists.
- Lingua principale
- TypeScript
- Stelle
- 2
- Fork
- 5
- Merge medio
- 22h 35m
- PR unite (30g)
- 91
Preparare l'ambiente
Non abbiamo ancora controllato i file di configurazione di questo progetto. Parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.
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 block52/ui
-
enhancement tech debt
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 1 giorno
-
Hero hole cards render face down after ~1h on a table (auth signature never renewed)Forse già presa @bitcoinbrisbane l’ha presa oggi. Apertabug UI Week 2
block52/ui#953 · 2 assegnatari ·
I maintainer di solito rispondono entro 1 giorno
-
Time bank clock is hidden under the NFT avatar: move it left of the seat numberForse già presa @Dzeezbtc l’ha presa oggi. Apertagame-flow UI ux Week 2
block52/ui#950 · 1 assegnatario ·
I maintainer di solito rispondono entro 1 giorno
-
feat(ui): player notes and tags — click a seat to note a player; tag icons (fish, shark, tight, loose…) show by their pillForse già presa @Dzeezbtc l’ha presa 1 giorno fa. Aperta
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
I maintainer di solito rispondono entro 1 giorno
-
chore(ui): move existing inline null/empty and typeof checks to the guardsForse già presa @bitcoinbrisbane l’ha presa 2 giorni fa. Apertarefactor tech debt UI
block52/ui#937 · 1 assegnatario ·
I maintainer di solito rispondono entro 1 giorno
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
I maintainer di solito rispondono entro 1 giorno
-
Broken links in the docsAperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 83/100
I maintainer di solito rispondono entro 1 giorno
-
🔊 a11y
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
betagouv/mon-entreprise#4726 ·
I maintainer di solito rispondono entro 9 giorni
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
Deltares/fews-pi-requests#357 ·
I maintainer di solito rispondono entro 1 giorno
-
bug confirmed js v6
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
I maintainer di solito rispondono entro 1 giorno