Implement Display on Baggage

[wperron]

Tests are taken from the Go test cases

Fixes #905

[codecov]

Codecov Report

Base: 67.1% // Head: 67.2% // Increases project coverage by +0.0% 🎉

Coverage data is based on head (47f9da5) compared to base (44b9020).
Patch coverage: 100.0% of modified lines in pull request are covered.

Additional details and impacted files

@@          Coverage Diff          @@
##            main    #921   +/-   ##
=====================================
  Coverage   67.1%   67.2%           
=====================================
  Files        115     115           
  Lines       8959    8998   +39     
=====================================
+ Hits        6016    6048   +32     
- Misses      2943    2950    +7     

Impacted Files	Coverage Δ
opentelemetry-api/src/baggage.rs	87.5% <100.0%> (+2.9%)	⬆️
opentelemetry-http/src/lib.rs	3.5% <0.0%> (-5.3%)	⬇️
opentelemetry-sdk/src/trace/span_processor.rs	83.0% <0.0%> (-0.3%)	⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[wperron]

Is that failing test a known flaky test? it's passing locally and I'm not sure why it'd be failing here 🤔

[CleanCut]

@wperron Hmm, cargo test passes for me as well. But if I run script/test.sh then it fails just as in CI, and it changes three files (which need to be committed and added to the PR according to the output).

[wperron]

Ah makes sense, I'll get that updated sometime today 👍

[TommyCpp]

@jtescher Looks like we need your approval to merge. Any more comments or it's good to merge?