[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