justin-millman / Kvasir / 15748752179

99%

master: 98%

push

github

When Deleting a Row of an Associative Relation during an Entity Update, Pass Only Primary Key Fields

This commit fixes a bug whereby we were passing too many Fields to the ICommand.Delete function when handling deleted
rows of an associative Relation while updating an Entity. Previously, in such a circumstance, we passed the entire row
to the Delete function; for Maps and OrderedLists, this included more than just the key/index, which is enough to
uniquely identify the row to be deleted. This commit fixes that so the minimal amount of data is forwarded. In addition,
this commit update the documentation of the ICommand.Delete function to make it clear exactly what is expected to be
passed.

3978 of 4110 branches covered (96.79%)

Branch coverage included in aggregate %.

2 of 2 new or added lines in 1 file covered. (100.0%)

6519 of 6545 relevant lines covered (99.6%)

4755.34 hits per line