Fix response namespace parsing

.github/workflows/test310.yml

@@ -38,3 +38,8 @@ jobs:
     - name: Run tests
       run: |
         make test
+
+    - name: Run linters
+      run: |
+        poetry run pre-commit install
+        poetry run pre-commit run --all-files

.github/workflows/test311.yml

@@ -38,3 +38,8 @@ jobs:
     - name: Run tests
       run: |
         make test
+
+    - name: Run linters
+      run: |
+        poetry run pre-commit install
+        poetry run pre-commit run --all-files

.github/workflows/test312.yml

@@ -38,3 +38,8 @@ jobs:
     - name: Run tests
       run: |
         make test
+
+    - name: Run linters
+      run: |
+        poetry run pre-commit install
+        poetry run pre-commit run --all-files

.github/workflows/test38.yml

@@ -38,3 +38,8 @@ jobs:
     - name: Run tests
       run: |
         make test
+
+    - name: Run linters
+      run: |
+        poetry run pre-commit install
+        poetry run pre-commit run --all-files

.github/workflows/test39.yml

@@ -38,3 +38,8 @@ jobs:
     - name: Run tests
       run: |
         make test
+
+    - name: Run linters
+      run: |
+        poetry run pre-commit install
+        poetry run pre-commit run --all-files

MAINTAIN.md

@@ -5,6 +5,7 @@ Set up environment:
 python3 -m pip install poetry==1.5.0
 poetry env use 3.11
 poetry install
+poetry run pre-commit install
 ```
 
 Run tests:

README.md

@@ -82,13 +82,3 @@ Close the client at the end:
 ```python
 await client.close()
 ```
-
-
-## Development
-
-Set up env and install dependencies:
-```shell
-python3 -m pip install poetry==1.5.0
-poetry env use 3.11
-poetry install
-```

aiosqs/parser.py

@@ -70,7 +70,7 @@ def parse_xml_result_response(action: str, body: str, logger: Optional[LoggerTyp
         )
 
     # Find if it's a single element or a list
-    xpath = f"./{action}Result"
+    xpath = f"./*[local-name() = '{action}Result']"
     elements = collect_elements(root=root, xpath=xpath)
 
     # Response is a list of objects of the same type

aiosqs/tests/fixtures/get_queue_url_namespace.xml

@@ -0,0 +1,10 @@
+<GetQueueUrlResponse xmlns="http://queue.amazonaws.com/doc/2012-11-05/">
+    <GetQueueUrlResult>
+        <QueueUrl>
+            http://sqs.mcs.mail.ru/account123/example_queue
+        </QueueUrl>
+    </GetQueueUrlResult>
+    <ResponseMetadata>
+        <RequestId/>
+    </ResponseMetadata>
+</GetQueueUrlResponse>

aiosqs/tests/test_get_queue_url.py

@@ -18,3 +18,15 @@ def test_get_queue_url(self):
                 "QueueUrl": "http://sqs.mcs.mail.ru/account123/example_queue",
             },
         )
+
+    def test_response_with_namespace(self):
+        res = parse_xml_result_response(
+            action=self.action,
+            body=load_fixture("get_queue_url_namespace.xml"),
+        )
+        self.assertEqual(
+            res,
+            {
+                "QueueUrl": "http://sqs.mcs.mail.ru/account123/example_queue",
+            },
+        )

pyproject.toml

@@ -57,6 +57,7 @@ lxml = "^4"
 [tool.poetry.group.dev.dependencies]
 nose2 = "0.12.0"
 freezegun = "1.2.2"
+pre-commit = "2.17.0"
 
 [tool.black]
 line-length = 140