Skip to content

[TEST] fill up alert group service test #15777

New issue

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

By clicking “No 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? No Sign in to your account

Merged
merged 2 commits into from
Mar 29, 2024

Conversation

abzymeinsjtu
Copy link

Purpose of the pull request

add test code to cover all alert group implement

@abzymeinsjtu abzymeinsjtu force-pushed the feat_fillup_alert_group_svc_test branch from 1422530 to a6ffef2 Compare March 28, 2024 14:30
@SbloodyS SbloodyS added improvement make more easy to user or prompt friendly test labels Mar 29, 2024
@SbloodyS SbloodyS added this to the 3.3.0 milestone Mar 29, 2024
@SbloodyS SbloodyS added the 3.3.0 label Mar 29, 2024
@abzymeinsjtu abzymeinsjtu force-pushed the feat_fillup_alert_group_svc_test branch from a6ffef2 to 3d52e86 Compare March 29, 2024 03:40
@github-actions github-actions bot removed the test label Mar 29, 2024
@abzymeinsjtu abzymeinsjtu force-pushed the feat_fillup_alert_group_svc_test branch from dee5e33 to 35ec16d Compare March 29, 2024 03:42
rickchengx
rickchengx previously approved these changes Mar 29, 2024
Copy link
Contributor

@rickchengx rickchengx left a comment

Choose a reason for hiding this comment

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

LGTM

EricGao888
EricGao888 previously approved these changes Mar 29, 2024
SbloodyS
SbloodyS previously approved these changes Mar 29, 2024
Copy link
Member

@SbloodyS SbloodyS left a comment

Choose a reason for hiding this comment

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

+1

@abzymeinsjtu abzymeinsjtu dismissed stale reviews from SbloodyS, EricGao888, and rickchengx via dd30d16 March 29, 2024 04:03
@abzymeinsjtu abzymeinsjtu force-pushed the feat_fillup_alert_group_svc_test branch from da29528 to dd30d16 Compare March 29, 2024 04:03
@codecov-commenter
Copy link

codecov-commenter commented Mar 29, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 39.15%. Comparing base (723126b) to head (f6e96d8).

❗ Current head f6e96d8 differs from pull request most recent head 88a25ea. Consider uploading reports for the commit 88a25ea to get more accurate results

Additional details and impacted files
@@             Coverage Diff              @@
##                dev   #15777      +/-   ##
============================================
+ Coverage     39.10%   39.15%   +0.05%     
- Complexity     4852     4862      +10     
============================================
  Files          1316     1316              
  Lines         44945    44945              
  Branches       4800     4800              
============================================
+ Hits          17574    17599      +25     
+ Misses        25479    25463      -16     
+ Partials       1892     1883       -9     

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

Copy link

Quality Gate Passed Quality Gate passed

Issues
2 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@rickchengx rickchengx merged commit ae1fe84 into apache:dev Mar 29, 2024
60 checks passed
@abzymeinsjtu abzymeinsjtu deleted the feat_fillup_alert_group_svc_test branch March 29, 2024 05:34
No Sign up for free to join this conversation on GitHub. Already have an account? No Sign in to comment
Labels
3.3.0 backend improvement make more easy to user or prompt friendly ready-to-merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants