Skip to content

Extend test suite with a range of additional invalid Package URLs#248

Open
wetterjames4 wants to merge 3 commits into
package-url:mainfrom
wetterjames4:main
Open

Extend test suite with a range of additional invalid Package URLs#248
wetterjames4 wants to merge 3 commits into
package-url:mainfrom
wetterjames4:main

Conversation

@wetterjames4
Copy link
Copy Markdown

This includes:

  • duplicated qualifiers keys
  • invalid subpaths
  • invalid types
  • invalid namespaces
  • non-unicode characters must be encoded

Comment thread test-suite-data.json
},
{
"description": "The package type is composed only of ASCII letters and numbers, '.', '+' and '-'",
"purl": "pkg:n@m/test",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Isn't this valid according to the rules in the description?

@johnmhoran johnmhoran added PURL encoding Ecma standard Part of the Ecma standard for PURL labels Dec 14, 2024
@johnmhoran johnmhoran added this to the 1.0-draft milestone Apr 4, 2025
@johnmhoran johnmhoran modified the milestones: PURL v1.0, PURL v0.90 May 29, 2025
@mjherzog mjherzog removed the Ecma standard Part of the Ecma standard for PURL label Jun 9, 2025
@pombredanne
Copy link
Copy Markdown
Member

After the merge of PR #514, PURL tests are now defined in new JSON schema 👼 😇 😁 :

With the new approach... this PR needs to be updated .... Your help is mucho welcomed!
Thanks for your understanding and patience!

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants