node/test/parallel/test-net-server-connections.js

'use strict';
require('../common');
var assert = require('assert');

var net = require('net');

// test that server.connections property is no longer enumerable now that it
// has been marked as deprecated

var server = new net.Server();

assert.equal(Object.keys(server).indexOf('connections'), -1);
test: enable linting for tests Enable linting for the test directory. A number of changes was made so all tests conform the current rules used by lib and src directories. The only exception for tests is that unreachable (dead) code is allowed. test-fs-non-number-arguments-throw had to be excluded from the changes because of a weird issue on Windows CI. PR-URL: https://github.com/nodejs/io.js/pull/1721 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> 10 years ago			`'use strict';`
test: remove unnecessary assignments common.js needs to be loaded in all tests so that there is checking for variable leaks and possibly other things. However, it does not need to be assigned to a variable if nothing in common.js is referred to elsewhere in the test. PR-URL: https://github.com/nodejs/node/pull/4408 Reviewed-By: James M Snell <jasnell@gmail.com> 9 years ago			`require('../common');`
net: Make server.connections un-enumerable The property server.connections should no longer be enumerable because it has been deprecated. This will prevent deprecation warnings when server objects are accessed by functions such as JSON.stringify. Fixes: https://github.com/joyent/node/issues/8373 Reviewed-by: Trevor Norris <trev.norris@gmail.com> 10 years ago			`var assert = require('assert');`

			`var net = require('net');`

			`// test that server.connections property is no longer enumerable now that it`
			`// has been marked as deprecated`

			`var server = new net.Server();`

			`assert.equal(Object.keys(server).indexOf('connections'), -1);`