From 77945d7f0fa6a0629ab7006d97d2e707e0431433 Mon Sep 17 00:00:00 2001 From: Timothy J Fontaine Date: Fri, 13 Jul 2012 15:11:38 -0400 Subject: [PATCH] add unref/ref tests for socket/server/timers --- test/simple/test-dgram-unref.js | 39 +++++++++++++++++ test/simple/test-net-connect-unref.js | 45 +++++++++++++++++++ test/simple/test-net-server-unref.js | 39 +++++++++++++++++ test/simple/test-pipe-unref.js | 39 +++++++++++++++++ test/simple/test-timers-unref.js | 62 +++++++++++++++++++++++++++ 5 files changed, 224 insertions(+) create mode 100644 test/simple/test-dgram-unref.js create mode 100644 test/simple/test-net-connect-unref.js create mode 100644 test/simple/test-net-server-unref.js create mode 100644 test/simple/test-pipe-unref.js create mode 100644 test/simple/test-timers-unref.js diff --git a/test/simple/test-dgram-unref.js b/test/simple/test-dgram-unref.js new file mode 100644 index 0000000000..38abbac274 --- /dev/null +++ b/test/simple/test-dgram-unref.js @@ -0,0 +1,39 @@ +// Copyright Joyent, Inc. and other Node contributors. +// +// Permission is hereby granted, free of charge, to any person obtaining a +// copy of this software and associated documentation files (the +// "Software"), to deal in the Software without restriction, including +// without limitation the rights to use, copy, modify, merge, publish, +// distribute, sublicense, and/or sell copies of the Software, and to permit +// persons to whom the Software is furnished to do so, subject to the +// following conditions: +// +// The above copyright notice and this permission notice shall be included +// in all copies or substantial portions of the Software. +// +// THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS +// OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF +// MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN +// NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, +// DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR +// OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE +// USE OR OTHER DEALINGS IN THE SOFTWARE. + +var common = require('../common'); +var assert = require('assert'); + +var dgram = require('dgram'); +var closed = false; + +var s = dgram.createSocket('udp4'); +s.bind(); +s.unref(); + +setTimeout(function() { + closed = true; + s.close(); +}, 1000).unref(); + +process.on('exit', function() { + assert.strictEqual(closed, false, 'Unrefd socket should not hold loop open'); +}); diff --git a/test/simple/test-net-connect-unref.js b/test/simple/test-net-connect-unref.js new file mode 100644 index 0000000000..29f43b10e4 --- /dev/null +++ b/test/simple/test-net-connect-unref.js @@ -0,0 +1,45 @@ +// Copyright Joyent, Inc. and other Node contributors. +// +// Permission is hereby granted, free of charge, to any person obtaining a +// copy of this software and associated documentation files (the +// "Software"), to deal in the Software without restriction, including +// without limitation the rights to use, copy, modify, merge, publish, +// distribute, sublicense, and/or sell copies of the Software, and to permit +// persons to whom the Software is furnished to do so, subject to the +// following conditions: +// +// The above copyright notice and this permission notice shall be included +// in all copies or substantial portions of the Software. +// +// THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS +// OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF +// MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN +// NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, +// DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR +// OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE +// USE OR OTHER DEALINGS IN THE SOFTWARE. + +var common = require('../common'); +var assert = require('assert'); +var net = require('net'); + +var client, killed = false, ended = false; +var TIMEOUT = 10 * 1000 + +client = net.createConnection(53, '8.8.8.8', function() { + client.unref(); +}); + +client.on('close', function() { + ended = true; +}); + +setTimeout(function() { + killed = true; + client.end(); +}, TIMEOUT).unref(); + +process.on('exit', function() { + assert.strictEqual(killed, false, 'A client should have connected'); + assert.strictEqual(ended, false, 'A client should stay connected'); +}); diff --git a/test/simple/test-net-server-unref.js b/test/simple/test-net-server-unref.js new file mode 100644 index 0000000000..d20cbe5068 --- /dev/null +++ b/test/simple/test-net-server-unref.js @@ -0,0 +1,39 @@ +// Copyright Joyent, Inc. and other Node contributors. +// +// Permission is hereby granted, free of charge, to any person obtaining a +// copy of this software and associated documentation files (the +// "Software"), to deal in the Software without restriction, including +// without limitation the rights to use, copy, modify, merge, publish, +// distribute, sublicense, and/or sell copies of the Software, and to permit +// persons to whom the Software is furnished to do so, subject to the +// following conditions: +// +// The above copyright notice and this permission notice shall be included +// in all copies or substantial portions of the Software. +// +// THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS +// OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF +// MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN +// NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, +// DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR +// OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE +// USE OR OTHER DEALINGS IN THE SOFTWARE. + +var common = require('../common'); +var assert = require('assert'); + +var net = require('net'); +var closed = false; + +var s = net.createServer(); +s.listen(); +s.unref(); + +setTimeout(function() { + closed = true; + s.close(); +}, 1000).unref(); + +process.on('exit', function() { + assert.strictEqual(closed, false, 'Unrefd socket should not hold loop open'); +}); diff --git a/test/simple/test-pipe-unref.js b/test/simple/test-pipe-unref.js new file mode 100644 index 0000000000..9f715a9d17 --- /dev/null +++ b/test/simple/test-pipe-unref.js @@ -0,0 +1,39 @@ +// Copyright Joyent, Inc. and other Node contributors. +// +// Permission is hereby granted, free of charge, to any person obtaining a +// copy of this software and associated documentation files (the +// "Software"), to deal in the Software without restriction, including +// without limitation the rights to use, copy, modify, merge, publish, +// distribute, sublicense, and/or sell copies of the Software, and to permit +// persons to whom the Software is furnished to do so, subject to the +// following conditions: +// +// The above copyright notice and this permission notice shall be included +// in all copies or substantial portions of the Software. +// +// THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS +// OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF +// MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN +// NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, +// DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR +// OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE +// USE OR OTHER DEALINGS IN THE SOFTWARE. + +var common = require('../common'); +var assert = require('assert'); + +var net = require('net'); +var closed = false; + +var s = net.Server(); +s.listen(common.PIPE); +s.unref(); + +setTimeout(function() { + closed = true; + s.close(); +}, 1000).unref(); + +process.on('exit', function() { + assert.strictEqual(closed, false, 'Unrefd socket should not hold loop open'); +}); diff --git a/test/simple/test-timers-unref.js b/test/simple/test-timers-unref.js new file mode 100644 index 0000000000..4be557e506 --- /dev/null +++ b/test/simple/test-timers-unref.js @@ -0,0 +1,62 @@ +// Copyright Joyent, Inc. and other Node contributors. +// +// Permission is hereby granted, free of charge, to any person obtaining a +// copy of this software and associated documentation files (the +// "Software"), to deal in the Software without restriction, including +// without limitation the rights to use, copy, modify, merge, publish, +// distribute, sublicense, and/or sell copies of the Software, and to permit +// persons to whom the Software is furnished to do so, subject to the +// following conditions: +// +// The above copyright notice and this permission notice shall be included +// in all copies or substantial portions of the Software. +// +// THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS +// OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF +// MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN +// NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, +// DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR +// OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE +// USE OR OTHER DEALINGS IN THE SOFTWARE. + +var common = require('../common'); +var assert = require('assert'); + +var interval_fired = false, + timeout_fired = false, + unref_interval = false, + unref_timer = false, + interval, check_unref, checks = 0; + +var LONG_TIME = 10 * 1000; +var SHORT_TIME = 100; + +setInterval(function() { + interval_fired = true; +}, LONG_TIME).unref(); + +setTimeout(function() { + timeout_fired = true; +}, LONG_TIME).unref(); + +interval = setInterval(function() { + unref_interval = true; + clearInterval(interval); +}, SHORT_TIME).unref(); + +setTimeout(function() { + unref_timer = true; +}, SHORT_TIME).unref(); + +check_unref = setInterval(function() { + if (checks > 5 || (unref_interval && unref_timer)) + clearInterval(check_unref); + checks += 1; +}, 100); + +process.on('exit', function() { + assert.strictEqual(interval_fired, false, 'Interval should not fire'); + assert.strictEqual(timeout_fired, false, 'Timeout should not fire'); + assert.strictEqual(unref_timer, true, 'An unrefd timeout should still fire'); + assert.strictEqual(unref_interval, true, 'An unrefd interval should still fire'); +});