CLI: reject invalid player seek and volume values before connecting
Nessuno ha ancora preso questa issue.
- #90 di @bfs2021 — chiusa senza merge
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 75/100
Direzione di ricerca
Inizia dal parsing degli argomenti della CLI per i sottocomandi spotapi player seek e spotapi player volume, e leggi i controlli di intervallo esistenti in Player.seek() e Player.volume() per adottarne gli stessi limiti. Aggiungi piccoli validatori di tipo argparse che rifiutino valori di seek negativi e volumi fuori da 0-100 con exit status 2, mantenendo validi 0 e 100. È completato quando i test offline coprono seek negativo, volume sotto zero, volume sopra 100 e entrambi i limiti, e la issue chiede di commentare prima di iniziare per evitare lavoro duplicato.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Problem
The Python player API correctly rejects negative seek positions and volume values outside 0–100. The CLI currently parses both as unrestricted integers, so obviously invalid input can reach account/session setup before producing an error.
Proposed change
Validate these values during argument parsing:
spotapi player seek MILLISECONDS: integer greater than or equal to zero.spotapi player volume PERCENT: integer from zero through 100.
Acceptance criteria
- Invalid values fail through
argparsewith a clear range message and exit status 2. - Invalid input is rejected before reading account state or opening a Spotify connection.
- Boundary values
0and100remain valid where applicable. - Existing valid commands retain their current behavior.
- Offline tests cover negative seek, volume below zero, volume above 100, and boundaries.
Scope
Keep validation helpers small and local to the CLI. The existing validation inside Player.seek() and Player.volume() must remain as the library-level safety boundary.
Comment before starting so duplicate work can be avoided.
- Lingua principale
- Python
- Stelle
- 493
- Fork
- 40
- Merge medio
- 5m
- PR unite (30g)
- 1
Preparare l'ambiente
Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: 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 Aran404/SpotAPI
-
CLI: expose public search language with --languageForse già presa @semajzandrews l’ha presa oggi. Apertaenhancement good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
-
documentation enhancement help wanted
Difficoltà 4/5 3-5 giorni Idoneità per principianti 42/100
Tutte le issue di Aran404/SpotAPI
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 66/100
deepchem/deepchem#5289 · 1 commento ·
I maintainer di solito rispondono entro 2 giorni
-
Pull Request for pretty_macAperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 75/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 60/100
UKGovernmentBEIS/inspect_evals#2892 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Versioning docs: say that no output from the 0.46 warning check means nothing is affectedForse già presa @grelinfo l’ha presa oggi. Apertadocumentation priority:low size:S
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 72/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 88/100
alphacrack/readme2demo#293 ·