Repository navigation
test(async-profiler): [Sample Tests 33] Add an end-to-end test of a profiled transaction - #6325
Draft
runningcode wants to merge 2 commits into
Draft
runningcode wants to merge 2 commits into
runningcode wants to merge 2 commits into
Conversation
ContinuousProfilerEndToEndTest starts a sampled transaction with the TRACE profile lifecycle and checks that its profile chunk reaches MockSentryServer, linked to the transaction by the profiler ID. It ports the profiling part of the console sample's system test. The profiler rejects a traces path with a dot anywhere in its absolute form, so the test puts it in the temp dir. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
This was referenced Oct 20, 2026
Draft
Draft
Draft
Draft
Draft
This was referenced Oct 20, 2026
📲 Install BuildsAndroid
|
This was referenced Oct 20, 2026
Draft
Draft
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
PR Stack (Sample Tests)
main)📜 Description
Adds
ContinuousProfilerEndToEndTesttosentry-async-profiler. It ports the profiling part ofsentry-samples-console's system test. A sampled transaction with theTRACEprofile lifecycle starts the async profiler, and the test checks that the profile chunk reachesMockSentryServerwith the transaction's profiler ID.The traces path is in the temp dir. The profiler rejects a traces path with a
.anywhere in its absolute form, as #6319 found, so a relative path fails in a checkout whose path has a dot in it.💡 Motivation and Context
sentry-async-profiler's tests mock the transport and the profiler's surroundings. This one runs the real profiler and sends the chunk over HTTP, in under a second.💚 How did you test it?
JavaContinuousProfilerTest's tests fail with or without this PR. The path contains/.claude/, and the same dot check disables profiling: one test expects,as the rejected character and gets.. In CI, whose checkout path has no dot, they pass.📝 Checklist
sendDefaultPIIis enabled.🔮 Next steps
The console sample's Kafka round trip into
sentry-kafka, which retires the sample.#skip-changelog
🤖 Generated with Claude Code