Implemented LocalizedError protocol for SpotifyError

(using updated SwiftyOAuth and Avenue libs)

Cartfile.resolved

@@ -1,5 +1,5 @@
 github "radianttap/Ambar" "6.2"
-github "radianttap/Avenue" "3.1"
+github "radianttap/Avenue" "3.1.1"
 github "radianttap/CardPresentationController" "1.5.1"
 github "radianttap/Coordinator" "6.2.1"
 github "radianttap/Kingfisher" "5.3.0"

Spotify/Spotify.swift

@@ -142,9 +142,8 @@ private extension Spotify {
 			case .success(let token):
 				print(token)
 			case .failure(let error):
-				print(error)
 				let callback = apiRequest.callback
-				callback(nil, .authError)
+				callback(nil, .authError(error))
 			}
 		}
 	}
@@ -167,7 +166,7 @@ private extension Spotify {
 		let callback = apiRequest.callback
 
 		guard let token = oauthProvider.token, token.isValid else {
-			callback(nil, .authError)
+			callback(nil, .invalidAuthToken)
 			return
 		}
 

Spotify/SpotifyError.swift

@@ -8,8 +8,9 @@
 
 import Foundation
 import Avenue
+import SwiftyOAuth
 
-enum SpotifyError: Error {
+enum SpotifyError: Swift.Error {
 	case invalidBaseURL(basePath: String)
 	case invalidFinalURL(url: URL, queryParams: JSON)
 	case requestBuildFailed(Swift.Error)
@@ -19,5 +20,64 @@ enum SpotifyError: Error {
 	case emptyResponse
 	case unexpectedResponse(HTTPURLResponse, String?)
 	case httpError(HTTPURLResponse)
-	case authError
+
+	case invalidAuthToken
+	case authError(SwiftyOAuth.Error)
+}
+
+
+extension SpotifyError: LocalizedError {
+	var errorDescription: String? {
+		switch self {
+		case .invalidBaseURL, .invalidFinalURL, .requestBuildFailed:
+			return NSLocalizedString("Internal error", comment: "")
+
+		case .invalidResponseType, .emptyResponse, .unexpectedResponse, .httpError:
+			return NSLocalizedString("Invalid or unexpected response from Spotify", comment: "")
+
+		case .networkError(let netErr):
+			return netErr.errorDescription
+
+		case .invalidAuthToken:
+			return NSLocalizedString("Authorization error", comment: "")
+
+		case .authError(let authError):
+			return authError.errorDescription
+		}
+	}
+
+	var failureReason: String? {
+		switch self {
+		case .invalidBaseURL(let basePath):
+			return String(format: NSLocalizedString("Failed to build URL using: %@", comment: ""), basePath)
+
+		case .invalidFinalURL(let url, let queryParams):
+			return String(format: NSLocalizedString("Failed to build URL using: %@ and query-string parameters: %@", comment: ""), url.path, queryParams)
+
+		case .requestBuildFailed(let error):
+			return String(format: NSLocalizedString("Failed to build URLRequest: %@", comment: ""), error.localizedDescription)
+
+		case .invalidResponseType:
+			return NSLocalizedString("Unexpected response (non HTTP)", comment: "")
+
+		case .emptyResponse:
+			return NSLocalizedString("Empty response received (no data)", comment: "")
+
+		case .unexpectedResponse(let httpURLResponse, let str):
+			let s = str ?? String(describing: httpURLResponse.allHeaderFields)
+			return s
+
+		case .httpError(let httpURLResponse):
+			return String(describing: httpURLResponse.allHeaderFields)
+
+		case .networkError(let netErr):
+			return netErr.failureReason
+
+		case .invalidAuthToken:
+			return NSLocalizedString("Authorization token is missing or expired", comment: "")
+
+		case .authError(let authError):
+			return authError.failureReason
+		}
+	}
 }