[sugar] review: Guillaume's pubsub-notif branch of Gadget

Dafydd Harries dafydd.harries at collabora.co.uk
Mon May 26 11:40:07 EDT 2008


Ar 26/05/2008 am 16:53, ysgrifennodd Guillaume Desmottes:
> > I don't see the connection between the presence probe and the disco query
> > here. Don't we already have a test for the latter?
> > 
> 
> Because this test check if the presence contains the right capa and
> gadget send it when the buddy turns online.

Oh, I read the test backwards. This tests for Gabble sending requests, not
responding to them. Perhaps we should assert that stanza['type'] == 'result'.

> > Otherwise, fine.
> > 
> 
> I pushed my changes. Thanks for the review.

Great!

-- 
Dafydd


More information about the Sugar mailing list