Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Develop merge into main #297

Open
wants to merge 28 commits into
base: main
Choose a base branch
from
Open

Develop merge into main #297

wants to merge 28 commits into from

Conversation

ndesai-newrelic
Copy link
Contributor

No description provided.

@ndesai-newrelic ndesai-newrelic changed the title Develop Develop merge into main Oct 21, 2024
Copy link

codecov bot commented Oct 21, 2024

Codecov Report

Attention: Patch coverage is 70.29361% with 172 lines in your changes missing coverage. Please review.

Project coverage is 57.53%. Comparing base (89838e5) to head (45c3f97).

Files with missing lines Patch % Lines
...m/newrelic/agent/android/aei/AEITraceReporter.java 48.14% 49 Missing and 21 partials ⚠️
...elic/agent/android/aei/ApplicationExitMonitor.java 86.24% 12 Missing and 14 partials ⚠️
...m/newrelic/agent/android/aei/AEISessionMapper.java 61.40% 12 Missing and 10 partials ⚠️
...relic/agent/android/payload/FileBackedPayload.java 60.78% 13 Missing and 7 partials ⚠️
.../java/com/newrelic/agent/android/aei/AEITrace.java 80.00% 6 Missing and 2 partials ⚠️
...m/newrelic/agent/android/logging/LogReporting.java 33.33% 6 Missing and 2 partials ⚠️
...com/newrelic/agent/android/aei/AEITraceSender.java 87.23% 5 Missing and 1 partial ⚠️
...ent/android/analytics/AnalyticsControllerImpl.java 50.00% 2 Missing and 2 partials ⚠️
.../newrelic/agent/android/payload/PayloadSender.java 76.47% 2 Missing and 2 partials ⚠️
...relic/agent/android/payload/PayloadController.java 71.42% 0 Missing and 2 partials ⚠️
... and 2 more
Additional details and impacted files
@@             Coverage Diff              @@
##               main     #297      +/-   ##
============================================
+ Coverage     57.40%   57.53%   +0.12%     
- Complexity     3349     3445      +96     
============================================
  Files           325      330       +5     
  Lines         16646    17031     +385     
  Branches       1820     1865      +45     
============================================
+ Hits           9555     9798     +243     
- Misses         6076     6168      +92     
- Partials       1015     1065      +50     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.


🚨 Try these New Features:

Copy link
Contributor

@cthomas-newrelic cthomas-newrelic left a comment

Choose a reason for hiding this comment

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

Tihs would be the third such merge since release and I'm afraid there is a synchronization issue between the 2 branches. Give it a go, let's see.

BTW, that action no longer works as it once did, so there's no reason to merge that (yet).

* adds new attribute `aeiSessionId` to AEI event when PID of aei is found
in session mapper cache.
* adds/updates tests
The set of AEI records returned by ART changes over time. Update the on-device
storage to remove artifacts no longer raported by ART.
@cthomas-newrelic
Copy link
Contributor

Develop has changed behind this commit, so don't commit to main.

This chnage is based on assumptions on an incubating spec.
It may be reverted or modified.
To be used in AEI trace reporting. Logging will be refactored to use this within it's LogForwarder.

* adds FileBackedPayload pulled up from LogForwarder
* adds compression to payload data
* adds tests
* parses system trace provided by AEI record into thread collection
* formats thread data as a String according to ingest spec
Since we've been sing the new vpn, some tests fail due to odd
networking exceptions in Mockito.

* DRY's up connection mocking code
* refactor ApplicationExitMonitorTest
* adds `fullLifecycle` for the complex sessionMapper interactions
* update AEITraceSender tests
* exposed FileBackedPayload `compress()` methods, updates tests
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.

3 participants