RDFLib / rdflib / 10154699676 / 3

90%

master: 91%

push

github

Prevent Collection from adding 'rdf:nil rdf:rest rdf:nil.' triples (#2818)

* fix: Prevent Collection from add 'rdf:nil rdf:rest rdf:nil.' triples

Creating 'Collection(graph, RDF.nil)' used to add a 'rdf:nil rdf:rest rdf:nil.'
triple to the graph, which turned the empty list into an infinite list.

This was a side-effect of unconditionally appending 'seq' to the collection,
which is not possible when the collection is 'nil' without ill side-effects.

This commit makes the append conditional; and also checks the collection
is non-empty in case the user explicitly tries to add item to the empty list.

* Move emptiness check out of _end()

This will allow using _end() in other methods that would not need to append

* Also prevent insertion at the end of finalized lists

Co-authored-by: Ashley Sommer <ashleysommer@gmail.com>

* Also prevent .append() on finalized lists

Co-authored-by: Ashley Sommer <ashleysommer@gmail.com>

---------

Co-authored-by: Nicholas Car <nick@kurrawong.net>
Co-authored-by: Ashley Sommer <ashleysommer@gmail.com>

6109 of 7834 branches covered (77.98%)

19072 of 21392 relevant lines covered (89.15%)

0.89 hits per line