diff options
author | Andrey Zelenkov <zelenkov@nginx.com> | 2018-04-26 17:37:24 +0300 |
---|---|---|
committer | Andrey Zelenkov <zelenkov@nginx.com> | 2018-04-26 17:37:24 +0300 |
commit | d9f8479375fb6231db26e53a6549f2b96108faee (patch) | |
tree | 51f2f782db679cd454260fc7da02db337386e8f2 | |
parent | a44d358f2949e420dc82b530e07f21ae158f00ae (diff) | |
download | unit-d9f8479375fb6231db26e53a6549f2b96108faee.tar.gz unit-d9f8479375fb6231db26e53a6549f2b96108faee.tar.bz2 |
Tests: response handling improved.
-rw-r--r-- | test/test_access_log.py | 10 | ||||
-rw-r--r-- | test/test_python_application.py | 2 | ||||
-rw-r--r-- | test/unit.py | 12 |
3 files changed, 14 insertions, 10 deletions
diff --git a/test/test_access_log.py b/test/test_access_log.py index a2b3287e..bee7a7e5 100644 --- a/test/test_access_log.py +++ b/test/test_access_log.py @@ -167,7 +167,7 @@ Connection: close def test_access_log_partial(self): self.load('empty') - self.http(b"""GE""", raw_resp=True, raw=True) + self.http(b"""GE""", raw=True) time.sleep(0.2) @@ -179,7 +179,7 @@ Connection: close def test_access_log_partial_2(self): self.load('empty') - self.http(b"""GET /\n""", raw_resp=True, raw=True) + self.http(b"""GET /\n""", raw=True) time.sleep(0.2) @@ -191,7 +191,7 @@ Connection: close def test_access_log_partial_3(self): self.load('empty') - self.http(b"""GET / HTTP/1.1""", raw_resp=True, raw=True) + self.http(b"""GET / HTTP/1.1""", raw=True) time.sleep(0.2) @@ -203,7 +203,7 @@ Connection: close def test_access_log_partial_4(self): self.load('empty') - resp = self.http(b"""GET / HTTP/1.1\n""", raw_resp=True, raw=True) + resp = self.http(b"""GET / HTTP/1.1\n""", raw=True) time.sleep(0.2) @@ -216,7 +216,7 @@ Connection: close def test_access_log_partial_5(self): self.load('empty') - self.http(b"""GET / HTTP/1.1\n\n""", raw_resp=True, raw=True) + self.http(b"""GET / HTTP/1.1\n\n""", raw=True) self.stop() diff --git a/test/test_python_application.py b/test/test_python_application.py index 1a6560e5..d7913a1d 100644 --- a/test/test_python_application.py +++ b/test/test_python_application.py @@ -230,7 +230,7 @@ class TestUnitPythonApplication(unit.TestUnitApplicationPython): def test_python_application_not_iterable(self): self.load('not_iterable') - self.get(raw_resp=True) + self.get() self.stop() diff --git a/test/unit.py b/test/unit.py index 90906ba8..e1bf2374 100644 --- a/test/unit.py +++ b/test/unit.py @@ -259,17 +259,21 @@ class TestUnitHTTP(TestUnit): return self.http('PUT', **kwargs) def _recvall(self, sock, buff_size=4096): - data = '' + data = b'' while select.select([sock], [], [], 1)[0]: - part = sock.recv(buff_size).decode() + part = sock.recv(buff_size) data += part - if part is '': + if not len(part): break - return data + return data.decode() def _resp_to_dict(self, resp): m = re.search('(.*?\x0d\x0a?)\x0d\x0a?(.*)', resp, re.M | re.S) + + if not m: + return {} + headers_text, body = m.group(1), m.group(2) p = re.compile('(.*?)\x0d\x0a?', re.M | re.S) |