Skip to content

build: Stop logging passed tests and test output to the console - #6554

Merged
runningcode merged 1 commit into
mainfrom
no/build-remove-passed-test-logging
Nov 11, 2026
Merged

build: Stop logging passed tests and test output to the console#6554
runningcode merged 1 commit into
mainfrom
no/build-remove-passed-test-logging

Conversation

@runningcode

@runningcode runningcode commented Nov 11, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

📜 Description

Drop test PASSED events because they are noisy. We also don't want stdout from passed test. It blows up our logs.

This is a logging only change.

📝 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

🤖 Generated with Claude Code

#skip-changelog

@runningcode @claude
build: Stop logging passed tests and test output to the console
2fb9ffc 
Every passing test printed a PASSED line plus its stdout/stderr, which
buried the failures in CI logs. The console now shows only skipped and
failed tests. Test output is still in the HTML and XML test reports.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@runningcode
runningcode marked this pull request as ready for review November 11, 2026 10:22
@runningcode
runningcode requested review from a team and adinauer as code owners November 11, 2026 10:22
@runningcode
runningcode requested review from markushi and romtsn and removed request for a team November 11, 2026 10:22
@runningcode runningcode added the ship-it PR is ready to merge from a reviewer perspective label Nov 11, 2026
@sentry

sentry Bot commented Nov 11, 2026

Copy link
Copy Markdown

📲 Install Builds

Android

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

⚙️ sentry-android Build Distribution Settings

0xadam-brown
0xadam-brown approved these changes Nov 11, 2026
View reviewed changes

@0xadam-brown 0xadam-brown left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes!

@runningcode
runningcode merged commit 0fb4971 into main Nov 11, 2026
50 checks passed
@runningcode
runningcode deleted the no/build-remove-passed-test-logging branch November 11, 2026 12:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@0xadam-brown 0xadam-brown 0xadam-brown approved these changes
@adinauer adinauer Awaiting requested review from adinauer adinauer is a code owner
@markushi markushi Awaiting requested review from markushi markushi is a code owner automatically assigned from getsentry/team-mobile-android-sdk
@romtsn romtsn Awaiting requested review from romtsn romtsn is a code owner automatically assigned from getsentry/team-mobile-android-sdk

Assignees

No one assigned

Labels

ship-it PR is ready to merge from a reviewer perspective

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@runningcode @0xadam-brown