Skip to content

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
no/remove-system-test-supportfrom
no/integration-test-docs
Draft

runningcode wants to merge 2 commits into
no/remove-system-test-supportfrom
no/integration-test-docs

Conversation

@runningcode

@runningcode runningcode commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor

PR Stack (Sample Tests)


📜 Description

Updates the docs that described the Python system tests:

  • .cursor/rules/e2e_tests.mdc keeps its name, which AGENTS.md refers to, and now describes:
    • MockSentryServer and the envelope matchers;
    • where each kind of end-to-end test lives;
    • the Spring Boot suites per flavor, and why a flavor is its own suite;
    • the embedded-Tomcat suites for plain Spring;
    • the OpenTelemetry agent fixture;
    • how CI and integrationTests run them;
    • the gotchas the stack ran into: the instrumented TestRestTemplate, the 5-second span batches, one live context per JVM, the profiler's dot check, and the embedded Kafka broker.
  • The test skill 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.mdc no longer asks for sample system tests. End-to-end coverage goes in the module the sample uses.
  • AGENTS.md drops sentry-system-test-support, and its rule index and testing requirements describe the in-module tests.

After a coverage audit against the sample's tests:

  • The rule also lists the integrationTestLog4j2 suite. Its test-client gotcha now covers Sentry customizing the RestTemplateBuilder, 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?

  • No Markdown, rule or skill file outside CHANGELOG.md mentions the system tests, the runner or sentry-system-test-support any more.

📝 Checklist

  • I added GH Issue ID & Linear ID
  • I added tests to verify the changes.
  • No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled.
  • I updated the docs if needed.
  • I updated the wizard if needed.
  • Review from the native team if needed.
  • No breaking change or entry added to the changelog.
  • No breaking change for hybrid SDKs or communicated to hybrid SDKs.
  • Public API changes reviewed by another Mobile SDK team member or implemented according to the develop docs spec.

🔮 Next steps

This is the last PR in the stack.

#skip-changelog

🤖 Generated with Claude Code

This was referenced Oct 20, 2026
@sentry

sentry Bot commented Oct 20, 2026 •

Copy link
Copy Markdown

📲 Install Builds

Android

🔗 App Name App ID Version Configuration
SDK Size io.sentry.tests.size 8.61.0 (1) release

⚙️ sentry-android Build Distribution Settings

@runningcode
runningcode force-pushed the no/remove-system-test-support branch from 211d9a2 to d577084 Compare October 21, 2026 09:19
@runningcode
runningcode force-pushed the no/integration-test-docs branch from 543d6ee to 05d85a2 Compare October 21, 2026 09:19
runningcode and others added 2 commits October 21, 2026 13:22
…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
runningcode force-pushed the no/remove-system-test-support branch from d577084 to fa0cc04 Compare October 21, 2026 11:27
@runningcode
runningcode force-pushed the no/integration-test-docs branch from 05d85a2 to a900da8 Compare October 21, 2026 11:27

This branch has not been deployed

No deployments
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.

1 participant