From 3868c174b7f91f4c81ab384b70f1821c8177ef3a Mon Sep 17 00:00:00 2001 From: Josh Elkins Date: Thu, 5 Sep 2024 13:53:50 -0500 Subject: [PATCH] chore: Fix compiler warnings --- Sources/SmithyHTTPAuth/SigV4Signer.swift | 2 +- .../DefaultRetryStrategy/DefaultRetryStrategyTests.swift | 2 +- .../ExponentialBackoffStrategyTests.swift | 8 ++++---- 3 files changed, 6 insertions(+), 6 deletions(-) diff --git a/Sources/SmithyHTTPAuth/SigV4Signer.swift b/Sources/SmithyHTTPAuth/SigV4Signer.swift index 3146ac27d..65cd4a9b5 100644 --- a/Sources/SmithyHTTPAuth/SigV4Signer.swift +++ b/Sources/SmithyHTTPAuth/SigV4Signer.swift @@ -52,7 +52,7 @@ public class SigV4Signer: SmithyHTTPAuthAPI.Signer { ) } - var signingConfig = try constructSigningConfig(identity: identity, signingProperties: signingProperties) + let signingConfig = try constructSigningConfig(identity: identity, signingProperties: signingProperties) let unsignedRequest = requestBuilder.build() let crtUnsignedRequest: HTTPRequestBase = isBidirectionalStreamingEnabled ? diff --git a/Tests/SmithyRetriesTests/DefaultRetryStrategy/DefaultRetryStrategyTests.swift b/Tests/SmithyRetriesTests/DefaultRetryStrategy/DefaultRetryStrategyTests.swift index fa930ddb9..963619d98 100644 --- a/Tests/SmithyRetriesTests/DefaultRetryStrategy/DefaultRetryStrategyTests.swift +++ b/Tests/SmithyRetriesTests/DefaultRetryStrategy/DefaultRetryStrategyTests.swift @@ -28,7 +28,7 @@ final class DefaultRetryStrategyTests: XCTestCase { override func setUp() { backoffStrategy = .init() - backoffStrategy.random = { 1.0 } + backoffStrategy.random = { @Sendable () -> Double in 1.0 } options = RetryStrategyOptions(backoffStrategy: backoffStrategy, maxRetriesBase: 2) subject = DefaultRetryStrategy(options: options) mockSleeper = { self.actualDelay = $0 } diff --git a/Tests/SmithyRetriesTests/DefaultRetryStrategy/ExponentialBackoffStrategyTests.swift b/Tests/SmithyRetriesTests/DefaultRetryStrategy/ExponentialBackoffStrategyTests.swift index 8a05f052f..bd92bcfbc 100644 --- a/Tests/SmithyRetriesTests/DefaultRetryStrategy/ExponentialBackoffStrategyTests.swift +++ b/Tests/SmithyRetriesTests/DefaultRetryStrategy/ExponentialBackoffStrategyTests.swift @@ -16,15 +16,15 @@ final class ExponentialBackoffStrategyTests: XCTestCase { override func setUp() { subject = ExponentialBackoffStrategy() // Randomization is disabled to allow easy, repeatable verification of basic behavior. - subject.random = { 1.0 } + subject.random = { @Sendable () -> Double in 1.0 } } func test_backoffStrategy_multipliesByBackoffFactor() { - subject.random = { 0.25 } + subject.random = { @Sendable () -> Double in 0.25 } XCTAssertEqual(subject.computeNextBackoffDelay(attempt: 0), 0.25) - subject.random = { 0.5 } + subject.random = { @Sendable () -> Double in 0.5 } XCTAssertEqual(subject.computeNextBackoffDelay(attempt: 0), 0.5) - subject.random = { 0.75 } + subject.random = { @Sendable () -> Double in 0.75 } XCTAssertEqual(subject.computeNextBackoffDelay(attempt: 0), 0.75) }