Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@YvCeung
Copy link
Contributor

@YvCeung YvCeung commented Jun 20, 2025

  • I have registered the PR changes.

Ⅰ. Describe what this PR did

improve unit test coverage of seata-rm moudle

Ⅱ. Does this pull request fix one issue?

fixes #7382

Ⅲ. Why don't you add test cases (unit test/integration test)?

Ⅳ. Describe how to verify it

mvn clean test

Ⅴ. Special notes for reviews

2.x:
image
now:
image

@codecov
Copy link

codecov bot commented Jun 21, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 60.14%. Comparing base (ac9c8ba) to head (cd29f13).
Report is 1 commits behind head on 2.x.

Additional details and impacted files
@@             Coverage Diff              @@
##                2.x    #7457      +/-   ##
============================================
+ Coverage     60.01%   60.14%   +0.12%     
  Complexity      648      648              
============================================
  Files          1281     1281              
  Lines         46280    46280              
  Branches       5580     5580              
============================================
+ Hits          27773    27833      +60     
+ Misses        15930    15863      -67     
- Partials       2577     2584       +7     

see 14 files with indirect coverage changes

Impacted file tree graph

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Contributor

@funky-eyes funky-eyes left a comment

Choose a reason for hiding this comment

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

LGTM

@funky-eyes funky-eyes changed the title test:improve unit test coverage of seata-rm moudle test: improve unit test coverage of seata-rm moudle Jun 21, 2025
@funky-eyes funky-eyes added this to the 2.5.0 milestone Jun 21, 2025
@funky-eyes funky-eyes added module/rm rm module type: test test case labels Jun 21, 2025
@funky-eyes funky-eyes merged commit 9432f72 into apache:2.x Jun 21, 2025
9 checks passed
slievrly pushed a commit to slievrly/fescar that referenced this pull request Oct 21, 2025
YvCeung added a commit to YvCeung/incubator-seata that referenced this pull request Dec 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

module/rm rm module type: test test case

Projects

None yet

Development

Successfully merging this pull request may close these issues.

add UT for rm module

2 participants