docs(ngMock/angular-mocks) added response to whenPOST mock #6744
Conversation
The ```whenPOST``` method should return a response object containing status, response body and headers. If omitted the following error will be thrown: ```Uncaught TypeError: Cannot read property '2' of undefined``` The documentation doesn't make it very clear, so I think it will be appropriate to add it here.
|
Thanks for the PR! Please check the items below to help us merge this faster. See the contributing docs for more information.
If you need to make changes to your pull request, you can update the commit with Thanks again for your help! |
sorenlouv
changed the title
sorenlouv
changed the title
sorenlouv
changed the title
|
I'm sorry, but I wasn't able to verify your Contributor License Agreement (CLA) signature. CLA signature is required for any code contributions to AngularJS. Please sign our CLA and ensure that the CLA signature email address and the email address in this PR's commits match. If you signed the CLA as a corporation, please let us know the company's name. Thanks a bunch! PS: If you signed the CLA in the past then most likely the email addresses don't match. Please sign the CLA again or update the email address in the commit of this PR. |
|
why are you open/closing? =) |
|
Sorry :) I am trying to get my CLA to go through. I created a new PR at #6761 |
|
Don't worry about the CLA for docs fixes, that message will get updated automatically in a few hours. I'll probably merge this tomorrow |
|
Sounds good. Sorry for creating havoc and spamming you with notifications. I'll leave this closed and you can merge #6761 instead |
Request Type: docs
How to reproduce: Use whenPOST without returning a response
Component(s): ngMocks
Impact: small
Complexity: small
This issue is related to: whenPOST (mocking)
Detailed Description: The
whenPOSTmethod should return a response containing status, response body and headers. If omitted the following error will be thrown:Uncaught TypeError: Cannot read property '2' of undefinedThe documentation doesn't make it very clear, so I think it will be appropriate to add.