diff options
author | Bert JW Regeer <bertjw@regeer.org> | 2022-04-21 13:32:07 -0600 |
---|---|---|
committer | Bert JW Regeer <bertjw@regeer.org> | 2022-04-21 13:45:40 -0600 |
commit | 99f7135d32009cc1980db16a2aa3935a1a150277 (patch) | |
tree | 08eeffc74002a0cb67a0422dde029c15c06f012c | |
parent | 603d2c12ce09fcd1be0b7a5734b4fca339820286 (diff) | |
download | waitress-99f7135d32009cc1980db16a2aa3935a1a150277.tar.gz |
Add failing test for error_traceback
-rw-r--r-- | tests/fixtureapps/error_traceback.py | 2 | ||||
-rw-r--r-- | tests/test_functional.py | 54 |
2 files changed, 56 insertions, 0 deletions
diff --git a/tests/fixtureapps/error_traceback.py b/tests/fixtureapps/error_traceback.py new file mode 100644 index 0000000..24e4cbf --- /dev/null +++ b/tests/fixtureapps/error_traceback.py @@ -0,0 +1,2 @@ +def app(environ, start_response): # pragma: no cover + raise ValueError("Invalid application: " + chr(8364)) diff --git a/tests/test_functional.py b/tests/test_functional.py index 60eb24a..43e9d0c 100644 --- a/tests/test_functional.py +++ b/tests/test_functional.py @@ -1269,6 +1269,49 @@ class InternalServerErrorTests: self.assertRaises(ConnectionClosed, read_http, fp) +class InternalServerErrorTestsWithTraceback: + def setUp(self): + from tests.fixtureapps import error_traceback + + self.start_subprocess(error_traceback.app, expose_tracebacks=True) + + def tearDown(self): + self.stop_subprocess() + + def test_expose_tracebacks_http_10(self): + to_send = b"GET / HTTP/1.0\r\n\r\n" + self.connect() + self.sock.send(to_send) + with self.sock.makefile("rb", 0) as fp: + line, headers, response_body = read_http(fp) + self.assertline(line, "500", "Internal Server Error", "HTTP/1.0") + cl = int(headers["content-length"]) + self.assertEqual(cl, len(response_body)) + self.assertTrue(response_body.startswith(b"Internal Server Error")) + self.assertEqual(headers["connection"], "close") + # connection has been closed + self.send_check_error(to_send) + self.assertRaises(ConnectionClosed, read_http, fp) + + def test_expose_tracebacks_http_11(self): + to_send = b"GET / HTTP/1.1\r\n\r\n" + self.connect() + self.sock.send(to_send) + with self.sock.makefile("rb", 0) as fp: + line, headers, response_body = read_http(fp) + self.assertline(line, "500", "Internal Server Error", "HTTP/1.1") + cl = int(headers["content-length"]) + self.assertEqual(cl, len(response_body)) + self.assertTrue(response_body.startswith(b"Internal Server Error")) + self.assertEqual( + sorted(headers.keys()), + ["connection", "content-length", "content-type", "date", "server"], + ) + # connection has been closed + self.send_check_error(to_send) + self.assertRaises(ConnectionClosed, read_http, fp) + + class FileWrapperTests: def setUp(self): from tests.fixtureapps import filewrapper @@ -1538,6 +1581,12 @@ class TcpInternalServerErrorTests( pass +class TcpInternalServerErrorTestsWithTraceback( + InternalServerErrorTestsWithTraceback, TcpTests, unittest.TestCase +): + pass + + class TcpFileWrapperTests(FileWrapperTests, TcpTests, unittest.TestCase): pass @@ -1604,6 +1653,11 @@ if hasattr(socket, "AF_UNIX"): ): pass + class UnixInternalServerErrorTestsWithTraceback( + InternalServerErrorTestsWithTraceback, UnixTests, unittest.TestCase + ): + pass + class UnixFileWrapperTests(FileWrapperTests, UnixTests, unittest.TestCase): pass |