Skip to content

Commit c82874c

Browse files
committed
Fix tests using CaptureRequestHandler to disable retries
1 parent 796622b commit c82874c

File tree

2 files changed

+11
-1
lines changed

2 files changed

+11
-1
lines changed

aws/codegen-aws-sdk/src/test/kotlin/software/amazon/smithy/rustsdk/HttpChecksumTest.kt

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -265,6 +265,7 @@ internal class HttpChecksumTest {
265265
.region(Region::from_static("doesntmatter"))
266266
.with_test_defaults()
267267
.http_client(http_client)
268+
.retry_config($moduleName::config::retry::RetryConfig::disabled())
268269
.build();
269270
270271
let client = $moduleName::Client::from_conf(config);
@@ -332,6 +333,7 @@ internal class HttpChecksumTest {
332333
.region(Region::from_static("doesntmatter"))
333334
.with_test_defaults()
334335
.http_client(http_client)
336+
.retry_config($moduleName::config::retry::RetryConfig::disabled())
335337
.build();
336338
337339
let client = $moduleName::Client::from_conf(config);
@@ -409,6 +411,7 @@ internal class HttpChecksumTest {
409411
.region(Region::from_static("doesntmatter"))
410412
.with_test_defaults()
411413
.http_client(http_client)
414+
.retry_config($moduleName::config::retry::RetryConfig::disabled())
412415
.build();
413416
414417
let client = $moduleName::Client::from_conf(config);
@@ -456,6 +459,7 @@ internal class HttpChecksumTest {
456459
.region(Region::from_static("doesntmatter"))
457460
.with_test_defaults()
458461
.http_client(http_client)
462+
.retry_config($moduleName::config::retry::RetryConfig::disabled())
459463
.build();
460464
461465
let client = $moduleName::Client::from_conf(config);
@@ -526,6 +530,7 @@ internal class HttpChecksumTest {
526530
.region(Region::from_static("doesntmatter"))
527531
.with_test_defaults()
528532
.http_client(http_client)
533+
.retry_config($moduleName::config::retry::RetryConfig::disabled())
529534
.build();
530535
531536
let client = $moduleName::Client::from_conf(config);
@@ -574,6 +579,7 @@ internal class HttpChecksumTest {
574579
.region(Region::from_static("doesntmatter"))
575580
.with_test_defaults()
576581
.http_client(http_client)
582+
.retry_config($moduleName::config::retry::RetryConfig::disabled())
577583
.request_checksum_calculation(
578584
aws_types::sdk_config::RequestChecksumCalculation::WhenRequired,
579585
)
@@ -604,6 +610,7 @@ internal class HttpChecksumTest {
604610
.region(Region::from_static("doesntmatter"))
605611
.with_test_defaults()
606612
.http_client(http_client)
613+
.retry_config($moduleName::config::retry::RetryConfig::disabled())
607614
.build();
608615
609616
let client = $moduleName::Client::from_conf(config);
@@ -636,6 +643,7 @@ internal class HttpChecksumTest {
636643
.region(Region::from_static("doesntmatter"))
637644
.with_test_defaults()
638645
.http_client(http_client)
646+
.retry_config($moduleName::config::retry::RetryConfig::disabled())
639647
.build();
640648
641649
let client = $moduleName::Client::from_conf(config);
@@ -668,6 +676,7 @@ internal class HttpChecksumTest {
668676
.region(Region::from_static("doesntmatter"))
669677
.with_test_defaults()
670678
.http_client(http_client)
679+
.retry_config($moduleName::config::retry::RetryConfig::disabled())
671680
.response_checksum_validation(
672681
aws_types::sdk_config::ResponseChecksumValidation::WhenRequired,
673682
)

aws/sdk/integration-tests/s3/tests/request_id.rs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55

66
use aws_sdk_s3::operation::get_object::GetObjectError;
77
use aws_sdk_s3::operation::{RequestId, RequestIdExt};
8-
use aws_sdk_s3::{config::Credentials, config::Region, Client, Config};
8+
use aws_sdk_s3::{config::{retry::RetryConfig, Credentials, Region}, Client, Config};
99
use aws_smithy_http_client::test_util::capture_request;
1010
use aws_smithy_types::body::SdkBody;
1111

@@ -81,6 +81,7 @@ async fn get_request_id_from_unmodeled_error() {
8181
.http_client(http_client)
8282
.credentials_provider(Credentials::for_tests())
8383
.region(Region::new("us-east-1"))
84+
.retry_config(RetryConfig::disabled())
8485
.build();
8586
let client = Client::from_conf(config);
8687
let err = client

0 commit comments

Comments
 (0)