[RFC] support for plugins to implement their own log formatters

Erik Bågfors zindar at gmail.com
Tue Feb 14 07:17:48 GMT 2006


2006/2/14, John A Meinel <john at arbash-meinel.com>:
> Erik Bågfors wrote:
> >> A blackbox test would be sufficient. You make 2 commits, and then you
> >> check the output of "bzr log --log-format=XXX".
> >> Look in "bzrlib/tests/test_log.py" it should give a basic idea. (we
> >> should be testing for --line, and --short).
> >>
> >> But mostly I want you to write a test that forces the code with the bug
> >> to be run. So that a bug doesn't show up there again.
> >
> > I've done that now. I am simply checking the output of bzr log/missing
> > by counting how many lines they output, since "short" and "line"
> > outputs a different number of lines.  Hope this is ok.
> >
> > The tests triggers the bug which you can test by reverting one
> > revision. (oops, just found a typo, revert two now).
> >
> > I've merged jam-integration into this branch as well, to get blackbox
> > tests to run.
> >
> > /Erik
> >
>
> There is a small thing that still needs to be done. The list of tests is
> kept alphabetical.
>
> So instead of:
>                       'bzrlib.tests.blackbox.test_export',
>                       'bzrlib.tests.blackbox.test_missing',
>                       'bzrlib.tests.blackbox.test_outside_wt',
> +                     'bzrlib.tests.blackbox.test_logformats',
>                       'bzrlib.tests.blackbox.test_pull',
>
> We would prefer
>                       'bzrlib.tests.blackbox.test_export',
> +                     'bzrlib.tests.blackbox.test_logformats',
>                       'bzrlib.tests.blackbox.test_missing',
>                       'bzrlib.tests.blackbox.test_outside_wt',
>                       'bzrlib.tests.blackbox.test_pull',
>
> Otherwise I think it looks good.

Ok, done.

/Erik




More information about the bazaar mailing list