Skip to content

test: [Sample Tests 38] Remove the system-test runner and its Python mock server - #6330

Draft
runningcode wants to merge 2 commits into
no/boot4-otlp-integration-testfrom
no/remove-system-test-runner
Draft

runningcode wants to merge 2 commits into
no/boot4-otlp-integration-testfrom
no/remove-system-test-runner

Conversation

@runningcode

@runningcode runningcode commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor

PR Stack (Sample Tests)


📜 Description

Every sample's system test now lives in the module it tests, so this removes what ran them:

  • test/system-test-runner.py and test/system-test-sentry-server.py, the Python runner and its mock Sentry server on :8000;
  • requirements.txt, which only the runner used;
  • the Makefile's setupPython, systemTest and systemTestInteractive targets;
  • .github/workflows/system-tests-backend.yml.

devenv, .envrc and pyproject.toml stay. They set up a general-purpose .venv, and pyproject.toml declares no dependencies.

This branch merges #6268's. #6268 moves the last system test, the logback sample's, into sentry-logback. It sits beside the rest of the stack rather than in it: both it and #6269 are based on #6267. The conflicts were in the two files this PR deletes. Until #6268 lands, its changes show up in this PR's diff.

The Gradle side, the io.sentry.systemtest plugin and sentry-system-test-support, goes in the next PR.

💡 Motivation and Context

The end of the migration: nothing runs the Python flow any more. The three Spring Boot matrices stopped using it in #6319, #6320 and #6321.

💚 How did you test it?

  • No sample has a test under io.sentry.systemtest or applies io.sentry.systemtest.
  • Nothing else references the deleted files: no workflow, CODEOWNERS, file filter or Dependabot config.
  • After the merge, SentryAppenderIntegrationTest passes, and the logback sample still builds.

📝 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

Delete the io.sentry.systemtest Gradle plugin and sentry-system-test-support, then update the docs.

#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/boot4-otlp-integration-test branch from 64a5df4 to cecf1e0 Compare October 21, 2026 09:19
@runningcode
runningcode force-pushed the no/remove-system-test-runner branch from 602df22 to 0e2839f Compare October 21, 2026 09:19
runningcode and others added 2 commits October 21, 2026 13:22
The test now configures the appender from XML in-process and sends to
MockSentryServer, so it runs as part of :sentry-logback:test instead of
needing the Python runner and a shadow JAR. It also checks that the XML
contextTag reaches the event.

The logback sample has no tests left, so its system-test wiring, CI
matrix entry and runner entry are removed.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Every sample's system test now lives in the module it tests, so the
Python runner, its mock Sentry server, requirements.txt, the Makefile
targets that ran them, and the backend workflow that ran them in CI
have nothing left to run.

This branch merges #6268's, which moves the last one, the logback
sample's, into sentry-logback.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@runningcode
runningcode force-pushed the no/boot4-otlp-integration-test branch from cecf1e0 to c8fc446 Compare October 21, 2026 11:27
@runningcode
runningcode force-pushed the no/remove-system-test-runner branch from 0e2839f to 6ba410a 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