Skip to content

Unit test for api_collect#1810

Merged
mappjzc merged 3 commits into
mainfrom
unit-test
May 9, 2022
Merged

Unit test for api_collect#1810
mappjzc merged 3 commits into
mainfrom
unit-test

Conversation

@mappjzc
Copy link
Copy Markdown
Contributor

@mappjzc mappjzc commented May 7, 2022

Summary

Unit test for api_collect

Key Points

  • This is a breaking change
  • New or existing documentation is updated

Description

TestSaveRawData
TestSaveRawData_Fail
TestHandleResponse
TestHandleResponse_Fail
TestFetchAsync
TestFetchAsync_Fail
TestHandleResponseWithPages
TestHandleResponseWithPages_Fail
TestStepFetch
TestStepFetch_Fail
TestExecWithOutPageSize
TestExecWithGetTotalPages
TestExecWithOutGetTotalPages
TestExecute
fix some bugs.

Does this close any open issues?

close #1656
close #1766
close #1802
close #1803

Current Behavior

Describe the current behaviour of this issue, if relevant.

New Behavior

Describe the new behaviour updated in this issue, if relevant.

Screenshots

Include any relevant screenshots here.

Other Information

Any other information that is important to this PR.

mappjzc added 3 commits May 7, 2022 10:50
add out to Execute of api_collect for break out the loop.

Nddtfjiang <zhicheng.jiang@merico.dev>
Add the 0 slots check in api_collect.

Nddtfjiang <zhicheng.jiang@merico.dev>
TestSaveRawData
TestSaveRawData_Fail
TestHandleResponse
TestHandleResponse_Fail
TestFetchAsync
TestFetchAsync_Fail
TestHandleResponseWithPages
TestHandleResponseWithPages_Fail
TestStepFetch
TestStepFetch_Fail
TestExecWithOutPageSize
TestExecWithGetTotalPages
TestExecWithOutGetTotalPages
TestExecute

Nddtfjiang <zhicheng.jiang@merico.dev>
@mappjzc mappjzc added this to the v0.11.0 milestone May 7, 2022
@klesh klesh requested a review from mindlesscloud May 9, 2022 02:28
@mappjzc mappjzc merged commit d4d76dc into main May 9, 2022
@mappjzc mappjzc deleted the unit-test branch May 9, 2022 11:13
@mappjzc mappjzc restored the unit-test branch May 9, 2022 14:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants