Motivation

The test test_reserved_concurrency_async_queue is currently skipped due to flakiness.

Changes

• Add aws_client_no_retry fixture and use it in the testcase
• Re-design test case to use SQS-based notifications instead of unpredictable sleep
• Add requests_id-based assertions

Testing

Temporarily using a botocore configuration with increased retries makes the retry problem reproducible causing the test to fail. This shows that we need an aws_client_no_retry to mitigate potential flakiness and reduce unnecessary retries.

To reproduce:

1. Ensure that retries are not disabled by configuration: TEST_DISABLE_RETRIES_AND_TIMEOUTS=0
2. Use the following config:

retry_delay_config = botocore.config.Config(retries={"max_attempts": 10})
retry_delay_client = aws_client_factory(config=retry_delay_config)
with pytest.raises(aws_client.lambda_.exceptions.TooManyRequestsException) as e:
    retry_delay_client.lambda_.invoke(
        FunctionName=fn_arn, InvocationType="RequestResponse"
    )