Add test package (#687)

* chore: update default client config properties' defaults, update protocol response tests rendering, fix generated client config type in client

* feat: add smithy-swift.smithy-swift-codegen-test-utils package

* feat: add smithy-swift.smithy-swift-codegen-test package

* chore: fix swiftlint

* chore: address PR comments

Author: AndrewFossAWS

.swiftlint.yml

@@ -1,10 +1,13 @@
 excluded:
   - .build
   - Sources/SmithyTestUtil/*
+  - Sources/WeatherSDK/*
   - Tests/ClientRuntimeTests/*
   - Tests/SmithyTestUtilTests/*
   - Tests/SmithyXMLTests/*
   - Tests/SmithyTimestampsTests/*
+  - Tests/WeatherSDKTests/*
+  - smithy-swift-codegen-test/build/*
 
 analyzer_rules:
   - unused_import

Package.swift

@@ -81,3 +81,18 @@ let package = Package(
         ),
     ].compactMap { $0 }
 )
+
+func addTestServiceTargets() {
+    package.targets += [
+        .target(
+            name: "WeatherSDK",
+            dependencies: ["SmithyTestUtil", "ClientRuntime"]
+        ),
+        .testTarget(
+            name: "WeatherSDKTests",
+            dependencies: ["WeatherSDK"]
+        )
+    ]
+}
+
+addTestServiceTargets()

Sources/ClientRuntime/Config/DefaultSDKRuntimeConfiguration.swift

@@ -89,7 +89,9 @@ public extension DefaultSDKRuntimeConfiguration {
     ///
     /// - Parameter httpClientConfiguration: The configuration for the HTTP client.
     /// - Returns: The `CRTClientEngine` client on Mac & Linux platforms, returns `URLSessionHttpClient` on non-Mac Apple platforms.
-    static func makeClient(httpClientConfiguration: HttpClientConfiguration) -> HTTPClient {
+    static func makeClient(
+        httpClientConfiguration: HttpClientConfiguration = defaultHttpClientConfiguration
+    ) -> HTTPClient {
         #if os(iOS) || os(tvOS) || os(watchOS) || os(visionOS) || os(macOS)
         return URLSessionHTTPClient(httpClientConfiguration: httpClientConfiguration)
         #else
@@ -119,4 +121,27 @@ public extension DefaultSDKRuntimeConfiguration {
     ///
     /// Defaults to `.request`.
     static var defaultClientLogMode: ClientLogMode { .request }
+
+    static var defaultAuthSchemeResolver: AuthSchemeResolver { DefaultAuthSchemeResolver() }
+}
+
+public class DefaultAuthSchemeResolverParameters: AuthSchemeResolverParameters {
+    public var operation: String
+    public init(operation: String) {
+        self.operation = operation
+    }
+}
+
+public class DefaultAuthSchemeResolver: AuthSchemeResolver {
+    public func resolveAuthScheme(params: AuthSchemeResolverParameters) throws -> [AuthOption] {
+        return []
+    }
+
+    public func constructParameters(context: HttpContext) throws -> AuthSchemeResolverParameters {
+        guard let opName = context.getOperation() else {
+            throw ClientError.dataNotFound(
+                "Operation name not configured in middleware context for auth scheme resolver params construction.")
+        }
+        return DefaultAuthSchemeResolverParameters(operation: opName)
+    }
 }

Sources/SmithyTestUtil/Errors/JSONError.swift

@@ -0,0 +1,25 @@
+/*
+ * Copyright Amazon.com, Inc. or its affiliates. All Rights Reserved.
+ * SPDX-License-Identifier: Apache-2.0.
+ */
+
+import Foundation
+import ClientRuntime
+
+/// A general error structure for protocols with JSON response
+public struct JSONError {
+    public let errorMessage: String?
+    public let errorType: String?
+
+    public init(httpResponse: HttpResponse) async throws {
+        var message: String?
+        var errorType: String?
+        if let data = try await httpResponse.body.readData() {
+            let output: JSONErrorPayload = try JSONDecoder().decode(responseBody: data)
+            message = output.message
+            errorType = output.errorType
+        }
+        self.errorMessage = message
+        self.errorType = errorType
+    }
+}

Sources/SmithyTestUtil/Errors/JSONErrorPayload.swift

@@ -0,0 +1,9 @@
+/*
+ * Copyright Amazon.com, Inc. or its affiliates. All Rights Reserved.
+ * SPDX-License-Identifier: Apache-2.0.
+ */
+
+public struct JSONErrorPayload: Decodable {
+    let message: String?
+    let errorType: String?
+}

Sources/WeatherSDK/AuthSchemeResolver.swift

@@ -0,0 +1,81 @@
+// Code generated by smithy-swift-codegen. DO NOT EDIT!
+
+import ClientRuntime
+
+public struct WeatherAuthSchemeResolverParameters: ClientRuntime.AuthSchemeResolverParameters {
+    public let operation: String
+    // Region is used for SigV4 auth scheme
+    public let region: String?
+}
+
+public protocol WeatherAuthSchemeResolver: ClientRuntime.AuthSchemeResolver {
+    // Intentionally empty.
+    // This is the parent protocol that all auth scheme resolver implementations of
+    // the service Weather must conform to.
+}
+
+public struct DefaultWeatherAuthSchemeResolver: WeatherAuthSchemeResolver {
+    public func resolveAuthScheme(params: ClientRuntime.AuthSchemeResolverParameters) throws -> [AuthOption] {
+        var validAuthOptions = [AuthOption]()
+        guard let serviceParams = params as? WeatherAuthSchemeResolverParameters else {
+            throw ClientError.authError("Service specific auth scheme parameters type must be passed to auth scheme resolver.")
+        }
+        switch serviceParams.operation {
+            case "onlyHttpApiKeyAuth":
+                validAuthOptions.append(AuthOption(schemeID: "smithy.api#httpApiKeyAuth"))
+            case "onlyHttpApiKeyAuthOptional":
+                validAuthOptions.append(AuthOption(schemeID: "smithy.api#httpApiKeyAuth"))
+                validAuthOptions.append(AuthOption(schemeID: "smithy.api#noAuth"))
+            case "onlyHttpBearerAuth":
+                validAuthOptions.append(AuthOption(schemeID: "smithy.api#httpBearerAuth"))
+            case "onlyHttpBearerAuthOptional":
+                validAuthOptions.append(AuthOption(schemeID: "smithy.api#httpBearerAuth"))
+                validAuthOptions.append(AuthOption(schemeID: "smithy.api#noAuth"))
+            case "onlyHttpApiKeyAndBearerAuth":
+                validAuthOptions.append(AuthOption(schemeID: "smithy.api#httpApiKeyAuth"))
+                validAuthOptions.append(AuthOption(schemeID: "smithy.api#httpBearerAuth"))
+            case "onlyHttpApiKeyAndBearerAuthReversed":
+                validAuthOptions.append(AuthOption(schemeID: "smithy.api#httpBearerAuth"))
+                validAuthOptions.append(AuthOption(schemeID: "smithy.api#httpApiKeyAuth"))
+            case "onlySigv4Auth":
+                var sigV4Option = AuthOption(schemeID: "aws.auth#sigv4")
+                sigV4Option.signingProperties.set(key: AttributeKeys.signingName, value: "weather")
+                guard let region = serviceParams.region else {
+                    throw ClientError.authError("Missing region in auth scheme parameters for SigV4 auth scheme.")
+                }
+                sigV4Option.signingProperties.set(key: AttributeKeys.signingRegion, value: region)
+                validAuthOptions.append(sigV4Option)
+            case "onlySigv4AuthOptional":
+                var sigV4Option = AuthOption(schemeID: "aws.auth#sigv4")
+                sigV4Option.signingProperties.set(key: AttributeKeys.signingName, value: "weather")
+                guard let region = serviceParams.region else {
+                    throw ClientError.authError("Missing region in auth scheme parameters for SigV4 auth scheme.")
+                }
+                sigV4Option.signingProperties.set(key: AttributeKeys.signingRegion, value: region)
+                validAuthOptions.append(sigV4Option)
+                validAuthOptions.append(AuthOption(schemeID: "smithy.api#noAuth"))
+            case "onlyFakeAuth":
+                validAuthOptions.append(AuthOption(schemeID: "common#fakeAuth"))
+            case "onlyFakeAuthOptional":
+                validAuthOptions.append(AuthOption(schemeID: "common#fakeAuth"))
+                validAuthOptions.append(AuthOption(schemeID: "smithy.api#noAuth"))
+            default:
+                var sigV4Option = AuthOption(schemeID: "aws.auth#sigv4")
+                sigV4Option.signingProperties.set(key: AttributeKeys.signingName, value: "weather")
+                guard let region = serviceParams.region else {
+                    throw ClientError.authError("Missing region in auth scheme parameters for SigV4 auth scheme.")
+                }
+                sigV4Option.signingProperties.set(key: AttributeKeys.signingRegion, value: region)
+                validAuthOptions.append(sigV4Option)
+        }
+        return validAuthOptions
+    }
+
+    public func constructParameters(context: HttpContext) throws -> ClientRuntime.AuthSchemeResolverParameters {
+        guard let opName = context.getOperation() else {
+            throw ClientError.dataNotFound("Operation name not configured in middleware context for auth scheme resolver params construction.")
+        }
+        let opRegion = context.getRegion()
+        return WeatherAuthSchemeResolverParameters(operation: opName, region: opRegion)
+    }
+}

Sources/WeatherSDK/Plugins.swift

@@ -0,0 +1,5 @@
+// Code generated by smithy-swift-codegen. DO NOT EDIT!
+
+import ClientRuntime
+
+

Sources/WeatherSDK/Waiters.swift

@@ -0,0 +1,6 @@
+// Code generated by smithy-swift-codegen. DO NOT EDIT!
+
+import ClientRuntime
+
+extension WeatherClient {
+}