[merge] allow for testing multiple Inventory implementations

Martin Pool mbp at sourcefrog.net
Fri Aug 24 07:00:02 BST 2007


On 8/23/07, Aaron Bentley <aaron.bentley at utoronto.ca> wrote:
> This all looks fine, assuming it's leading up to a new Inventory class.

Yes, I'm working on one which avoids loading/storing the whole thing in one go.

> > === added directory 'bzrlib/tests/inventory_implementations'
> > === added file 'bzrlib/tests/inventory_implementations/__init__.py'
>
> ^^^ It's a bit unusual to add a package with nothing in it.  I realize
> we often do this for plugins, but aside from that, I think it makes
> sense to either do
>
> bzrlib/tests/inventory_implementations/
> bzrlib/tests/inventory_implementations/__init__.py
> bzrlib/tests/inventory_implementations/basic_tests.py
> bzrlib/tests/inventory_implementations/advanced_tests.py
>
> Or else do:
> bzrlib/tests/inventory_implementations.py

I'll add more later.

-- 
Martin



More information about the bazaar mailing list