-
Notifications
You must be signed in to change notification settings - Fork 8.7k
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
test:Increase [integration-tx-api] module test case coverage #6533
base: 2.x
Are you sure you want to change the base?
Conversation
Pls register this PR in https://github.com/apache/incubator-seata/tree/2.x/changes |
@xingfudeshi Done |
@xingfudeshi Can you resolve merge conflict? |
Yes.The current version is being released.I'll resolve conflicts after the release. |
How long does it usually take? |
It won’t take long. After version 2.1 is released, I will start addressing the conflicting files in this PR. |
@@ -153,6 +153,8 @@ Add changes here for all PR submitted to the 2.x branch. | |||
- [[#6456](https://github.com/apache/incubator-seata/pull/6456)] adjust the test cases related to dynamic configuration | |||
- [[#6466](https://github.com/apache/incubator-seata/pull/6466)] support redis integration testing | |||
- [[#6484](https://github.com/apache/incubator-seata/pull/6484)] fix FileConfigurationTest and MockServerTest fail | |||
- [[#6545](https://github.com/apache/incubator-seata/pull/6545)] fix TestConfigCustomSPI compatibility test fail | |||
- [[#6533](https://github.com/apache/incubator-seata/pull/6533)] increase integration-tx-api module unit test coverage |
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.
alse need to add into zh-cn/2.x.md
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.
alse need to add into zh-cn/2.x.md
@lightClouds917 Done
Ⅰ. Describe what this PR did
increase test case coverage of [integration-tx-api] module
Ⅱ. Does this pull request fix one issue?
fixes #6507
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews