Merge branch 'refs/heads/master' into gh0001_Implement_Season_Service
Basic tests for failure cases working

Need to add more tests

Signed-off-by: Ippoippo <ippoippo-software@ippoippo-eco.com>
Test update

New test added and working.
However, I have a code smell in handleRequestForLocationWhenServicesEnabled method.

It relies on a private method to work out if 'locationRefreshRequired'. That method then works on a private property in the delegate which 'caches' the last stored location and timestamp.

WHY am I doing that. LocationManager already tracks history.
So, I should get rid of my properties, and rewrite locationRefreshRequired to use the locations stored in Location Manager.
Then when I inject the mock LocationManager into the class under test, I'll stub out the methods as appropriate.

Signed-off-by: Ippoippo <ippoippo-software@ippoippo-eco.com>
Reworked Location Manager Delegate

New tests, refactored code.
Tests still need to be refactored and new tests added for the CLLocationManagerDelegate methods

Signed-off-by: Ippoippo <ippoippo-software@ippoippo-eco.com>
Tests completed

Signed-off-by: Ippoippo <ippoippo-software@ippoippo-eco.com>
Merge branch 'refs/heads/gh0001_Implement_Season_Service'