envoyproxy / envoyproxy/nighthawk

Coverage: ensure we hit the case where the warmup request fails.

Đang mở
#145 0 bình luận 0 reaction 0 người được giao Xem trên GitHub
good first issue help wanted tech-debt
Ngôn ngữ chính
C++
Star
414
Fork
95
Merge trung bình
2 ngày 9 giờ
Pull request đã merge (30 ngày)
11

Mô tả

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.

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Đánh giá

Issue này chưa được đánh giá.

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.