incubator-seata icon indicating copy to clipboard operation
incubator-seata copied to clipboard

fix(6502): supplement the store unit test cases in the [core] module.

Open llzcx opened this issue 9 months ago • 3 comments

  • [ ] I have registered the PR changes.

Ⅰ. Describe what this PR did

  1. supplementary SqlServerLockStoreSql unit testing
  2. supplementary DistributedLockSqlFactory、BaseDistributedLockSqlServer、BaseDistributedLockSql unit testing

Ⅱ. Does this pull request fix one issue?

fixes #6502

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

Ⅳ. Describe how to verify it

Ⅴ. Special notes for reviews

llzcx avatar May 07 '24 14:05 llzcx

Finished or in progress? If possible, increase the overall coverage of the core package

lightClouds917 avatar May 25 '24 11:05 lightClouds917

add the change log please,You can refer to others

lightClouds917 avatar May 25 '24 12:05 lightClouds917

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 50.51%. Comparing base (7d3cf99) to head (b735252).

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##                2.x    #6524      +/-   ##
============================================
+ Coverage     50.47%   50.51%   +0.03%     
- Complexity     5831     5841      +10     
============================================
  Files          1051     1051              
  Lines         36340    36340              
  Branches       4319     4319              
============================================
+ Hits          18342    18356      +14     
+ Misses        16183    16167      -16     
- Partials       1815     1817       +2     

see 5 files with indirect coverage changes

codecov[bot] avatar Jul 29 '24 14:07 codecov[bot]