-
Notifications
You must be signed in to change notification settings - Fork 762
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
Failing test: escaping inconsistent to requestInterceptor #1252
Conversation
974ba2a
to
c4fac2d
Compare
Thanks for the failing tests - I'll do my best to find some time to figure out what's going on. |
It's Fetch that escapes |
@srl295 would be able to rebase this PR on top of current swagger-js? Thank you! |
@char0n I'll try! |
For: swagger-api#1220 `npm t` gives: > Error: Expected 'percent-twentyseven' to equal 'quote' * Request has a quote (') in it * The interceptor sees req.url ending in quote (') which is a perfectly valid thing for a URL to end in… * However, the xmock shows the actual URL ends in /%27
c4fac2d
to
5e17a17
Compare
The moment we switch to [email protected], these tests should pass, and I'll incorporate them into codebase. @srl295 thanks again! |
Welcome, thank you for swagger! |
For: #1220
npm t
gives:Description
Utterly fails. (see issue above, PR not ready to merge)
Motivation and Context
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
package.json
)Checklist: