Update Test Suite #52
Conversation
…appears to be a copy paste error
| "purl": "pkg:docker/customer/dockerimage@sha256:244fd47e07d1004f0aed9c?repository_url=gcr.io", | ||
| "canonical_purl": "pkg:docker/customer/dockerimage@sha256:244fd47e07d1004f0aed9c?repository_url=gcr.io", | ||
| "purl": "pkg:docker/customer/dockerimage@sha256%3A244fd47e07d1004f0aed9c?repository_url=gcr.io", | ||
| "canonical_purl": "pkg:docker/customer/dockerimage@sha256%3A244fd47e07d1004f0aed9c?repository_url=gcr.io", |
There was a problem hiding this comment.
This hasn't been clarified yet and is a subject of discussion in #39 which needs to be clarified before merge.
| @@ -252,7 +252,7 @@ | |||
| "is_invalid": false | |||
| }, | |||
| { | |||
| "description": "slash /// after type is not significant", | |||
| "description": "slash /// after scheme is not significant", | |||
There was a problem hiding this comment.
Thanks for the correction. This is a carryover from a time before PackageURLs began with pkg:/
| @@ -263,6 +263,30 @@ | |||
| "subpath": null, | |||
| "is_invalid": false | |||
| }, | |||
| { | |||
| "description": "slash / between version and qualifiers separator is not significant", | |||
| "purl": "pkg:maven/org.apache.commons/io@2.6/?scope=test", | |||
There was a problem hiding this comment.
The spec states that leading and trailing / is not significant on namespace and subpath only. This may be an invalid test case. Need clarification @pombredanne
There was a problem hiding this comment.
This was added as part of code coverage in the java implementation. If this is invalid - then the How to parse may contain unnecessary steps. The step after Split the remainder once from left on ':' is Strip the remainder from leading and trailing '/'. Trying to figure out what other type of URL would end up with a trailing '/' at this point in parsing.
Correct versions that included a purl with an un-encoded colon in the version (test-suite-data.json:88).
Per purl-spec#character-encoding
Corrected copy paste issue in description of check for /// (test-suite-data.json:255).
Added test for un-necessary trailing slash (test-suite-data.json:267).
Added a bintray example to test namespaces with multiple segments (test-suite-data.json:279).
Added test case to check for invalid '..' segments in the subpath (test-suite-data.json:327).
Added test case to check for invalid '.' segments in the subpath (test-suite-data.json:339).