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

bugfix: ensure environment is fully loaded before ConfigurationFactory initialization

Open WangzJi opened this issue 2 months ago • 2 comments

fixes: #7641

Ⅰ. Describe what this PR did

Ⅱ. Does this pull request fix one issue?

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

Ⅳ. Describe how to verify it

Ⅴ. Special notes for reviews

WangzJi avatar Oct 17 '25 02:10 WangzJi

Codecov Report

:white_check_mark: All modified and coverable lines are covered by tests. :white_check_mark: Project coverage is 61.42%. Comparing base (d239a81) to head (34e4cdb).

Additional details and impacted files
@@             Coverage Diff              @@
##                2.x    #7705      +/-   ##
============================================
+ Coverage     61.38%   61.42%   +0.03%     
- Complexity      684      688       +4     
============================================
  Files          1323     1323              
  Lines         49943    49941       -2     
  Branches       5885     5884       -1     
============================================
+ Hits          30657    30675      +18     
+ Misses        16524    16508      -16     
+ Partials       2762     2758       -4     
Files with missing lines Coverage Δ
...ot/autoconfigure/loader/SeataPropertiesLoader.java 63.88% <100.00%> (+0.73%) :arrow_up:

... and 12 files with indirect coverage changes

Impacted file tree graph

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

codecov[bot] avatar Oct 17 '25 03:10 codecov[bot]

This branch has conflicts that must be resolved, and maybe use spotless:apply before merged is a good choice.

psxjoy avatar Oct 19 '25 16:10 psxjoy