Skip to content

Commit 082cc8d

Browse files
TrottTrott
committed
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: nodejs#4408 Reviewed-By: James M Snell <jasnell@gmail.com>
1 parent 686a85f commit 082cc8d

313 files changed

Lines changed: 317 additions & 317 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

‎test/internet/test-dns-txt-sigsegv.js‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
'use strict';
2-
var common = require('../common');
2+
require('../common');
33
var assert = require('assert');
44
var dns = require('dns');
55

‎test/internet/test-dns.js‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
'use strict';
2-
var common = require('../common');
2+
require('../common');
33
var assert = require('assert'),
44
dns = require('dns'),
55
net = require('net'),

‎test/internet/test-http-dns-fail.js‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
* should trigger the error event after each attempt.
55
*/
66

7-
var common = require('../common');
7+
require('../common');
88
var assert = require('assert');
99
var http = require('http');
1010

‎test/internet/test-net-connect-timeout.js‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
// https://groups.google.com/forum/#!topic/nodejs/UE0ZbfLt6t8
44
// https://groups.google.com/forum/#!topic/nodejs-dev/jR7-5UDqXkw
55

6-
var common = require('../common');
6+
require('../common');
77
var net = require('net');
88
var assert = require('assert');
99

‎test/internet/test-net-connect-unref.js‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
'use strict';
2-
var common = require('../common');
2+
require('../common');
33
var assert = require('assert');
44
var net = require('net');
55

‎test/message/2100bytes.js‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
'use strict';
2-
var common = require('../common');
2+
require('../common');
33
var assert = require('assert');
44
var util = require('util');
55

‎test/message/core_line_numbers.js‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
'use strict';
2-
const common = require('../common');
2+
require('../common');
33
const punycode = require('punycode');
44

55
// This test verifies that line numbers in core modules are reported correctly.

‎test/message/error_exit.js‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
'use strict';
2-
var common = require('../common');
2+
require('../common');
33
var assert = require('assert');
44

55
process.on('exit', function(code) {

‎test/message/eval_messages.js‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
'use strict';
22

3-
var common = require('../common');
3+
require('../common');
44
var assert = require('assert');
55

66
var spawn = require('child_process').spawn;

‎test/message/hello_world.js‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
'use strict';
2-
var common = require('../common');
2+
require('../common');
33
var assert = require('assert');
44

55
console.log('hello world');

0 commit comments

Comments
 (0)