diff options
author | Bert Belder <bertbelder@gmail.com> | 2012-06-08 03:10:35 +0200 |
---|---|---|
committer | Bert Belder <bertbelder@gmail.com> | 2012-06-08 03:11:53 +0200 |
commit | b5a15f5cdf4679775bd9e6baf0568d97dd588170 (patch) | |
tree | cb6ea62e9615a87a8f0b4995056ec32ba8a2412d /test/test-tcp-shutdown-after-write.c | |
parent | 23b4e38d206dff24de67470493e31ca06b18be29 (diff) | |
download | libuv-b5a15f5cdf4679775bd9e6baf0568d97dd588170.tar.gz libuv-b5a15f5cdf4679775bd9e6baf0568d97dd588170.zip |
test: fix test-tcp-shutdown-after-write bug
It was calling uv_read_start before the uv_connect call had completed.
Although we want to allow this in the future, right now it's not
supported.
Diffstat (limited to 'test/test-tcp-shutdown-after-write.c')
-rw-r--r-- | test/test-tcp-shutdown-after-write.c | 14 |
1 files changed, 8 insertions, 6 deletions
diff --git a/test/test-tcp-shutdown-after-write.c b/test/test-tcp-shutdown-after-write.c index 8bf91236..219a3b40 100644 --- a/test/test-tcp-shutdown-after-write.c +++ b/test/test-tcp-shutdown-after-write.c @@ -70,13 +70,18 @@ static void timer_cb(uv_timer_t* handle, int status) { } +static void read_cb(uv_stream_t* handle, ssize_t nread, uv_buf_t buf) { +} + + static void connect_cb(uv_connect_t* req, int status) { + int r; + ASSERT(status == 0); connect_cb_called++; -} - -static void read_cb(uv_stream_t* handle, ssize_t nread, uv_buf_t buf) { + r = uv_read_start((uv_stream_t*)&conn, alloc_cb, read_cb); + ASSERT(r == 0); } @@ -113,9 +118,6 @@ TEST_IMPL(tcp_shutdown_after_write) { r = uv_tcp_connect(&connect_req, &conn, addr, connect_cb); ASSERT(r == 0); - r = uv_read_start((uv_stream_t*)&conn, alloc_cb, read_cb); - ASSERT(r == 0); - r = uv_run(loop); ASSERT(r == 0); |