test: fix redeclared vars in test-vm-*

PR-URL: https://github.com/nodejs/node/pull/4997
Reviewed-By: Michaël Zasso <mic.besace@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

diff --git a/test/parallel/test-vm-basic.js b/test/parallel/test-vm-basic.js
index 8f9cab90eb5a1b9196bfae895a0e9cbddb764eb8..f08c8383b84a18ccf226adcaacb9c02f6f472c2d 100644 (file)
--- a/test/parallel/test-vm-basic.js
+++ b/test/parallel/test-vm-basic.js
@@ -18,7 +18,7 @@ assert.strictEqual(result, 'function');
 // Test 2: vm.runInContext
 var sandbox2 = { foo: 'bar' };
 var context = vm.createContext(sandbox2);
-var result = vm.runInContext(
+result = vm.runInContext(
   'baz = foo; this.typeofProcess = typeof process; typeof Object;',
   context
 );
@@ -30,7 +30,7 @@ assert.deepEqual(sandbox2, {
 assert.strictEqual(result, 'function');
 
 // Test 3: vm.runInThisContext
-var result = vm.runInThisContext(
+result = vm.runInThisContext(
   'vmResult = "foo"; Object.prototype.toString.call(process);'
 );
 assert.strictEqual(global.vmResult, 'foo');
@@ -38,7 +38,7 @@ assert.strictEqual(result, '[object process]');
 delete global.vmResult;
 
 // Test 4: vm.runInNewContext
-var result = vm.runInNewContext(
+result = vm.runInNewContext(
   'vmResult = "foo"; typeof process;'
 );
 assert.strictEqual(global.vmResult, undefined);

diff --git a/test/parallel/test-vm-debug-context.js b/test/parallel/test-vm-debug-context.js
index b648592eaea901ad3c7eab3f5a6be8fcbd3f59b9..7789ddc83fc18e6d2b18f48b3b51c120dca87bb0 100644 (file)
--- a/test/parallel/test-vm-debug-context.js
+++ b/test/parallel/test-vm-debug-context.js
@@ -87,7 +87,7 @@ proc.once('exit', common.mustCall(function(exitCode, signalCode) {
   assert.equal(signalCode, null);
 }));
 
-var proc = spawn(process.execPath, [script, 'handle-fatal-exception']);
+proc = spawn(process.execPath, [script, 'handle-fatal-exception']);
 proc.stdout.on('data', common.fail);
 proc.stderr.on('data', common.fail);
 proc.once('exit', common.mustCall(function(exitCode, signalCode) {

diff --git a/test/parallel/test-vm-harmony-proxies.js b/test/parallel/test-vm-harmony-proxies.js
index 78ee144582741a490f802a13fc68112e8f89d582..71d9faffd9cedf1f7e04485837512e0af091936c 100644 (file)
--- a/test/parallel/test-vm-harmony-proxies.js
+++ b/test/parallel/test-vm-harmony-proxies.js
@@ -13,7 +13,7 @@ assert(typeof sandbox.Proxy === 'object');
 assert(sandbox.Proxy !== Proxy);
 
 // Unless we copy the Proxy object explicitly, of course.
-var sandbox = { Proxy: Proxy };
+sandbox = { Proxy: Proxy };
 vm.runInNewContext('this.Proxy = Proxy', sandbox);
 assert(typeof sandbox.Proxy === 'object');
 assert(sandbox.Proxy === Proxy);

diff --git a/test/parallel/test-vm-harmony-symbols.js b/test/parallel/test-vm-harmony-symbols.js
index e4b27c9ebb857969b5a43c8217962ef0b5e5d6d7..456a28f4d54215dce5112a059d94801d2c2c1f71 100644 (file)
--- a/test/parallel/test-vm-harmony-symbols.js
+++ b/test/parallel/test-vm-harmony-symbols.js
@@ -10,7 +10,7 @@ assert(typeof sandbox.Symbol === 'function');
 assert(sandbox.Symbol !== Symbol);
 
 // Unless we copy the Symbol constructor explicitly, of course.
-var sandbox = { Symbol: Symbol };
+sandbox = { Symbol: Symbol };
 vm.runInNewContext('this.Symbol = Symbol', sandbox);
 assert(typeof sandbox.Symbol === 'function');
 assert(sandbox.Symbol === Symbol);