Skip to content

feat(tests): support async tests with Telemetry Processor#1055

Merged
solnic merged 2 commits into
masterfrom
feat/tests-support-async-tests-with-tp
May 20, 2026
Merged

feat(tests): support async tests with Telemetry Processor#1055
solnic merged 2 commits into
masterfrom
feat/tests-support-async-tests-with-tp

Conversation

@solnic
Copy link
Copy Markdown
Collaborator

@solnic solnic commented May 13, 2026

Reworks Sentry.Test process ownership model further with NimbleOwnership and makes TelemetryProcessor work in async tests.

@solnic solnic force-pushed the feat/tests-support-async-tests-with-tp branch 3 times, most recently from c2e2253 to 5c821f9 Compare May 14, 2026 10:44
@solnic solnic marked this pull request as ready for review May 14, 2026 10:47
Comment thread lib/sentry/telemetry_processor.ex
Comment thread lib/sentry/test.ex Outdated
Comment thread lib/sentry/test/registry.ex Outdated
cursor[bot]

This comment was marked as resolved.

@solnic solnic marked this pull request as draft May 15, 2026 06:56
@solnic solnic force-pushed the feat/tests-support-async-tests-with-tp branch from 62e1118 to 8d4ffae Compare May 15, 2026 07:41
@solnic solnic changed the title feat(tests): support async tests with TP feat(tests): support async tests with Telemetry Processor May 15, 2026
@solnic solnic force-pushed the feat/tests-support-async-tests-with-tp branch 3 times, most recently from a0780a2 to 9f7c9b8 Compare May 18, 2026 07:37
- Use `NimbleOwnership` to simplify ownership handling
- Handle per-test telemetry processors correctly

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@solnic solnic force-pushed the feat/tests-support-async-tests-with-tp branch from 9f7c9b8 to d18a6d5 Compare May 18, 2026 10:39
@solnic solnic marked this pull request as ready for review May 18, 2026 11:11
@solnic solnic merged commit 02a5f39 into master May 20, 2026
12 checks passed
@solnic solnic deleted the feat/tests-support-async-tests-with-tp branch May 20, 2026 11:50
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