digitalbazaar / jsonld.js

A JSON-LD Processor and API implementation in JavaScript

https://json-ld.org/

Other

1.65k stars 195 forks source link

Add expand and toRDF for non-IRI test. #497

Closed davidlehn closed 1 year ago

davidlehn commented 1 year ago

Test showing a potentially non-intuitive issue where expand passes safe mode but toRDF does not.

codecov-commenter commented 1 year ago

Codecov Report

Merging #497 (c67be87) into main (96ce33b) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #497   +/-   ##
=======================================
  Coverage   93.75%   93.75%           
=======================================
  Files          24       24           
  Lines        2960     2960           
=======================================
  Hits         2775     2775           
  Misses        185      185

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

davidlehn commented 1 year ago

I'll commit this since it's what the code does now. But it might need some discussion later to determine if the expand behavior is what is best.