Skip to content

Add integration tests of set account information in reliability module#2725

Merged
ambushwork merged 1 commit into
developfrom
yl/account-info/single-fit
Jun 17, 2025
Merged

Add integration tests of set account information in reliability module#2725
ambushwork merged 1 commit into
developfrom
yl/account-info/single-fit

Conversation

@ambushwork

Copy link
Copy Markdown
Member

What does this PR do?

Add integration tests of set account information in reliability module

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • Make sure you discussed the feature or bugfix with the maintaining team in an Issue
  • Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

@ambushwork
ambushwork force-pushed the yl/account-info/single-fit branch from 9350e22 to a39a513 Compare June 12, 2025 15:37
@codecov-commenter

codecov-commenter commented Jun 12, 2025

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 69.77%. Comparing base (87d7506) to head (4ef04d1).
Report is 9 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #2725      +/-   ##
===========================================
- Coverage    69.78%   69.77%   -0.00%     
===========================================
  Files          825      825              
  Lines        30760    30759       -1     
  Branches      5174     5173       -1     
===========================================
- Hits         21463    21462       -1     
+ Misses        7842     7840       -2     
- Partials      1455     1457       +2     

see 31 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ambushwork
ambushwork marked this pull request as ready for review June 13, 2025 06:48
@ambushwork
ambushwork requested review from a team as code owners June 13, 2025 06:48

@0xnm 0xnm left a comment

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.

lgtm, but we also need to add tests in the single-fit for RUM, Logs and Traces.

@ambushwork
ambushwork force-pushed the yl/account-info/single-fit branch 4 times, most recently from 197b2c4 to df2c69e Compare June 13, 2025 15:45
0xnm
0xnm previously approved these changes Jun 16, 2025

@0xnm 0xnm left a comment

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.

thank you!

@ambushwork
ambushwork merged commit 5556aff into develop Jun 17, 2025
25 checks passed
@ambushwork
ambushwork deleted the yl/account-info/single-fit branch June 17, 2025 15:11
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