What is per_inventory supposed to test
Robert Collins
robertc at robertcollins.net
Fri Sep 25 04:23:37 BST 2009
On Thu, 2009-09-24 at 22:07 -0500, John Arbash Meinel wrote:
> So we only have 1 "memory inventory" implementation, and as such I don't
> see it being particularly useful to have a 'per_inventory' that has 1
> implementation.
Me neither ;)
> So it seemed more reasonable to change 'per_inventory' to mean 'run the
> tests that different inventories should support for query methods'.
I think its fine either way.
> The permutations in 'test_inv.py' should really be split out into some
> sort of "per_delta" sort of suite. I think they are very valuable, but
> they aren't *really* testing things that claim to be Inventories.
>
> Specifically, it is testing 'add_inventory_by_delta' across all
> Repository formats, and 'update_basis_by_delta' and
> 'apply_inventory_delta' across all Working Tree formats.
>
> You don't have stuff like 'filter()' on working trees, for example.
In interface terms I guess its
IUpdatableInventory
IAddableInventory
or something similar.
I agree that its awkward, but I don't think it creates issues where it
is, at least not at the moment.
-Rob
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: This is a digitally signed message part
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20090925/7584f52e/attachment-0002.pgp
More information about the bazaar
mailing list