-
Notifications
You must be signed in to change notification settings - Fork 5
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
Validation error should have in result-set primitive types resolved #267
Conversation
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.
@blcham |
Inputs of last 2 steps of conversion to JSON-LD 1. Frameing expanded JSON-LD
|
@palagdan message above is what i needed before from you to play with it in JSON-LD playground -- i.e. the frame and the input. Hopefully, I have fixed the issue. Note how it works in https://tinyurl.com/4e6t9yum and how I fixed it in the following two commits:
|
@palagdan one more thing is missing. Could you do a test for getFramedAndCompactedJsonLd() ? The test should be simple, just testing evidences or possibly something like message using structure of object that the method returns. We can use example above to test it or example from hello world -- i believe it could be done in < 15 mins. In test called something like |
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.
Consider using multiline string literal in Java. With or without that change you can rebase and merge now.
s-pipes-web/src/test/java/cz/cvut/spipes/rest/handler/ErrorValidationResponseTest.java
Outdated
Show resolved
Hide resolved
@palagdan please before merging this ticket also improve documentation of https://github.com/kbss-cvut/s-pipes/blob/main/doc/examples/constraint-validation/constraint-validation.md:
|
foaf:firstName ?firstName; | ||
. | ||
} | ||
Evidences: |
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.
how come there is:
SELECT ?person ?lastName ?firstName
but lastName is not in the table header???
Resolves #265