Skip to content

test: cover URL canonicalization utilities#28

Open
saurabhhhcodes wants to merge 1 commit into
tanmayjoddar:mainfrom
saurabhhhcodes:test/url-canonicalization-3
Open

test: cover URL canonicalization utilities#28
saurabhhhcodes wants to merge 1 commit into
tanmayjoddar:mainfrom
saurabhhhcodes:test/url-canonicalization-3

Conversation

@saurabhhhcodes

@saurabhhhcodes saurabhhhcodes commented May 20, 2026

Copy link
Copy Markdown

Summary\n- add Jest coverage for numeric path segment canonicalization\n- verify UUID and high-entropy path segments are normalized\n- cover root path preservation and normalizeUrl query/hash removal\n\n## Validation\n- npm test -- --runInBand\n- node --check tests/utils/canonicalize.test.js\n- git diff --check\n\nCloses #3

Suggested GSSoC labels

If accepted for scoring, this should fit gssoc, level:beginner, dx, type:testing, quality:clean based on the linked issue labels and the focused validation in this PR.

@saurabhhhcodes

Copy link
Copy Markdown
Author

Quick status for review: this PR is mergeable from GitHub's side and intentionally scoped to the linked apidrift-cli issue.

Validation already included in the PR body:

  • focused Jest coverage for the touched behavior
  • syntax check for the touched test/source file where applicable
  • git diff --check

I also added the suggested GSSoC scoring labels in the PR body so maintainers can apply them directly if the change is accepted.

@saurabhhhcodes

Copy link
Copy Markdown
Author

Hi @tanmayjoddar, this PR is still open, mergeable, and clean against the current base branch. I rechecked the repo test baseline locally (node --experimental-vm-modules node_modules/jest/bin/jest.js -> 3 suites / 11 tests passed). Please review when you get a chance, and add the GSSoC labels on the PR to mirror the linked issue if you plan to merge it.

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.

1 participant