Merge pull request #72 from saulurias/saulurias-fixment

Added case errror: Unable To Find Location

Author: JakeLin

SwiftWeather/Error.swift

@@ -11,6 +11,7 @@ struct SWError {
     case networkRequestFailed     = -6001
     case jsonSerializationFailed  = -6002
     case jsonParsingFailed        = -6003
+    case unableToFindLocation  = -6004
   }
 
   let errorCode: Code

SwiftWeather/LocationService.swift

@@ -8,6 +8,7 @@ import CoreLocation
 
 protocol LocationServiceDelegate {
   func locationDidUpdate(_ service: LocationService, location: CLLocation)
+    func locationDidFail(withError error: SWError)
 }
 
 class LocationService: NSObject {
@@ -38,6 +39,8 @@ extension LocationService : CLLocationManagerDelegate {
   }
 
   func locationManager(_ manager: CLLocationManager, didFailWithError error: Error) {
+    let swError = SWError(errorCode: .unableToFindLocation)
+    delegate?.locationDidFail(withError: swError)
     print("Error finding location: \(error.localizedDescription)")
   }
 }

SwiftWeather/WeatherViewModel.swift

@@ -71,8 +71,10 @@ class WeatherViewModel {
         errorMessage.value = "We're having trouble processing weather data."
       case .jsonParsingFailed:
         errorMessage.value = "We're having trouble parsing weather data."
+      case .unableToFindLocation:
+        errorMessage.value = "We're having trouble getting user location."
       }
-
+    
       location.value = emptyString
       iconText.value = emptyString
       temperature.value = emptyString
@@ -98,4 +100,8 @@ extension WeatherViewModel: LocationServiceDelegate {
       })
     }
   }
+  
+  func locationDidFail(withError error: SWError) {
+      self.update(error)
+  }
 }