gtest target tests a locally redefined function, not production code
Los mantenedores suelen responder en 4 días
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 48/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Activo
- Stack tecnológico
- cpp
- Área
- build-system, testing-qa
Línea de trabajo
Comienza con xplane_plugin/tests/test_threading_tools.cpp y xplane_plugin/meson.build, y luego compara la definición local de generate_packet y las llamadas de prueba con threading-tools.h. Revisa los issues relacionados #190, #66 y #103 antes de decidir si el objetivo puede probar código de producción o si debería eliminarse. Se considera terminado cuando meson test ya no presenta un objetivo de prueba aprobado que ejercite únicamente código definido localmente.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Describe the bug
xplane_plugin/tests/test_threading_tools.cpp includes threading-tools.h, which declares generate_packet(vector<string>), then defines its own generate_packet(const string&, const string&, const string&, const string&) at line 11. Both tests call the local four-string version, so overload resolution never reaches the header's declaration. That is also why the header's missing definitions never cause a link error.
test_threading_tools is the only test target in xplane_plugin/meson.build, so the C++ suite currently exercises no production code at all. meson test passing means nothing.
Expected behavior
The suite tests real production code, or the target is removed so a passing run is not mistaken for coverage.
Additional context
ThreadMessage in the header is a four-float struct, predating the current eight-value payload, so the test reflects an obsolete data model. Related to #66 and #103. Should be resolved together with #190.
Found while writing the setup documentation in #138 / #178.
- Lenguaje dominante
- Rust
- Estrellas
- 4
- Forks
- 8
- Merge medio
- 3 d 12 h
- PR fusionados (30 d)
- 16
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una 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 oss-slu/PilotDataSynchronization
-
cluster_pilots.py shows a traceback for a bad --k and leaves outputs when --truth is invalidAbiertoarea:ml bug python
Dificultad 2/5 1-3 horas Aptitud para principiantes 83/100
oss-slu/PilotDataSynchronization#232 ·
Los mantenedores suelen responder en 4 días
-
area:docs area:xplane-plugin documentation ready-for-human
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
oss-slu/PilotDataSynchronization#194 ·
Los mantenedores suelen responder en 4 días
-
area:ml enhancement needs-info python
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
oss-slu/PilotDataSynchronization#233 ·
Los mantenedores suelen responder en 4 días
-
area:ml enhancement python
Dificultad 4/5 3-5 días Aptitud para principiantes 44/100
oss-slu/PilotDataSynchronization#231 ·
Los mantenedores suelen responder en 4 días
-
area:ml enhancement python
Dificultad 4/5 3-5 días Aptitud para principiantes 52/100
oss-slu/PilotDataSynchronization#230 ·
Los mantenedores suelen responder en 4 días
Todos los issues de oss-slu/PilotDataSynchronization
Issues similares
-
Custom hosts written as trailing-dot FQDNs (`host.local.`) aren't detected as local backendsPosiblemente ocupada @roboomp la tomó hoy. Abiertobug prio:p2 providers triaged
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
can1357/oh-my-pi#15798 · 3 comentarios ·
Los mantenedores suelen responder en 1 día
-
requires-triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
apache/datafusion-comet#7001 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
-
builtin-port enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
cmlarsen/writemorebetter#235 ·
Los mantenedores suelen responder en 1 día
-
Add a Storybook section to CONTRIBUTING.mdPosiblemente ocupada @ChanakyaG2004 la tomó hoy. Abiertodocumentation good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
FlowwStar/FlowStar#995 · 2 comentarios ·
Los mantenedores suelen responder en 1 día