Ubuntu Single Sign On Client

Code review comment for lp:~diegosarmentero/ubuntu-sso-client/931452

  1. 931452
  2. Merge into trunk
Revision history for this message
Diego Sarmentero (diegosarmentero) wrote :

> Nice use of functools.partial
>
> The only change I would suggest is fairly minor: Your test conditions are
> always assertTrue(x in y). You could instead do assertIn(x, y).

Thanks for the suggestion Brian, done!

« Back to merge proposal