-
Notifications
You must be signed in to change notification settings - Fork 110
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
Added cookie interaction integration tests for pm.sendRequest #359
Conversation
22d50aa
to
ae636d0
Compare
ae636d0
to
d9223d4
Compare
pm.expect(_response).to.have.property('code', 200); | ||
pm.expect(_response).to.have.property('status', 'OK'); | ||
pm.expect(_response.json).to.equal({ cookies: {} }); | ||
pm.expect(pm.cookies.has('foo')).to.be(false); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@kunagpal shouldn't we make sure these assertions pass, on testRun.test.getCall(0)
or testRun.assertion.getCall(3)
. In order to make sure our tests fail for failures in sandbox scripts?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good idea, I'll add them.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added.
* Added explicit response body checks * Made test names consistent and useful * Rectified overall flakiness
a0172e1
to
b7034c0
Compare
* Added explicit response body checks * Made test names consistent and useful * Rectified overall flakiness
* Added explicit response body checks * Made test names consistent and useful * Rectified overall flakiness
b7034c0
to
9466edc
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
* release/6.3.1: fix(package): update postman-sandbox to version 2.3.3 fix(package): update postman-collection to version 2.1.3 6.3.1-beta.2 Bumped collection SDK to v2.3.1-beta.1 6.3.1-beta.1 Make sure `request.url.toString` is present before using it Addressed PR #359 comments * Added explicit response body checks * Made test names consistent and useful * Rectified overall flakiness Make sure empty Urls do not break runs Added cookie tests for pm.sendRequest
No description provided.