Have `WithSeeds` accept the `GoogleTest` `ParameterGenerator`s (such as `ValuesIn`, `Combine`, etc).
Los mantenedores suelen responder en 2 días
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 35/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- cpp
- Área
- api, testing-qa
Línea de trabajo
Comienza con la API de WithSeeds y la documentación de GoogleTest sobre generadores de parámetros, especialmente ValuesIn y Combine. Determina cómo las entradas de generadores existentes pueden conservar valores deterministas y admitir parámetros individuales sin envolverlos en tuplas. Se considera completado cuando WithSeeds acepta los generadores de GoogleTest pertinentes y mantiene el comportamiento esperado de las semillas.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
I'm currently in the process of pitching my team on using FUZZ_TEST instead of regular old GoogleTest TEST_P. Most of our tests currently use TEST_P, and my goal is to incrementally convert them from TEST_P to FUZZ_TEST keeping the behavior the same other than fuzzing. This means keeping the deterministically-executed values the same, which I plan to do with WithSeeds.
Unfortunately, WithSeeds is less flexible than the GoogleTest parameter generators in a number of important ways:
-
In the case where a
TestWithParamis parameterized on a single parameter,TestWithParamdoes not insist that that single parameter is wrapped in astd::tuple, butWithSeedsdoes. -
GoogleTestprovidesCombine, which allows taking lists of parameter generators and performing the cartesian product of them, producing all possible combinations of the parameter generator's outputs. To replicate this, it looks like I'll either have to manually generate each possible combination or whittle down the inputs and argue to my team that it's ok.
Ideally, I'd like to do neither of these things. I'd like to just continue passing the same parameter generators I sent into INSTANTIATE_TEST_SUITE_WITH_PARAMS into WithSeeds and have it figure it out. Doing so would make convincing others that I am not introducing regressions much simpler.
- Lenguaje dominante
- C++
- Estrellas
- 1.1k
- Forks
- 139
- Merge medio
- 3 d 2 h
- PR fusionados (30 d)
- 23
Preparar el entorno
- 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 google/fuzztest
-
setup_configs should respect deprecation of _LIBCPP_ENABLE_ASSERTIONSPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 2 días
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 68/100
Los mantenedores suelen responder en 2 días
-
Dificultad 3/5 1-2 días Aptitud para principiantes 52/100
Los mantenedores suelen responder en 2 días
-
Dificultad 3/5 1-2 días Aptitud para principiantes 52/100
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 45/100
google/fuzztest#1898 · 1 reacción ·
Los mantenedores suelen responder en 2 días
Todos los issues de google/fuzztest
Issues similares
-
【问题】在文件树重命名文件,发现无法用鼠标选中一段文字Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 76/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 65/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 77/100
ros-navigation/navigation2#6648 · 1 comentario ·
Los mantenedores suelen responder en 1 día