I think we can assume a mismatch between the commit builders new_inventory, and the commit objects _basis_delta object.
However both calls to record_entry_contents in commit.py have ann appropriate self._basis_delta.append(delta) call afterwards, which is good.
And the _report_and_accumulate_deletes method which looks for the file ids removed from from the basis inventory vs the new inventory also seems correct.
_populate_from_inventory tells the working tree appropriately about unversioning steps it takes, but the working tree inventory is not considered in delta creation (this is appropriate).
I think we can assume a mismatch between the commit builders new_inventory, and the commit objects _basis_delta object.
However both calls to record_ entry_contents in commit.py have ann appropriate self._basis_ delta.append( delta) call afterwards, which is good.
And the _report_ and_accumulate_ deletes method which looks for the file ids removed from from the basis inventory vs the new inventory also seems correct.
_populate_ from_inventory tells the working tree appropriately about unversioning steps it takes, but the working tree inventory is not considered in delta creation (this is appropriate).