swagger-js icon indicating copy to clipboard operation
swagger-js copied to clipboard

Failing test: escaping inconsistent to requestInterceptor

Open srl295 opened this issue 7 years ago • 7 comments

For: https://github.com/swagger-api/swagger-js/issues/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

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

  • [ ] No code changes (changes to documentation, CI, metadata, etc)
  • [ ] Dependency changes (any modification to dependencies in package.json)
  • [ ] Bug fix (non-breaking change which fixes an issue)
  • [ ] New feature (non-breaking change which adds functionality)
  • [ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • [ ] My code follows the code style of this project.
  • [ ] My change requires a change to the documentation.
  • [ ] I have updated the documentation accordingly.
  • [ ] I have added tests to cover my changes.
  • [ ] All new and existing tests passed.

srl295 avatar Feb 27 '18 16:02 srl295

Thanks for the failing tests - I'll do my best to find some time to figure out what's going on.

shockey avatar Mar 03 '18 01:03 shockey

It's Fetch that escapes

srl295 avatar Mar 07 '18 06:03 srl295

@srl295 would be able to rebase this PR on top of current swagger-js? Thank you!

char0n avatar May 01 '20 11:05 char0n

@char0n I'll try!

srl295 avatar May 04 '20 14:05 srl295

@char0n done! 5e17a17

srl295 avatar May 04 '20 15:05 srl295

The moment we switch to [email protected], these tests should pass, and I'll incorporate them into codebase. @srl295 thanks again!

char0n avatar May 04 '20 15:05 char0n

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!

srl295 avatar May 04 '20 15:05 srl295

@srl295 took five years, but you test have been finally integrated in https://github.com/swagger-api/swagger-js/pull/3137/commits/f9207bd1d2663e87d10cdab7ff5b9aefe026c8fa ;]

Thank you!

char0n avatar Sep 12 '23 07:09 char0n

@srl295 took five years, but you test have been finally integrated in https://github.com/swagger-api/swagger-js/pull/3137/commits/f9207bd1d2663e87d10cdab7ff5b9aefe026c8fa ;]

Thank you!

Welcome!

srl295 avatar Sep 14 '23 05:09 srl295