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-170090] Extend current logging to cache log requests #171

Closed cthomas-newrelic closed 9 months ago

cthomas-newrelic commented 9 months ago

This commit implements the storage mechanism for accumulated logs data. Log forwarding is implemented, but working at the moment, so is disabled.

codecov[bot] commented 9 months ago

Codecov Report

Attention: 236 lines in your changes are missing coverage. Please review.

Comparison is base (aa6c662) 53.61% compared to head (d918d4b) 54.89%. Report is 24 commits behind head on epic/nr92548-mobileLoggingSupport.

Files Patch % Lines
...ic/agent/android/harvest/HarvestConfiguration.java 21.53% 27 Missing and 24 partials :warning:
...om/newrelic/agent/android/logging/LogReporter.java 79.49% 32 Missing and 17 partials :warning:
...m/newrelic/agent/android/logging/LogReporting.java 63.95% 25 Missing and 6 partials :warning:
...m/newrelic/agent/android/logging/RemoteLogger.java 71.42% 19 Missing and 7 partials :warning:
...nt/android/instrumentation/LogInstrumentation.java 30.55% 22 Missing and 3 partials :warning:
...m/newrelic/agent/android/logging/LogForwarder.java 0.00% 22 Missing :warning:
...main/java/com/newrelic/agent/android/NewRelic.java 76.62% 14 Missing and 4 partials :warning:
.../com/newrelic/agent/android/harvest/Harvester.java 89.18% 3 Missing and 1 partial :warning:
.../java/com/newrelic/agent/android/util/Streams.java 89.74% 2 Missing and 2 partials :warning:
...in/java/com/newrelic/agent/android/SavedState.java 75.00% 3 Missing :warning:
... and 3 more
Additional details and impacted files ```diff @@ Coverage Diff @@ ## epic/nr92548-mobileLoggingSupport #171 +/- ## ======================================================================= + Coverage 53.61% 54.89% +1.28% - Complexity 2906 3094 +188 ======================================================================= Files 305 315 +10 Lines 15237 15929 +692 Branches 1643 1712 +69 ======================================================================= + Hits 8170 8745 +575 - Misses 6224 6289 +65 - Partials 843 895 +52 ```

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