-
Notifications
You must be signed in to change notification settings - Fork 34
Add checks for terms including a colon #82
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…ovide for terms ending with a colon. Modify, obsolete and add tests accordingly.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM from an eye balling it perspective. Left a minor naming consistency comment.
"@type": ["jld:NegativeEvaluationTest", "jld:ExpandTest"], | ||
"name": "Term definition with @id: @type", | ||
"purpose": "Expanding term mapping to @type uses @type syntax now illegal", | ||
"input": "expand/e043-in.jsonld", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The label #te043
here doesn't match the input file name prefix e043
(same for e044
) -- which seems to break the naming convention, so I suspect this is a typo. Just checking here, was this intentional or a typo?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tests all of an identifier such as #txxxx
with corresponding files expand/xxxx-in.jsonld
and expand/xxx-out.jsonld
. In this case the #te043
is consistent with the file named e043-in
and follows the same pattern as that for #te042
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the convention has been the test ids have a t
prefix, the file names don't. Is that what you meant?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, the ids all start with a "t", the filename's don't. This convention was established in the 1.0 days. I established the practice if replacing one or two leading zeros with letters to help separate out the new tests by broad feature categorization, which helps in filtering as well.
The #texxx
tests typically are for errors, some of which previously came from a separate error manifest.
which do not expand in a natural way.
For w3c/json-ld-syntax#155
Preview | Diff