Skip to content

metricdatatest: support BYTESLICE attribute comparisons#8167

Open
pellared wants to merge 2 commits intoopen-telemetry:mainfrom
pellared:fix-byteslice
Open

metricdatatest: support BYTESLICE attribute comparisons#8167
pellared wants to merge 2 commits intoopen-telemetry:mainfrom
pellared:fix-byteslice

Conversation

@pellared
Copy link
Copy Markdown
Member

@pellared pellared commented Apr 9, 2026

Address #7948 (comment)

@pellared pellared added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Apr 9, 2026
@pellared pellared changed the title metricdatatest: support byte slice attribute comparisons metricdatatest: support BYTESLICE attribute comparisons Apr 9, 2026
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 9, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.3%. Comparing base (112bed7) to head (05ec79a).

Additional details and impacted files

Impacted file tree graph

@@          Coverage Diff          @@
##            main   #8167   +/-   ##
=====================================
  Coverage   82.2%   82.3%           
=====================================
  Files        309     309           
  Lines      24211   24214    +3     
=====================================
+ Hits       19906   19932   +26     
+ Misses      3927    3904   -23     
  Partials     378     378           
Files with missing lines Coverage Δ
...dk/metric/metricdata/metricdatatest/comparisons.go 97.4% <100.0%> (+4.7%) ⬆️

... and 3 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@pellared pellared marked this pull request as ready for review April 9, 2026 14:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Skip Changelog PRs that do not require a CHANGELOG.md entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants