test: Nit test addition to otlp

[cijothomas]

Minor test addition.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.6%. Comparing base (373fc51) to head (e8e424e).
⚠️ Report is 16 commits behind head on main.

Additional details and impacted files

@@          Coverage Diff          @@
##            main   #3390   +/-   ##
=====================================
  Coverage   82.6%   82.6%           
=====================================
  Files        128     128           
  Lines      24641   24663   +22     
=====================================
+ Hits       20355   20384   +29     
+ Misses      4286    4279    -7     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[cijothomas]

@scottgerring PTAL.

[scottgerring]

Good idea.

I thought we might be able to prevent even expressing this by guarding the enum enumerants against the feature flags, but it is shared between both transport layer choices as well as the env parsing and I think the implicit error there - "this encoding doesn't exist" - is worse.

LGTM

[cijothomas]

Good idea.

I thought we might be able to prevent even expressing this by guarding the enum enumerants against the feature flags, but it is shared between both transport layer choices as well as the env parsing and I think the implicit error there - "this encoding doesn't exist" - is worse.

LGTM

Did you meant to request changes to the PR?