test: refactor test-cluster-setup-master-argv

Oscar Martinez · addaleax · commit 346ea432b4e8 · 2016-12-08T03:02:54.000+01:00
- updated vars to const - add common.mustCall() to setup callback PR-URL: #9960 Reviewed-By: Prince John Wesley <princejohnwesley@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
diff --git a/test/parallel/test-cluster-setup-master-argv.js b/test/parallel/test-cluster-setup-master-argv.js
@@ -1,16 +1,16 @@
 'use strict';
-var common = require('../common');
-var assert = require('assert');
-var cluster = require('cluster');
+const common = require('../common');
+const assert = require('assert');
+const cluster = require('cluster');
 
 setTimeout(common.fail.bind(assert, 'setup not emitted'), 1000).unref();
 
-cluster.on('setup', function() {
+cluster.on('setup', common.mustCall(function() {
 var clusterArgs = cluster.settings.args;
 var realArgs = process.argv;
 assert.strictEqual(clusterArgs[clusterArgs.length - 1],
 realArgs[realArgs.length - 1]);
-});
+}));
 
 assert.notStrictEqual(process.argv[process.argv.length - 1], 'OMG,OMG');
 process.argv.push('OMG,OMG');

-Original file line number
+Diff line change
@@ @@ -1,16 +1,16 @@ @@
 'use strict';
 -varcommon=require('../common');
 -varassert=require('assert');
 -varcluster=require('cluster');
 +constcommon=require('../common');
 +constassert=require('assert');
 +constcluster=require('cluster');
 setTimeout(common.fail.bind(assert,'setup not emitted'),1000).unref();
 -cluster.on('setup',function(){
 +cluster.on('setup',common.mustCall(function(){
 varclusterArgs=cluster.settings.args;
 varrealArgs=process.argv;
 assert.strictEqual(clusterArgs[clusterArgs.length-1],
 realArgs[realArgs.length-1]);
 -});
 +}));
 assert.notStrictEqual(process.argv[process.argv.length-1],'OMG,OMG');
 process.argv.push('OMG,OMG');