[PATCH 19/25] tests: Clean up 'test_user'

Andy Doan andy.doan at linaro.org
Thu Jun 30 03:10:35 AEST 2016


On 06/23/2016 04:53 PM, Stephen Finucane wrote:
> * Don't use hardcode routes: use the reverse function instead
> * Make use of 'create_' helper functions
> * Minimize duplication of code
> * Remove unneeded 'XXX.objects.delete()' calls (all objects are deleted
>    on teardown of each test)
> * Include every import on its own line
> * Use underscore_case, rather than camelCase
>
> This includes one trivial, albeit necessary, removal of an import from
> 'test_user_browser'.
>
> Signed-off-by: Stephen Finucane <stephen.finucane at intel.com>

Reviewed-by: Andy Doan <andy.doan at linaro.org>


More information about the Patchwork mailing list