chore: Fixing SwiftLint warnings (#2732)

* chore: Fixing SwiftLint warnings in Amplify project

* chore: Fixing SwiftLint warnings in APICategoryPlugin  project

* chore: Fixing SwiftLint warnings in AnalyticsCategoryPlugin project

* chore: Fixing SwiftLint warnings in AWSCognitoAuthPlugin project

* chore: Fixing SwiftLint warnings in DataStoreCategoryPlugin project

* chore: Fixing SwiftLint warnings in PredictionsCategoryPlugin project

* chore: Fixing SwiftLint warnings in StoragePlugin project

Author: sebaland

Amplify/Categories/API/Interceptor/URLRequestInterceptor.swift

@@ -19,6 +19,7 @@ import Foundation
 /// plugin.
 public protocol URLRequestInterceptor {
 
+    // swiftlint:disable:next todo
     // TODO: turn async https://github.com/aws-amplify/amplify-ios/issues/73
     /// Inspect and optionally modify the request, returning either the original
     /// unmodified request or a modified copy.

Amplify/Categories/API/Operation/GraphQLOperation.swift

@@ -12,7 +12,6 @@ open class GraphQLOperation<R: Decodable>: AmplifyOperation<
     APIError
 > { }
 
-
 /// GraphQL Subscription Operation
 open class GraphQLSubscriptionOperation<R: Decodable>: AmplifyInProcessReportingOperation<
     GraphQLOperationRequest<R>,

Amplify/Categories/API/Request/GraphQLOperationRequest.swift

@@ -46,7 +46,7 @@ public struct GraphQLOperationRequest<R: Decodable>: AmplifyOperationRequest {
     }
 }
 
-// MARK:  GraphQLOperationRequest + Options
+// MARK: GraphQLOperationRequest + Options
 public extension GraphQLOperationRequest {
     struct Options {
         public let pluginOptions: Any?

Amplify/Categories/API/Request/GraphQLRequest.swift

@@ -44,7 +44,6 @@ public struct GraphQLRequest<R: Decodable> {
     }
 }
 
-
 // MARK: GraphQLRequest + Options
 
 public extension GraphQLRequest {

Amplify/Categories/Auth/Request/AuthChangePasswordRequest.swift

@@ -32,6 +32,7 @@ public extension AuthChangePasswordRequest {
 
     struct Options {
 
+        // swiftlint:disable:next todo
         // TODO: Move this metadata to plugin options. All other request has the metadata
         // inside the plugin options.
 

Amplify/Categories/Auth/Request/AuthDeleteUserRequest.swift

@@ -20,4 +20,3 @@ public struct AuthDeleteUserRequest: AmplifyOperationRequest {
 public extension AuthDeleteUserRequest {
     struct Options {}
 }
-

Amplify/Categories/DataStore/DataStoreCategory+Behavior.swift

@@ -22,6 +22,7 @@ extension DataStoreCategory: DataStoreBaseBehavior {
                                 byIdentifier id: String,
                                 completion: (DataStoreResult<M?>) -> Void) where M: ModelIdentifiable,
                                                                                  M.IdentifierFormat == ModelIdentifierFormat.Default {
+        // swiftlint:disable:previous line_length
         plugin.query(modelType, byIdentifier: id, completion: completion)
     }
 
@@ -57,7 +58,7 @@ extension DataStoreCategory: DataStoreBaseBehavior {
                                  where predicate: QueryPredicate? = nil,
                                  completion: @escaping DataStoreCallback<Void>) where M: ModelIdentifiable,
                                                                                       M.IdentifierFormat == ModelIdentifierFormat.Default {
-
+        // swiftlint:disable:previous line_length
         plugin.delete(modelType, withIdentifier: id, where: predicate, completion: completion)
     }
 

Amplify/Categories/DataStore/DataStoreCategoryBehavior.swift

@@ -51,15 +51,16 @@ public protocol DataStoreBaseBehavior {
                           where predicate: QueryPredicate?,
                           completion: @escaping DataStoreCallback<Void>) where M: ModelIdentifiable,
                                                                                M.IdentifierFormat == ModelIdentifierFormat.Default
+    // swiftlint:disable:previous line_length
 
     func delete<M: Model>(_ modelType: M.Type,
                           withIdentifier id: ModelIdentifier<M, M.IdentifierFormat>,
                           where predicate: QueryPredicate?,
                           completion: @escaping DataStoreCallback<Void>) where M: ModelIdentifiable
 
     func delete<M: Model>(_ modelType: M.Type,
-                           where predicate: QueryPredicate,
-                           completion: @escaping DataStoreCallback<Void>)
+                          where predicate: QueryPredicate,
+                          completion: @escaping DataStoreCallback<Void>)
 
     /**
      Synchronization starts automatically whenever you run any DataStore operation (query(), save(), delete())

Amplify/Categories/DataStore/DataStoreError.swift

@@ -63,6 +63,7 @@ extension DataStoreError: AmplifyError {
         case .invalidDatabase(let path, _):
             return "Make sure the path \(path) is valid and the device has available storage space."
         case .invalidModelName(let modelName):
+            // swiftlint:disable:next todo
             // TODO: Is this the right command to run to generate models?
             return "Make sure the model named '\(modelName)' is registered by running `amplify codegen`"
         case .invalidOperation(let causedBy):

Amplify/Categories/DataStore/Model/Collection/List+Model.swift

@@ -125,6 +125,7 @@ public class List<ModelType: Model>: Collection, Codable, ExpressibleByArrayLite
     // MARK: - Persistent Operations
 
     public var totalCount: Int {
+        // swiftlint:disable:next todo
         // TODO handle total count
         return 0
     }
@@ -143,6 +144,7 @@ public class List<ModelType: Model>: Collection, Codable, ExpressibleByArrayLite
     /// the plugin to successfully return an instance of `ModelListProvider`.
     required convenience public init(from decoder: Decoder) throws {
         for listDecoder in ModelListDecoderRegistry.listDecoders.get() {
+            // swiftlint:disable:next for_where
             if listDecoder.shouldDecode(modelType: ModelType.self, decoder: decoder) {
                 let listProvider = try listDecoder.makeListProvider(modelType: ModelType.self, decoder: decoder)
                 self.init(listProvider: listProvider)