Skip to content

Ensure RDF4J datatypes are reference-equal#534

Merged
Ostrzyciel merged 3 commits intomainfrom
piotr/fix-rdf4j-dt-iri
Sep 16, 2025
Merged

Ensure RDF4J datatypes are reference-equal#534
Ostrzyciel merged 3 commits intomainfrom
piotr/fix-rdf4j-dt-iri

Conversation

@Ostrzyciel
Copy link
Member

We had an issue where it was possible for RDF4J to throw assertion errors when constructing datatype literals. I don't think this is well-documented, but you are expected to have the core datatype's iri and the datatype iri strictly equal by reference. Oh well.

We had an issue where it was possible for RDF4J to throw assertion errors when constructing datatype literals. I don't think this is well-documented, but you are expected to have the core datatype's iri and the datatype iri strictly equal by reference. Oh well.
@codecov
Copy link

codecov bot commented Sep 16, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

Files with missing lines Coverage Δ Complexity Δ
...ink/jelly/convert/rdf4j/Rdf4jDecoderConverter.java 55.17% <100.00%> (+24.40%) 0.00 <0.00> (ø)

... and 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Ostrzyciel Ostrzyciel enabled auto-merge (squash) September 16, 2025 10:58
Copy link
Contributor

@niegrzybkowski niegrzybkowski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

@Ostrzyciel Ostrzyciel merged commit 5a80f22 into main Sep 16, 2025
11 checks passed
@Ostrzyciel Ostrzyciel deleted the piotr/fix-rdf4j-dt-iri branch September 16, 2025 11:24
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.

2 participants