Skip to content

Commit 80dda21

Browse files
committed
update to test changes
1 parent ebd56ee commit 80dda21

File tree

1 file changed

+9
-1
lines changed

1 file changed

+9
-1
lines changed

tests/aws-cpp-sdk-s3-integration-tests/BucketAndObjectOperationTest.cpp

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -152,6 +152,13 @@ namespace
152152
}
153153
}
154154

155+
class NoRetryStrategy: public Aws::Client::RetryStrategy
156+
{
157+
public:
158+
bool ShouldRetry(const AWSError<CoreErrors>&, long) const override { return false; }
159+
long CalculateDelayBeforeNextRetry(const AWSError<CoreErrors>&, long) const override { return 0; }
160+
};
161+
155162
class RetryFiveTimesRetryStrategy: public Aws::Client::RetryStrategy
156163
{
157164
public:
@@ -219,6 +226,7 @@ namespace
219226
config.writeRateLimiter = Limiter;
220227
config.executor = Aws::MakeShared<Aws::Utils::Threading::PooledThreadExecutor>(ALLOCATION_TAG, 4);
221228
config.enableHttpClientTrace = true;
229+
config.retryStrategy = Aws::MakeShared<NoRetryStrategy>(ALLOCATION_TAG);
222230

223231
//to use a proxy, uncomment the next two lines.
224232
if (USE_PROXY_FOR_TESTS)
@@ -240,7 +248,7 @@ namespace
240248
AWSAuthV4Signer::PayloadSigningPolicy::Never /*signPayloads*/, true /*useVirtualAddressing*/);
241249
m_HttpClient = Aws::Http::CreateHttpClient(config);
242250

243-
config.retryStrategy = Aws::MakeShared<RetryFiveTimesRetryStrategy>(ALLOCATION_TAG);
251+
// Keep retry strategy as NoRetryStrategy for retryClient too
244252
retryClient = Aws::MakeShared<S3TestClient>(ALLOCATION_TAG, ssoCredentials, config,
245253
AWSAuthV4Signer::PayloadSigningPolicy::Never /*signPayloads*/, true /*useVirtualAddressing*/);
246254
// Using client side monitoring for endpoint override testing.

0 commit comments

Comments
 (0)