Skip to content

Use postman response to validate body and arguments #56

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

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

jamitJona
Copy link
Contributor

Description

For the post, put and patch test request the postman response is compared with the contents of the mock body to check if the body was sent correctly.

The postman response is also used to check if the arguments of the get request were sent correctly.

Solves #55

@jamitJona jamitJona linked an issue Dec 2, 2020 that may be closed by this pull request
@JensK611
Copy link
Contributor

JensK611 commented Dec 7, 2020

Should we use this adjustment or should we skip this implementation since I'm removing the dependency completely?

@jamitJona
Copy link
Contributor Author

With postman we can also test if the request parameters were sent correctly. Is this also possible with your approach?

@JensK611
Copy link
Contributor

Yes that is possible as I showed you. We tested the url parameter encoding and the sent body is tested as well.

@JensK611
Copy link
Contributor

I suggest to close this PR since the adjustment is not necessary anymore.

@JensK611
Copy link
Contributor

JensK611 commented Jul 6, 2021

@jamitJona can I close this PR?

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.

[Testing] Use postman response to validate body
2 participants