newrelic / newrelic-android-agent

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

[NR-169897] Integrate remote Log Reporting configuration model #219

Closed cthomas-newrelic closed 4 months ago

cthomas-newrelic commented 4 months ago

Builds off same work done for ApplicationExitInfo feature. Configuration data is returned in connect response configuration element. Adds/updates tests.

NR-169897 Add parsing annotations to log level enums

The agent spec indicates the log level will be returned as a lower-case string, rather than the name of the enum (uppercase). Add Gson annotations to look for both.

codecov[bot] commented 4 months ago

Codecov Report

Attention: Patch coverage is 66.66667% with 26 lines in your changes are missing coverage. Please review.

Project coverage is 55.82%. Comparing base (961a2d1) to head (f566db6). Report is 1 commits behind head on epic/nr92548-mobileLoggingSupport.

Files Patch % Lines
...ent/android/logging/LogReportingConfiguration.java 33.33% 1 Missing and 5 partials :warning:
...com/newrelic/agent/android/AgentConfiguration.java 54.54% 5 Missing :warning:
...ic/agent/android/ApplicationExitConfiguration.java 54.54% 2 Missing and 3 partials :warning:
...om/newrelic/agent/android/RemoteConfiguration.java 75.00% 1 Missing and 3 partials :warning:
...a/com/newrelic/agent/android/AndroidAgentImpl.java 0.00% 2 Missing and 1 partial :warning:
...ic/agent/android/harvest/HarvestConfiguration.java 71.42% 1 Missing and 1 partial :warning:
...in/java/com/newrelic/agent/android/SavedState.java 87.50% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## epic/nr92548-mobileLoggingSupport #219 +/- ## ======================================================================= - Coverage 55.84% 55.82% -0.03% - Complexity 3207 3221 +14 ======================================================================= Files 321 323 +2 Lines 16373 16415 +42 Branches 1781 1789 +8 ======================================================================= + Hits 9143 9163 +20 - Misses 6271 6283 +12 - Partials 959 969 +10 ```

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