test(NODE-5621): unskip and fix the unicode auth prose spec tests #3965
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Unskip and fix the unicode auth prose spec tests.
What is changing?
serverOptions
are not overwritten by test configuration authURL
(normalized and encoded) andserverOptions
(normalized)Note: We can pass auth credentials as
queryOptions
(btw I renameddbOptions
toqueryOptions
, because the naming was misleading) in tests, but those should be not encoded, because otherwisenewClient()
will runencodeURIComponent()
on already encoded values.Is there new documentation needed for these changes?
None
What is the motivation for this change?
NODE-5621
Double check the following
npm run check:lint
scripttype(NODE-xxxx)[!]: description
feat(NODE-1234)!: rewriting everything in coffeescript