Repository navigation
docs: [Sample Tests 40] Describe the in-module end-to-end tests instead of the system tests - #6332
Draft
runningcode wants to merge 2 commits into
Draft
runningcode wants to merge 2 commits into
runningcode wants to merge 2 commits into
Conversation
This was referenced Oct 20, 2026
Draft
Draft
Draft
Draft
Draft
This was referenced Oct 20, 2026
Draft
Draft
Draft
📲 Install BuildsAndroid
|
runningcode
force-pushed
the
no/remove-system-test-support
branch
from
October 21, 2026 09:19
211d9a2 to
d577084
Compare
runningcode
force-pushed
the
no/integration-test-docs
branch
from
October 21, 2026 09:19
543d6ee to
05d85a2
Compare
…ests The e2e_tests rule now covers MockSentryServer, where each kind of end-to-end test lives, the Spring suites per flavor, the OpenTelemetry agent fixture, how to run them and the gotchas the migration found. The test skill runs integration suites by their source set instead of the Python runner, the new_module rule no longer asks for sample tests, and AGENTS.md drops sentry-system-test-support. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…client and log4j2 suite The context's TestRestTemplate adds trace headers of its own whenever Sentry customizes the RestTemplateBuilder, not only in OpenTelemetry mode, and the Spring Boot modules have an integrationTestLog4j2 suite. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
runningcode
force-pushed
the
no/remove-system-test-support
branch
from
October 21, 2026 11:27
d577084 to
fa0cc04
Compare
runningcode
force-pushed
the
no/integration-test-docs
branch
from
October 21, 2026 11:27
05d85a2 to
a900da8
Compare
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
Updates the docs that described the Python system tests:
.cursor/rules/e2e_tests.mdckeeps its name, whichAGENTS.mdrefers to, and now describes:MockSentryServerand the envelope matchers;integrationTestsrun them;TestRestTemplate, the 5-second span batches, one live context per JVM, the profiler's dot check, and the embedded Kafka broker.testskill runs an integration suite by its source set, including both agent tasks, instead of the Python runner. For a sample, it says samples have no tests..cursor/rules/new_module.mdcno longer asks for sample system tests. End-to-end coverage goes in the module the sample uses.AGENTS.mddropssentry-system-test-support, and its rule index and testing requirements describe the in-module tests.After a coverage audit against the sample's tests:
integrationTestLog4j2suite. Its test-client gotcha now covers Sentry customizing theRestTemplateBuilder, not only OpenTelemetry instrumenting it.💡 Motivation and Context
So that agents and contributors writing tests follow the new setup rather than the deleted runner.
💚 How did you test it?
CHANGELOG.mdmentions the system tests, the runner orsentry-system-test-supportany more.📝 Checklist
sendDefaultPIIis enabled.🔮 Next steps
This is the last PR in the stack.
#skip-changelog
🤖 Generated with Claude Code