[Buildbot-devel] Request For Comments/Review : Console view

Nicolas Sylvain nsylvain at chromium.org
Fri Jul 17 15:16:27 UTC 2009


On Fri, Jul 17, 2009 at 8:10 AM, Jean-Paul Calderone <exarkun at divmod.com>wrote:

> On Fri, 17 Jul 2009 10:32:30 -0400, "Dustin J. Mitchell" <
> dustin at zmanda.com> wrote:
>
>> I agree -- this looks great!
>>
>> Given my spotty history of effectively testing Buildbot, I'd like to
>> hear from at least two people that this patch is working for them
>> before I commit.  Can you tell I'm trying to democratize the testing
>> process? :)
>>
>> The patch is here:
>>
>> http://github.com/nsylvain/buildbot/commit/d609080387d292bc7e873ceadeb0637168488232.patch
>>
>>
> Hey guys,
>
> I'm pretty excited to see this being contributed upstream.  I long for a
> nice view like this. :)
>
> That said, maybe there should be some unit tests to go with it?


I was thinking about that too. Do you have an idea about the best way of
testing such things? I was thinking about having a small test master with
some data in it, and having a way to generate a json file or csv file with
the color of each box, and compare it with what I expect. this is not great
though. (and a lot of work) I'm not really familiar with the current
buildbot unittests, but I heard it's pretty limited.

Nicolas


>
> Jean-Paul
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://buildbot.net/pipermail/devel/attachments/20090717/57128e13/attachment.html>


More information about the devel mailing list