Long poll for 20s instead of 18s

gregfurman · gregfurman · commit 340a96325015 · 2025-03-03T14:12:16.000+02:00
diff --git a/tests/aws/services/lambda_/event_source_mapping/test_lambda_integration_sqs.py b/tests/aws/services/lambda_/event_source_mapping/test_lambda_integration_sqs.py
@@ -530,7 +530,7 @@ def test_report_batch_item_failures(
     # now wait for the second invocation result which is expected to have processed message 2 and 3
     # Since we are re-queuing twice, with a visiblity timeout of 8s, this should instead be waiting for 20s => 8s x 2 retries (+ 4s margin).
     # See AWS docs: https://docs.aws.amazon.com/AWSSimpleQueueService/latest/APIReference/API_ReceiveMessage.html#API_ReceiveMessage_RequestSyntax
-    second_timeout_with_margin = (visibility_timeout * 2) + 2
+    second_timeout_with_margin = (visibility_timeout * 2) + 4
     assert second_timeout_with_margin <= 20, (
         "An SQS ReceiveMessage operation cannot wait for more than 20s"
     )

Original file line number	Diff line number	Diff line change
`@@ -530,7 +530,7 @@ def test_report_batch_item_failures(`
`530`	`530`	`# now wait for the second invocation result which is expected to have processed message 2 and 3`
`531`	`531`	`# Since we are re-queuing twice, with a visiblity timeout of 8s, this should instead be waiting for 20s => 8s x 2 retries (+ 4s margin).`
`532`	`532`	`# See AWS docs: https://docs.aws.amazon.com/AWSSimpleQueueService/latest/APIReference/API_ReceiveMessage.html#API_ReceiveMessage_RequestSyntax`
`533`		`- second_timeout_with_margin = (visibility_timeout * 2) + 2`
	`533`	`+ second_timeout_with_margin = (visibility_timeout * 2) + 4`
`534`	`534`	`assert second_timeout_with_margin <= 20, (`
`535`	`535`	`"An SQS ReceiveMessage operation cannot wait for more than 20s"`
`536`	`536`	`)`