[CODE HEALTH] Move trace and baggage propagation test classes into anonymous namespace - #4199
Conversation
…onymous namespace Resolves the misc-use-internal-linkage clang-tidy warnings for the TextMapCarrierTest and BaggageCarrierTest helper classes in the b3, http_text_format, jaeger and baggage propagation tests by giving them internal linkage. Part of open-telemetry#4196. Signed-off-by: Mateen Anjum <mateenali66@gmail.com>
Signed-off-by: Mateen Anjum <mateenali66@gmail.com>
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #4199 +/- ##
==========================================
+ Coverage 82.89% 82.90% +0.01%
==========================================
Files 405 405
Lines 17292 17289 -3
==========================================
- Hits 14332 14331 -1
+ Misses 2960 2958 -2 🚀 New features to boost your workflow:
|
|
These look clean. One cross-PR heads-up since #4199 and #4200 overlap on the ratchet file: both lower |
Moves the
TextMapCarrierTesthelper in the b3, http_text_format and jaeger trace propagation tests, andBaggageCarrierTestin the baggage propagation test, into anonymous namespaces, resolving theirmisc-use-internal-linkageclang-tidy warnings. Test-only, no behavior change.Lowered the clang-tidy
warning_limitby 4 for both ABI presets accordingly.Part of #4196.