SQS: delete_message_batch() now validates there's at least one entry (#…

…8062)

moto/sqs/exceptions.py

@@ -38,10 +38,10 @@ def __init__(self, message: str):
 class EmptyBatchRequest(RESTError):
     code = 400
 
-    def __init__(self) -> None:
+    def __init__(self, action: str = "Send") -> None:
         super().__init__(
-            "EmptyBatchRequest",
-            "There should be at least one SendMessageBatchRequestEntry in the request.",
+            "AWS.SimpleQueueService.EmptyBatchRequest",
+            f"There should be at least one {action}MessageBatchRequestEntry in the request.",
         )
 
 

moto/sqs/responses.py

@@ -489,6 +489,9 @@ def delete_message_batch(self) -> str:
         else:
             receipts = self._get_multi_param("DeleteMessageBatchRequestEntry")
 
+        if not receipts:
+            raise EmptyBatchRequest(action="Delete")
+
         for r in receipts:
             for key in list(r.keys()):
                 if key == "Id":

tests/test_sqs/test_sqs.py

@@ -1972,6 +1972,20 @@ def test_delete_message_batch_with_invalid_receipt_id():
     ]
 
 
+@sqs_aws_verified()
+@pytest.mark.aws_verified
+def test_delete_message_batch_with_zero_entries(queue_name=None, queue_url=None):
+    sqs = boto3.client("sqs", "us-east-1")
+    with pytest.raises(ClientError) as exc:
+        sqs.delete_message_batch(QueueUrl=queue_url, Entries=[])
+    err = exc.value.response["Error"]
+    assert err["Code"] == "AWS.SimpleQueueService.EmptyBatchRequest"
+    assert (
+        err["Message"]
+        == "There should be at least one DeleteMessageBatchRequestEntry in the request."
+    )
+
+
 @mock_aws
 def test_message_attributes_in_receive_message():
     sqs = boto3.resource("sqs", region_name=REGION)