Skip to content

Update sample app to test new Sentry features#270

Merged
wzieba merged 3 commits intotrunkfrom
sample_app_changes
Aug 7, 2025
Merged

Update sample app to test new Sentry features#270
wzieba merged 3 commits intotrunkfrom
sample_app_changes

Conversation

@wzieba
Copy link
Member

@wzieba wzieba commented Aug 5, 2025

Description

This PR adds a few changes to the sample app that allow to test #269

Testing

Before tests cp gradle.properties-example gradle.properties and set a valid sentryTestProjectDSN (tracks-demo-android project on A8c Sentry.

isAttachAnrThreadDump

  1. Install the app in release
  2. Tap "Force ANR"
  3. See a new Sentry issue reported. I couldn't find any additional data regarding ANR thread dump, but that's likely it's a very simple stacktrace from the sample app - I guess in production apps, in more complex scenarios we'll have some data on this.

isStartProfilerOnAppStart

  1. Go to https://a8c.sentry.io/profiling/?project=6626116&statsPeriod=24h&tab=flamegraph
  2. In the bottom right corner see the last id of a profile
  3. Run the sample app
  4. Wait a few seconds, refresh the UI a few times
  5. See that there's a new profile available.

Image

image
Screenshot 2025-08-05 at 13 39 53

☝️ a sampling trace from a test method I've added (it waited 3s on the main thread).

wzieba added 2 commits August 5, 2025 14:52
And run in in `release` variant to test `attachAnrThreadDump`
@wzieba wzieba marked this pull request as ready for review August 5, 2025 15:16
@wzieba wzieba changed the title Sample app changes Update sample app to test new Sentry features Aug 5, 2025
@wzieba wzieba requested a review from ParaskP7 August 5, 2025 15:16
Copy link
Contributor

@ParaskP7 ParaskP7 left a comment

Choose a reason for hiding this comment

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

👋 @wzieba !

I have reviewed and tested this PR as per the instructions, everything works as expected, thanks for the thorough testing instructions and TIL on tracks-demo-android, this makes testing so much easier for me! 🧪 x 🧪 ^ 🧪

isAttachAnrThreadDump
...
3. See a new Sentry issue reported. I couldn't find any additional data regarding ANR thread dump, but that's likely it's a very simple stacktrace from the sample app - I guess in production apps, in more complex scenarios we'll have some data on this.

👍

isStartProfilerOnAppStart
...
5. See that there's a new profile available.

👌

Base automatically changed from update_dependencies to trunk August 7, 2025 11:38
@wzieba wzieba enabled auto-merge August 7, 2025 11:38
@wzieba wzieba merged commit ec9af8e into trunk Aug 7, 2025
9 of 11 checks passed
@wzieba wzieba deleted the sample_app_changes branch August 7, 2025 11:50
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.

2 participants