[tests]: Set modules in all plot_* calls within test functions
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 68/100
Línea de trabajo
El payload no nombra los archivos de prueba, así que empieza ejecutando la suite de pruebas localmente y anotando dónde aparece el aviso interactivo del módulo. Luego busca las llamadas a plot_* en las funciones de prueba que omiten el argumento module. Añade un valor válido de module a cada una de esas llamadas; el issue deja abierta la elección del valor, así que revisa cómo lo usan las funciones de graficado. Está terminado cuando la suite se ejecuta sin ningún aviso, y una prueba opcional para filter_data cubre la ruta de selección del usuario.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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.
- Lenguaje dominante
- R
- Estrellas
- 20
- Forks
- 8
- Merge medio
- 23 h 33 min
- PR fusionados (30 d)
- 9
Preparar el entorno
Inicia el contenedor de desarrollo del proyecto en tu navegador, con tu propia cuenta de GitHub.
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de nmfs-ost/stockplotr
-
Dificultad 3/5 1-2 días Aptitud para principiantes 40/100
nmfs-ost/stockplotr#415 ·
Los mantenedores suelen responder en 1 día
-
[feat[: add citation file: https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/customizing-your-repository/about-citation-filesPosiblemente ocupada @sbreitbart-NOAA la tomó hoy. Abierto
nmfs-ost/stockplotr#414 · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
[feat]: Remove `html_all_figs_tables()` in favor of dashboardPosiblemente ocupada @Schiano-NOAA la tomó hoy. Abierto
nmfs-ost/stockplotr#413 · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
[feat]: Create function to help user directly add alt text and caption to csv for external objectsPosiblemente ocupada @sbreitbart-NOAA la tomó hoy. Abierto
nmfs-ost/stockplotr#412 · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
Reduce runtime of `convert_output` testsPosiblemente ocupada @JZahner1 la tomó hace 4 días. Abierto
nmfs-ost/stockplotr#409 · 2 comentarios · 1 asignado ·
Los mantenedores suelen responder en 1 día
Todos los issues de nmfs-ost/stockplotr
Issues similares
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
-
add bootmlmAbierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 62/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
extendr/extendr.github.io#134 ·