Skip to content

test: [Sample Tests 39] Remove the io.sentry.systemtest plugin and sentry-system-test-support - #6331

Draft
runningcode wants to merge 1 commit into
no/remove-system-test-runnerfrom
no/remove-system-test-support
Draft

runningcode wants to merge 1 commit into
no/remove-system-test-runnerfrom
no/remove-system-test-support

Conversation

@runningcode

@runningcode runningcode commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor

PR Stack (Sample Tests)


📜 Description

With the runner gone in #6330, nothing uses the Gradle side of the system tests, so this removes it:

  • the io.sentry.systemtest convention plugin and its SystemTestExtension, which tracked the app archive and agent jar as systemTest inputs;
  • sentry-system-test-support, which held the HTTP and Apollo GraphQL clients and the TestHelper the system tests used. The envelope matchers it delegated to since test: [Sample Tests 2] Share the system-test envelope matchers #6267 stay in sentry-test-support;
  • the module's entries in settings.gradle.kts, the API validation ignore list and the publishing exclusion in the root build.

💡 Motivation and Context

One less module to build, and no Apollo 4 code generation for test-only code.

💚 How did you test it?

  • No build file or source references io.sentry.systemtest, SystemTestExtension or sentry-system-test-support.
  • ./gradlew spotlessApply apiDump configures every project and passes.
  • :sentry-test-support:check and :sentry-logback:test pass, and a Boot 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

Update the docs that describe the system tests.

#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
…upport

With the runner gone, nothing applies the io.sentry.systemtest
convention plugin or depends on sentry-system-test-support, which held
the HTTP and GraphQL clients and the TestHelper the system tests used.
The envelope matchers it delegated to stay in sentry-test-support.

Co-Authored-By: Claude Opus 5.5 (1M context) <[email protected]>
@runningcode
runningcode force-pushed the no/remove-system-test-support branch from d577084 to fa0cc04 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