Skip to content

Conversation

@slievrly
Copy link
Member

@slievrly slievrly commented Jul 16, 2025

  • I have registered the PR changes.

Ⅰ. Describe what this PR did

fix jakarta UT failed in jdk17+

Ⅱ. Does this pull request fix one issue?

fix #7516

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

Ⅳ. Describe how to verify it

Ⅴ. Special notes for reviews

@slievrly slievrly modified the milestones: 2.5.0, 2.6.0 Jul 16, 2025
@codecov
Copy link

codecov bot commented Jul 16, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 60.48%. Comparing base (d481d55) to head (59ec826).
Report is 1 commits behind head on 2.x.

Additional details and impacted files
@@             Coverage Diff              @@
##                2.x    #7541      +/-   ##
============================================
- Coverage     60.53%   60.48%   -0.06%     
- Complexity      656      658       +2     
============================================
  Files          1298     1298              
  Lines         49081    49081              
  Branches       5771     5771              
============================================
- Hits          29713    29686      -27     
- Misses        16734    16764      +30     
+ Partials       2634     2631       -3     

see 10 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
Member

@xingfudeshi xingfudeshi left a comment

Choose a reason for hiding this comment

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

LGTM

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 merged commit 88bba2f into apache:2.x Jul 17, 2025
10 checks passed
@funky-eyes funky-eyes added the type: test test case label Jul 17, 2025
slievrly added a commit to slievrly/fescar that referenced this pull request Oct 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: test test case

Projects

None yet

Development

Successfully merging this pull request may close these issues.

some UT failed in SeataSagaAutoConfigurationTest

3 participants