-
Notifications
You must be signed in to change notification settings - Fork 3
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
Add tests for integrity of merged contexts #26
Comments
If we don't skip these, then
will fail, but this is expected. this is because we have:
from prefix.cc -- which is junk because, the correct semantic URL uses http not https, so there is no direct deterministic join point with the correct canonical prefix or IRI:
so this is effectively orphaned now, we could extend the test to do a transitive walk over aliases and use the following as a join point:
but this would be overkill, and not necessary, as the fundamental assumption here doesn't hold - the merged context does have integrity, it provides the correct bijective mapping for schema.org, according to the precedence rules in which the merged context is built (with the junky prefix.cc having lowest priority). The thing to remember here is that the only thing the API exposes is the canonical mappings, the rest is just there for debugging purposes. It could be argued that a cleaner design would be for the CSVs to only include the canonical mappings for that context, and to put additional anciliary metadata that arises from the ETL elsewhere - a separate issue could be made for this if it's a priority (this would only be a priority for making the library easier to understand, as it wouldn't affect the users of the library) |
#15 introduced data integrity tests, but did not apply them to the merged contexts, because there seems to be some issues with the logic that generates them.
The
merged
andmerged.oak
strings should be removed from the following (effectively leavingskip = set()
to go along with any updates that it takes to generate merged contexts with bijectiveness guarantees.prefixmaps/tests/test_core/test_integrity.py
Line 20 in 433da1f
The text was updated successfully, but these errors were encountered: