fix: default value for SQS's receive message (#2405)

* fix: default value for SQS's receive message

After #2300, `MessageAttributeNames` and `MessageSystemAttributeNames`
had `None` as default values, but `None` is not a valid value for those
fields. See:

https://boto3.amazonaws.com/v1/documentation/api/latest/reference/services/sqs/client/receive_message.html

This commit suggests adding `[]` as the new default value for those
fields following the error message below and the docs above.

Closes #2403

* Adds test to SQS's MessageAttributeNames and MessageSystemAttributeNames

* Fixes tests

* Removes `None` as mock's default

---------

Co-authored-by: Asif Saif Uddin {"Auvi":"অভি"} <auvipy@gmail.com>

Author: cuducos

kombu/transport/SQS.py

@@ -545,9 +545,10 @@ def _receive_message(
         client = self.sqs(queue=queue)
 
         message_system_attribute_names = self.get_message_attributes.get(
-            'MessageSystemAttributeNames')
+            'MessageSystemAttributeNames') or []
+
         message_attribute_names = self.get_message_attributes.get(
-            'MessageAttributeNames')
+            'MessageAttributeNames') or []
 
         params: dict[str, Any] = {
             'QueueUrl': q_url,
@@ -964,7 +965,7 @@ def get_message_attributes(self) -> dict[str, Any]:
 
         if fetch is None or isinstance(fetch, str):
             return {
-                'MessageAttributeNames': None,
+                'MessageAttributeNames': [],
                 'MessageSystemAttributeNames': [APPROXIMATE_RECEIVE_COUNT],
             }
 
@@ -988,7 +989,7 @@ def get_message_attributes(self) -> dict[str, Any]:
                 )
 
         return {
-            'MessageAttributeNames': sorted(message_attrs) if message_attrs else None,
+            'MessageAttributeNames': sorted(message_attrs) if message_attrs else [],
             'MessageSystemAttributeNames': (
                 sorted(message_system_attrs) if message_system_attrs else [APPROXIMATE_RECEIVE_COUNT]
             )

t/unit/transport/test_SQS.py

@@ -125,9 +125,11 @@ def receive_message(
         QueueUrl=None,
         MaxNumberOfMessages=1,
         WaitTimeSeconds=10,
-        MessageAttributeNames=None,
-        MessageSystemAttributeNames=None
+        MessageAttributeNames=[],
+        MessageSystemAttributeNames=[],
     ):
+        assert isinstance(MessageAttributeNames, (list, tuple))
+        assert isinstance(MessageSystemAttributeNames, (list, tuple))
         self._receive_messages_calls += 1
         for q in self._queues.values():
             if q.url == QueueUrl: