Code review comment for lp:~ssweeny/location-service/espoo-delayed-provider.15.04

Thomas Voß (thomas-voss) wrote :

A remark inline about splitting changes between espoo-delayed.provider.15.04 and delayed-provider.15.04.

One other remark: I would prefer a set of more sophisticated test cases that exercise the timing logic. Otherwise, we would have to invest significant manual testing efforts for each landing to ensure correct behavior. Would you mind introducing such tests into this MP?

review: Needs Fixing

« Back to merge proposal