Skip to content

Conversation

@jxs1211
Copy link

@jxs1211 jxs1211 commented Jul 22, 2022

Signed-off-by: xian-jie.shen 327411586@qq.com

Pre-Checklist

Note: please complete ALL items in the following checklist.

  • I have read through the CONTRIBUTING.md documentation.
  • My code has the necessary comments and documentation (if needed).
  • I have added relevant tests

Description

Try to mock an HTTP server for testing github client API(inspired by go-github client's test case), the delta codes of this testing API method are a little bit more than before, so this pr is just given it a try.

Related Issues

see #833 for the details

New Behavior (screenshots if needed)

the ut of release by calling a local mock server
image

Signed-off-by: xian-jie.shen <327411586@qq.com>
@jxs1211 jxs1211 requested a review from a team as a code owner July 22, 2022 07:51
Copy link
Member

@iyear iyear left a comment

Choose a reason for hiding this comment

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

Nice testings!

Copy link

@steinliber steinliber left a comment

Choose a reason for hiding this comment

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

Thanks for your testing

@steinliber steinliber merged commit 0432eaf into devstream-io:main Jul 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants