replaced encodeURI with sdk.Url, issue #435

[ShauryaAg]

Replaced encodeURI function with sdk.Url(urlString) from postman-collection to parse the URL

[ShauryaAg]

One of test in golang is failing because the generated code snippet includes url := "https://google.com/" instead of url := "https://google.com"

Can someone help me with this? @webholik

[ShauryaAg]

Made a few changes in test files as mentioned in #438

[webholik]

A url may not have protocol present. For example if the url is postman.com/post, then finalURL.protocol will return undefined and the finalUrl will be undefined://postman.com/post which is invalid.

Infact I don't see why we need to manually construct the final URL. url.toString() does that for us.

[ShauryaAg]

Ah, sorry. I was unfamiliar with url.toString(). I will make a necessary changes as soon as possible.

[ShauryaAg]

@webholik I have made the necessary changes, however url.toString() method doesn't URL encode ' (single-quote) and " (double-quote) character.

[ShauryaAg]

@webholik the test fails as it doesn't URL encode ' and " character.

I think a better approach would be to parse URLs using postman-collection only but then put checks if protocol is empty (and any other conditions that I might have missed)

Something like
finalUrl = `${url.protocol ? `${url.protocol}://` : '' }${url.getRemote()}${finalUrl.getPathWithQuery(true)}`