add test for bad encoding alternative handling

aiohttp/client_reqrep.py

@@ -749,7 +749,7 @@ def text(self, encoding=None, errors='strict'):
         if encoding is None:
             encoding = self._get_encoding()
 
-        return self._content.decode(encoding, errors)
+        return self._content.decode(encoding, errors=errors)
 
     @asyncio.coroutine
     def json(self, *, encoding=None, loads=json.loads):

tests/test_client_response.py

@@ -139,6 +139,29 @@ def side_effect(*args, **kwargs):
     assert response._connection is None
 
 
+@asyncio.coroutine
+def test_text_bad_encoding(loop):
+    response = ClientResponse('get', URL('http://def-cl-resp.org'))
+    response._post_init(loop)
+
+    def side_effect(*args, **kwargs):
+        fut = helpers.create_future(loop)
+        fut.set_result('{"тестkey": "пройденvalue"}'.encode('cp1251'))
+        return fut
+
+    # lie about the encoding
+    response.headers = {
+        'Content-Type': 'application/json;charset=utf-8'}
+    content = response.content = mock.Mock()
+    content.read.side_effect = side_effect
+    with pytest.raises(UnicodeDecodeError):
+        yield from response.text()
+    # only the valid utf-8 characters will be returned
+    res = yield from response.text(errors='ignore')
+    assert res == '{"key": "value"}'
+    assert response._connection is None
+
+
 @asyncio.coroutine
 def test_text_custom_encoding(loop):
     response = ClientResponse('get', URL('http://def-cl-resp.org'))