Skip to content

Improve UT coverage for rum, logs and trace features#2299

Merged
xgouchet merged 1 commit into
developfrom
xgouchet/reliability/coverage_rum
Oct 2, 2024
Merged

Improve UT coverage for rum, logs and trace features#2299
xgouchet merged 1 commit into
developfrom
xgouchet/reliability/coverage_rum

Conversation

@xgouchet

@xgouchet xgouchet commented Oct 1, 2024

Copy link
Copy Markdown
Contributor

No description provided.

@xgouchet
xgouchet requested review from a team as code owners October 1, 2024 09:45
@codecov-commenter

codecov-commenter commented Oct 1, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 70.36%. Comparing base (21025fd) to head (8ed116a).
Report is 2 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #2299      +/-   ##
===========================================
- Coverage    70.44%   70.36%   -0.07%     
===========================================
  Files          730      730              
  Lines        27268    27264       -4     
  Branches      4584     4583       -1     
===========================================
- Hits         19207    19184      -23     
+ Misses        6815     6805      -10     
- Partials      1246     1275      +29     
Files with missing lines Coverage Δ
...otlin/com/datadog/android/rum/_RumInternalProxy.kt 78.57% <ø> (ø)

... and 35 files with indirect coverage changes

}

@Test
fun `builder can disable the bundle with rum feature`() {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

why aren't we using the new test definitions pattern here ? Must...When

@xgouchet
xgouchet force-pushed the xgouchet/reliability/coverage_rum branch from 4160691 to 8ed116a Compare October 2, 2024 10:37
@xgouchet
xgouchet merged commit 7fcb18e into develop Oct 2, 2024
@xgouchet
xgouchet deleted the xgouchet/reliability/coverage_rum branch October 2, 2024 11:26
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.

4 participants