Skip to content

test: cover URL canonicalization utilities - #16

Closed
saurabhhhcodes wants to merge 1 commit into
tanmayjoddar:mainfrom
saurabhhhcodes:codex/canonicalize-tests-3
Closed

test: cover URL canonicalization utilities#16
saurabhhhcodes wants to merge 1 commit into
tanmayjoddar:mainfrom
saurabhhhcodes:codex/canonicalize-tests-3

Conversation

@saurabhhhcodes

Copy link
Copy Markdown

Summary

  • add Jest coverage for canonicalizePathname, canonicalEndpoint, and normalizeUrl
  • cover numeric IDs, UUIDs, high-entropy path segments, root path handling, method normalization, and query/hash stripping

Fixes #3

Validation

  • git diff --check
  • node --check tests/utils/canonicalize.test.js
  • npm test -- --runInBand could not run locally because npm is not available in this shell

GSSoC labels requested

Please add/keep gssoc and the required difficulty/type scoring labels if this is accepted.

@saurabhhhcodes

Copy link
Copy Markdown
Author

Closing this older version to keep the review queue clean. The current replacement for URL canonicalization coverage is #28.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

test: add unit tests for URL/path canonicalization utilities

1 participant