-
-
Notifications
You must be signed in to change notification settings - Fork 242
Closed
Labels
featureNew feature or requestNew feature or requesthelp wantedContributions are welcomeContributions are welcomeimportantImportant taskImportant task
Milestone
Description
Request and Websocket have the following workflow:
- create instance
- likely, call
WithXXX()methods multiple times - call
Expect()method
It is not allowed to call WithXXX() after Expect() is called. It is also not allowed to call Expect() more than once. However, these requirements are not documented and checked.
What we should do:
- Document these requirements in comment for
Expect. - Report failure if these requirements are violated with
Typeset toAssertUsageand an explanatory message. - Add unit tests that cover these failure reports.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
featureNew feature or requestNew feature or requesthelp wantedContributions are welcomeContributions are welcomeimportantImportant taskImportant task