Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: enable transcribe concurrency tests on linux and windows #1825

Open
wants to merge 13 commits into
base: main
Choose a base branch
from
Original file line number Diff line number Diff line change
Expand Up @@ -23,16 +23,12 @@ final class TranscribeStreamingTests: XCTestCase {
// such as:
// code: 2058, message: "The connection has closed or is closing."
// code: 2087, message: "Stream acquisition failed because stream manager failed to acquire a connection"
//
// Disable this test on the CRT client until these failures can be investigated & corrected.
#if !os(Linux) && !os(Windows)
func test_25xConcurrent_streamTranscription() async throws {
// By default the TranscribeStreaming service allows 25 concurrent transcriptions.
// More than that (which can happen when multiple test runs are being performed) will result
// in throttling / resource exceeded errors, which may be retried (see retry logic below.)
try await repeatConcurrently(count: 25, test: attempt)
}
#endif

// MARK: - Private / implementation methods

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -45,6 +45,7 @@ public enum AWSRetryErrorInfoProvider: RetryErrorInfoProvider {
private static let transientCRTErrorCodes: [Int32] = [
2058, // httpConnectionClosed
2070, // httpServerClosed
2087, // AWS_ERROR_HTTP_STREAM_MANAGER_CONNECTION_ACQUIRE_FAILURE
]

// HTTP status codes that the Retry Behavior 2.0 specification say should be treated as transient errors.
Expand Down
Loading