newrelic / newrelic-android-agent

SDK to enable instrumentation of Android mobile apps in New Relic
Apache License 2.0
14 stars 14 forks source link

[NR-247776] Report AEI all reasons #209

Closed ywang-nr closed 6 months ago

ywang-nr commented 6 months ago

What's changed:

  1. We would like to report all reasons rather than just ANR reason
  2. Modify existing unit tests
codecov[bot] commented 6 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 54.20%. Comparing base (67a46b4) to head (96ebe44).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## feature/nr230991-appExitInfo #209 +/- ## ================================================================== - Coverage 54.22% 54.20% -0.03% + Complexity 2970 2969 -1 ================================================================== Files 309 309 Lines 15543 15538 -5 Branches 1694 1692 -2 ================================================================== - Hits 8428 8422 -6 - Misses 6229 6231 +2 + Partials 886 885 -1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

cthomas-newrelic commented 6 months ago

Oops, just noticed this PR was a draft.

ywang-nr commented 6 months ago

Oops, just noticed this PR was a draft.

Yeah, I was looking into whether we can add more unit tests for other reasons.