envoyproxy / envoyproxy/nighthawk
Coverage: ensure we hit the case where the warmup request fails.
Abierto
good first issue
help wanted
tech-debt
- Lenguaje dominante
- C++
- Estrellas
- 414
- Forks
- 95
- Merge medio
- 2 d 9 h
- PR fusionados (30 d)
- 11
Descripción
Just observed that this line isn't always hit, and caused a dip under the coverage threshold in CI:
https://3858-180498819-gh.circle-artifacts.com/0/coverage/coverage/proc/self/cwd/source/client/client_worker_impl.cc.html#L38
We should add a dedicated test to ensure we hit it.
Guía de contribución
Evaluación
Este issue todavía no se ha evaluado.