[tests]: are "rda file made when indicated" tests needed for every plot?
I maintainer di solito rispondono entro 1 giorno
@Schiano-NOAA ci sta già lavorando.
Dal 14/10/2026.
Valutazione
Questa issue non è ancora stata valutata.
Descrizione
For every plot, there's an associated "rda file made when indicated" test that runs a simple version of the plotting function with make_rda=TRUE specified, and then checks whether the associated ".rda" and ".csv" file are made, before deleting them. Given that all of these tests are identical (minus the exact plotting function call), would it make more sense to remove all of these tests, and instead just test the
export_kqs(...)
insert_kqs(...)
create_rda(...)
workflow, once? This eliminates a large number of largely redundant tests, and would alleviate many I/O issues associated with moving to a parallel testing framework.
The case for retaining all of the plot specific "rda file made when indicated" tests would be to verify that the feature was correctly implemented for the specific plot, but this check only really needs to be conducted once, rather than every time the test suite is executed. Maybe this is an automated check that could be done before a PR is approved?
- Lingua principale
- R
- Stelle
- 20
- Fork
- 9
- Merge medio
- 1g 8h
- PR unite (30g)
- 11
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
-
[tests]: Set modules in all plot_* calls within test functionsForse già presa @Schiano-NOAA l’ha presa 2 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
nmfs-ost/stockplotr#416 · 5 commenti · 1 reazione · 2 assegnatari ·
I maintainer di solito rispondono entro 1 giorno
-
Make tests more efficientForse già presa @JZahner1 l’ha presa oggi. Aperta
nmfs-ost/stockplotr#420 · 3 commenti · 2 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
-
[feat]: Remove `html_all_figs_tables()` in favor of dashboardForse già presa @Schiano-NOAA l’ha presa 2 giorni fa. Aperta
nmfs-ost/stockplotr#413 · 1 assegnatario ·
I maintainer di solito rispondono entro 1 giorno
-
[feat]: Create function to help user directly add alt text and caption to csv for external objectsForse già presa @sbreitbart-NOAA l’ha presa 2 giorni fa. Aperta
nmfs-ost/stockplotr#412 · 1 assegnatario ·
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 66/100
bennettoxford/opencodecounts#170 ·
-
bug complexity::1-low effort::1-hours priority::2-should
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
jumpingrivers/audit.base#61 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
Pascal-Kueng/dyadMLM#92 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 64/100
urbio-ecology/urbioconnect#170 ·
-
bug new windows
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
I maintainer di solito rispondono entro 1 giorno