Skip to content
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

feature : mock server #6205

Merged
merged 16 commits into from
Jan 23, 2024
Merged

feature : mock server #6205

merged 16 commits into from
Jan 23, 2024

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

test-mock-server/pom.xml Outdated Show resolved Hide resolved
@xingfudeshi
Copy link
Member

Please add change log

test-mock-server/pom.xml Outdated Show resolved Hide resolved
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
5 checks passed
@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
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