[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