Merge pull request #140 from NeedleInAJayStack/fix/warnings

Fixes compiler warnings

Author: NeedleInAJayStack

Sources/GraphQL/Execution/Execute.swift

@@ -187,7 +187,7 @@ public struct ConcurrentDispatchFieldExecutionStrategy: QueryFieldExecutionStrat
             .mapValues { _ -> Future<Any>? in nil }
         var err: Error?
 
-        fields.forEach { field in
+        for field in fields {
             let fieldASTs = field.value
             let fieldKey = field.key
             let fieldPath = path.appending(fieldKey)

Sources/GraphQL/Type/Directives.swift

@@ -1,3 +1,5 @@
+import OrderedCollections
+
 public enum DirectiveLocation: String, Encodable {
     // Operations
     case query = "QUERY"

Sources/GraphQL/Validation/Rules/Custom/NoDeprecatedCustomRule.swift

@@ -52,8 +52,6 @@ public func NoDeprecatedCustomRule(context: ValidationContext) -> Visitor {
                 }
             }
             if let node = node as? ObjectField {
-                let inputObjectDef = context.parentInputType as? GraphQLInputObjectType
-
                 if
                     let inputObjectDef = context.parentInputType as? GraphQLInputObjectType,
                     let inputFieldDef = inputObjectDef.fields[node.name.value],

Tests/GraphQLTests/FieldExecutionStrategyTests/FieldExecutionStrategyTests.swift

@@ -259,7 +259,7 @@ class FieldExecutionStrategyTests: XCTestCase {
         XCTAssertEqual(result.value.data, multiThrowsExpectedData)
         let resultErrors = result.value.errors
         XCTAssertEqual(resultErrors.count, multiThrowsExpectedErrors.count)
-        multiThrowsExpectedErrors.forEach { m in
+        for m in multiThrowsExpectedErrors {
             XCTAssertTrue(resultErrors.contains(m), "Expecting result errors to contain \(m)")
         }
         // XCTAssertEqualWithAccuracy(1.0, result.seconds, accuracy: 0.5)
@@ -308,7 +308,7 @@ class FieldExecutionStrategyTests: XCTestCase {
         XCTAssertEqual(result.value.data, multiThrowsExpectedData)
         let resultErrors = result.value.errors
         XCTAssertEqual(resultErrors.count, multiThrowsExpectedErrors.count)
-        multiThrowsExpectedErrors.forEach { m in
+        for m in multiThrowsExpectedErrors {
             XCTAssertTrue(resultErrors.contains(m), "Expecting result errors to contain \(m)")
         }
         // XCTAssertEqualWithAccuracy(0.1, result.seconds, accuracy: 0.25)