[MERGE] selftest enhancements: --exclude, --randomize, --list-only
Martin Pool
mbp at canonical.com
Thu Apr 5 07:05:57 BST 2007
Martin Pool has voted +0.
Status is now: Waiting
Comment:
Very nice.
Needs tests though. :) Since this mostly touches the ui, I think just
adding more in tests/blackbox/test_selftest should be enough: run 'bzr
selftest --list' and look at the output and so on. The nice thing is
that --list makes it easier to test the others.
Arguably filter_suite_by_re could have some unit tests.
I know these may seem trivial to verify by hand, but once you accumulate
enough features that are individually trivial it's easy for one of them
to break.
--exclude is ok. It might be nice to add -x as a short name.
As a separate patch maybe you could allow options to be repeated, if
that's not already possible.
Should be in NEWS
For details, see:
http://bundlebuggy.aaronbentley.com/request/%3C4613A8BF.609%40internode.on.net%3E
More information about the bazaar
mailing list