aboutsummaryrefslogtreecommitdiff
path: root/tests/test_sockets.py
diff options
context:
space:
mode:
authorFraser Adams <fraser.adams@blueyonder.co.uk>2013-11-18 19:28:21 +0000
committerFraser Adams <fraser.adams@blueyonder.co.uk>2013-11-18 19:28:21 +0000
commit5e49fbbfdcca308b7f9d747fc66d3c92c03ac241 (patch)
tree2c92453871495a66c9a387793eeb3a749a2335b8 /tests/test_sockets.py
parent047280d98decd0c4071b0a044461569b84193023 (diff)
fix a bug in the accept call in library.js whereby if accept was called with non-NULL addr and addrlen parameters a ReferenceError occurs because accept had a parameter of addrp but later used addr. Modified tests_sockets_echo_server.c to use non-NULL addr if TEST_ACCEPT_ADDR is defined and added test case to test_sockets.py
Diffstat (limited to 'tests/test_sockets.py')
-rw-r--r--tests/test_sockets.py4
1 files changed, 3 insertions, 1 deletions
diff --git a/tests/test_sockets.py b/tests/test_sockets.py
index e1caa150..1229aa70 100644
--- a/tests/test_sockets.py
+++ b/tests/test_sockets.py
@@ -235,7 +235,9 @@ class sockets(BrowserCore):
harnesses = [
(WebsockifyServerHarness(os.path.join('sockets', 'test_sockets_echo_server.c'), [sockets_include], 49160), 0),
(CompiledServerHarness(os.path.join('sockets', 'test_sockets_echo_server.c'), [sockets_include, '-DTEST_DGRAM=0'], 49161), 0),
- (CompiledServerHarness(os.path.join('sockets', 'test_sockets_echo_server.c'), [sockets_include, '-DTEST_DGRAM=1'], 49162), 1)
+ (CompiledServerHarness(os.path.join('sockets', 'test_sockets_echo_server.c'), [sockets_include, '-DTEST_DGRAM=1'], 49162), 1),
+ # The following forces non-NULL addr and addlen parameters for the accept call
+ (CompiledServerHarness(os.path.join('sockets', 'test_sockets_echo_server.c'), [sockets_include, '-DTEST_DGRAM=0', '-DTEST_ACCEPT_ADDR=1'], 49163), 0)
]
for harness, datagram in harnesses: