[tests]: Set modules in all plot_* calls within test functions
I maintainer di solito rispondono entro 1 giorno
@Schiano-NOAA ci sta già lavorando.
Dal 13/10/2026.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 68/100
Direzione di ricerca
Il payload non indica i file di test, quindi inizia eseguendo la suite di test in locale e annotando dove compare il prompt interattivo del modulo. Poi trova le chiamate plot_* nelle funzioni di test che omettono l'argomento module. Aggiungi un valore valido di module a ciascuna di queste chiamate; la issue lascia aperta la scelta del valore, quindi verifica come le funzioni di plotting lo usano. Il lavoro è completato quando la suite viene eseguita senza alcun prompt e un test opzionale per filter_data copre il percorso di selezione da parte dell'utente.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Presently, all tests of the plotting function omit the module parameter, which eventually prompts the user to select a module via the R terminal. This makes running the tests suite locally not automatic (the user has to continuously interact with the tests by selecting modules). It would probably be more efficient to set the module parameter in each call to a plot_* function within the testing suite, rather than leaving it to wait for a user prompt.
An additional test could be created specifically for filter_data that tests that the user selection works, if there are concerns about that functionality breaking.
- Lingua principale
- R
- Stelle
- 20
- Fork
- 9
- Merge medio
- 1g 5h
- PR unite (30g)
- 12
Preparare l'ambiente
Avvia il container di sviluppo del progetto nel browser, con il tuo account GitHub.
- Nessun Dockerfile né file Docker Compose
- Nessun 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 nmfs-ost/stockplotr
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 15/100
nmfs-ost/stockplotr#422 ·
I maintainer di solito rispondono entro 1 giorno
-
[fix]: module name selection retained in environment when `make_rda = T`Forse già presa @sbreitbart-NOAA l’ha presa 1 giorno fa. Apertabug
nmfs-ost/stockplotr#421 · 5 commenti · 1 assegnatario ·
I maintainer di solito rispondono entro 1 giorno
-
Make tests more efficientForse già presa @JZahner1 l’ha presa 1 giorno fa. Aperta
nmfs-ost/stockplotr#420 · 4 commenti · 2 assegnatari ·
I maintainer di solito rispondono entro 1 giorno
-
[tests]: are "rda file made when indicated" tests needed for every plot?Forse già presa @Schiano-NOAA l’ha presa 2 giorni fa. Apertaquestion
nmfs-ost/stockplotr#418 · 10 commenti · 3 assegnatari ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 40/100
nmfs-ost/stockplotr#415 ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di nmfs-ost/stockplotr
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 64/100
mitchelloharawild/ggtime#39 ·
-
enhancement good first issue help wanted
Difficoltà 2/5 Mezza giornata Idoneità per principianti 82/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
-
question
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
-
Breaking 🏴☠️ Consistency :green_apple: :apple:
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 70/100
easystats/bayestestR#789 ·
I maintainer di solito rispondono entro 1 giorno