[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