Skip to content

Conversation

@Bughue
Copy link
Contributor

@Bughue Bughue commented Dec 25, 2023

  • I have registered the PR changes.

Ⅰ. Describe what this PR did

提供mock-server能力,允许客户端进行通信方面的测试,同时为多版本协议的完整测试提供基础。
目前所有接口都默认返回成功,打算这几天用http接口控制模式,模拟普通提交、普通回滚、一阶段超时等等场景。大家有更好的建议也可以提出
Provides mock-server capability to allow client-side testing of communication aspects while providing a basis for complete testing of multiple versions of protocols.
Currently all interfaces return success by default, and intend to use the http interface control mode in the next few days to simulate scenarios such as normal commits, normal rollbacks, one-phase timeouts, and so on.
You can also suggest better suggestions

Ⅱ. Does this pull request fix one issue?

fixes #6212

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

Ⅳ. Describe how to verify it

Ⅴ. Special notes for reviews

@codecov-commenter
Copy link

codecov-commenter commented Dec 26, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (1a57f2d) 49.03% compared to head (818c1db) 49.81%.
Report is 20 commits behind head on 2.x.

❗ Current head 818c1db differs from pull request most recent head ea1d9c4. Consider uploading reports for the commit ea1d9c4 to get more accurate results

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##                2.x    #6205      +/-   ##
============================================
+ Coverage     49.03%   49.81%   +0.78%     
- Complexity     4786     4931     +145     
============================================
  Files           915      917       +2     
  Lines         31896    32010     +114     
  Branches       3855     3862       +7     
============================================
+ Hits          15641    15947     +306     
+ Misses        14699    14505     -194     
- Partials       1556     1558       +2     

see 63 files with indirect coverage changes

@xingfudeshi
Copy link
Member

Please add change log

Copy link
Member

@slievrly slievrly left a comment

Choose a reason for hiding this comment

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

Pls add mock-server's test set to the test module.

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

@Bughue
Copy link
Contributor Author

Bughue commented Jan 18, 2024

Pls add mock-server's test set to the test module.

done

@xingfudeshi xingfudeshi merged commit e9281f0 into apache:2.x Jan 23, 2024
@funky-eyes funky-eyes added this to the 2.1.0 milestone Jan 23, 2024
@funky-eyes funky-eyes added the type: feature Category issues or prs related to feature request. label Jan 23, 2024
YvCeung pushed a commit to YvCeung/incubator-seata that referenced this pull request Dec 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: feature Category issues or prs related to feature request.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Requires mockserver to support communication related coverage tests

6 participants