Skip to content

chore: Add integration tests for metrics#1226

Merged
tustanivsky merged 8 commits intomainfrom
chore/metrics-e2e-tests
Feb 16, 2026
Merged

chore: Add integration tests for metrics#1226
tustanivsky merged 8 commits intomainfrom
chore/metrics-e2e-tests

Conversation

@tustanivsky
Copy link
Copy Markdown
Collaborator

@tustanivsky tustanivsky commented Feb 11, 2026

Key changes:

  • Add metric-capture test scenario to the sample app that emits counter, distribution and gauge metrics with a shared test_id attribute
  • Add CppBeforeMetricHandler to the sample app that adds and removes attributes on metrics before they are sent
  • Add metrics integration tests for Desktop (Windows/Linux) and Android that verify:
    • All three metric types arrive in Sentry with correct name, type, and value
    • BeforeMetricHandler correctly adds new attributes and removes existing ones
  • Skip metrics tests on macOS since Apple platforms currently don't support custom metrics
  • Bump app-runner to include Get-SentryMetrics, Get-SentryMetricsByAttribute, and Get-SentryTestMetric helpers that query the tracemetrics dataset
  • Remove redundant BeforeMetricHandler unit tests

Depends on:

Related items:

#skip-changelog

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Feb 11, 2026

Semver Impact of This PR

🟢 Patch (bug fixes)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


This PR will not appear in the changelog.


🤖 This preview updates automatically when you update the PR.

@tustanivsky tustanivsky marked this pull request as ready for review February 11, 2026 19:53
# Conflicts:
#	scripts/packaging/package.snapshot
@tustanivsky tustanivsky merged commit 46f0715 into main Feb 16, 2026
95 of 98 checks passed
@tustanivsky tustanivsky deleted the chore/metrics-e2e-tests branch February 16, 2026 06:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants