Merge pull request #1 from sjaensch/fix-text-response-unmarshalling

Fix unmarshalling of non-JSON/msgpack responses
sjaensch · Jan 11, 2018 · a239ff7 · a239ff7
2 parents c3e8075 + ddf29f0
commit a239ff7
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 2 deletions.
diff --git a/aiobravado/http_future.py b/aiobravado/http_future.py
@@ -206,7 +206,7 @@ async def unmarshal_response_inner(response, op):
         )
 
     # TODO: Non-json response contents
-    return response.text
+    return await response.text
 
 
 def raise_on_unexpected(http_response):

diff --git a/changelog.rst b/changelog.rst
@@ -4,6 +4,7 @@ Changelog
 0.9.1 (2018-XX-XX)
 ------------------
 - bravado version: 9.2.2
+- Fix unmarshalling of non-JSON/msgpack responses.
 
 0.9.0 (2017-11-17)
 ------------------

diff --git a/tests/http_future/unmarshall_response_inner_test.py b/tests/http_future/unmarshall_response_inner_test.py
@@ -83,7 +83,7 @@ def test_text_content(mock_get_response_spec, empty_swagger_spec, response_spec,
         spec=IncomingResponse,
         status_code=200,
         headers={'content-type': 'text/plain'},
-        text='Monday',
+        text=coroutine(lambda: 'Monday')(),
     )
 
     mock_get_response_spec.return_value = response_spec