[MERGE] selftest enhancements: --exclude, --randomize, --list-only
Ian Clatworthy
ian.clatworthy at internode.on.net
Wed Apr 11 03:47:19 BST 2007
Martin Pool wrote:
> 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.
>
Blackbox tests added. Whitebox tests (for both filter_suite_by_re and
sort_suite_by_re) added as well.
> --exclude is ok. It might be nice to add -x as a short name.
>
Done.
> As a separate patch maybe you could allow options to be repeated, if
> that's not already possible.
>
I'll add it as a 'wishlist' bug on the Bug Tracker.
> Should be in NEWS
>
Done. Updated HACKING text as well.
> For details, see:
> http://bundlebuggy.aaronbentley.com/request/%3C4613A8BF.609%40internode.on.net%3E
>
>
Ian C.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: selftest-exclude-and-randomize_2.patch
Type: text/x-patch
Size: 62163 bytes
Desc: not available
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20070411/b02f5d48/attachment-0001.bin
More information about the bazaar
mailing list