[sugar] review: gadget buddy search code

Guillaume Desmottes guillaume.desmottes at collabora.co.uk
Fri May 16 05:46:10 EDT 2008


Le jeudi 15 mai 2008 à 21:41 +0100, Dafydd Harries a écrit :
> Ar 15/05/2008 am 20:33, ysgrifennodd Dafydd Harries:
> > Otherwise, go ahead and merge.
> 
> A couple of other nitpicks:
> 

both are fixed.

> > +class ParsePropertiesTest(TrialTestCase):
> ...
> > +class PropertiesToXmlTest(TrialTestCase):
> 
> These don't use any Trial features, so could just be unittest.TestCases.
> 
> > +        try:
> > +            parse_properties(properties)
> > +        except PropertyTypeError, e:
> > +            pass
> > +        else:
> > +            assert False, "should not be reached"
> 
> self.assertRaises is the idiomatic way to do this.
> 



More information about the Sugar mailing list