[sugar] review: Guillaume's pubsub-notif branch of Gadget
Dafydd Harries
dafydd.harries
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-devel
mailing list