Skip to content
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

Make sure SPARQL checks cover the entire ontology. #1155

Merged
merged 2 commits into from
Dec 12, 2024
Merged

Conversation

gouttegd
Copy link
Contributor

ROBOT's verify command does not follow OWL Import declarations, so the checks it performs only applies strictly to the file it is given as input.

Components and pattern-derived definitions are part of the "local" ontology and should be covered by the SPARQL checks, and we ensure they are by running the verify command on the $(SRCMERGED) intermediate file, in which components and pattern-derived definitions (basically all files listed in $(OTHER_SRC)`) are explicitly merged (but not ODK imports).

closes #1154

ROBOT's `verify` command does not follow OWL Import declarations, so the
checks it performs only applies strictly to the file it is given as
input.

Components and pattern-derived definitions are part of the "local"
ontology and should be covered by the SPARQL checks, and we ensure they
are by running the `verify` command on the `$(SRCMERGED)` intermediate
file, in which components and pattern-derived definitions (basically all
files listed in $(OTHER_SRC)`) are explicitly merged (but _not_ ODK
imports).

closes #1154
@gouttegd gouttegd self-assigned this Dec 12, 2024
CHANGELOG.md Outdated Show resolved Hide resolved
matentzn
matentzn previously approved these changes Dec 12, 2024
Co-authored-by: Nico Matentzoglu <[email protected]>
@gouttegd gouttegd merged commit aa7594e into master Dec 12, 2024
1 check passed
@gouttegd gouttegd deleted the fix-sparql-test branch December 12, 2024 21:53
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.

sparql_test does not cover components
2 participants