-
Notifications
You must be signed in to change notification settings - Fork 8.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “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? Sign in to your account
bugfix:Fixed the issue of failing to parse annotations in TCC mode when the business object is a proxy object. #7107
base: 2.x
Are you sure you want to change the base?
Conversation
spring/src/test/java/org/apache/seata/spring/tcc/EasyTccAction.java
Outdated
Show resolved
Hide resolved
spring/src/test/java/org/apache/seata/spring/tcc/EasyTccActionImpl.java
Outdated
Show resolved
Hide resolved
spring/src/test/java/org/apache/seata/spring/tcc/EasyTccActionImpl.java
Outdated
Show resolved
Hide resolved
spring/src/test/java/org/apache/seata/spring/tcc/EasyTccAction.java
Outdated
Show resolved
Hide resolved
@xingfudeshi 老哥 这里再帮忙看看 我还需要做些什么吗 多谢🌹 |
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The unit test does not pass, please fix it. Also, please add this PR to the change log.
@xingfudeshi 老哥 这里我处理好了 帮忙再看看 另外我有个问题想请教一下 这里跑的workflow是等价于mvn clean test吗?我在本地跑的test没法复现这里的问题,反而有其他失败的单测用例,请问有遇到过类似问题吗? |
Yes.The workflow will run the following command:
For details, Please refer to https://github.com/apache/incubator-seata/blob/2.x/.github/workflows/build.yml |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## 2.x #7107 +/- ##
============================================
+ Coverage 53.08% 53.16% +0.08%
- Complexity 6795 6803 +8
============================================
Files 1136 1136
Lines 40341 40345 +4
Branches 4723 4724 +1
============================================
+ Hits 21416 21451 +35
+ Misses 16885 16850 -35
- Partials 2040 2044 +4
|
多谢老哥,这里还有我需要修改的地方吗,我看单测都通过,覆盖率的部分需要修改吗? |
Ⅰ. Describe what this PR did
Ⅱ. Does this pull request fix one issue?
#7047
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews