Code review comment for lp:~brendan-donegan/ubuntu-clock-app/disable_location_prompt

Revision history for this message
Leo Arias (elopio) wrote :

6 +# Copyright 2014 Canonical

notice that this copyright header is missing a paragraph. You can copy the right header from test_alarm.py

17 +def set_location_service_testing(test_mode):

This would be nicer as a Fixture. And it should live in a testhelper module of the location service, not on the clock.

review: Needs Fixing

« Back to merge proposal