Skip to content

Commit 5ea1e7d

Browse files
Trottr-52
authored andcommitted
test: fix redeclared vars in sequential tests
PR-URL: #4999 Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Roman Klauke <[email protected]>
1 parent cb51688 commit 5ea1e7d

File tree

3 files changed

+5
-5
lines changed

3 files changed

+5
-5
lines changed

‎test/sequential/test-child-process-execsync.js‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ var msgBuf = new Buffer(msg + '\n');
3838

3939
cmd=`"${process.execPath}" -e "console.log(\'${msg}\');"`;
4040

41-
varret=execSync(cmd);
41+
ret=execSync(cmd);
4242

4343
assert.strictEqual(ret.length,msgBuf.length);
4444
assert.deepEqual(ret,msgBuf,'execSync result buffer should match');

‎test/sequential/test-require-cache-without-stat.js‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,14 +35,14 @@ var counterBefore = counter;
3535

3636
// Now load the module a bunch of times with equivalent paths.
3737
// stat should not be called.
38-
for(vari=0;i<100;i++){
38+
for(leti=0;i<100;i++){
3939
require(common.fixturesDir+'/a');
4040
require('../fixtures/a.js');
4141
require('./../fixtures/a.js');
4242
}
4343

4444
// Do the same with a built-in module
45-
for(vari=0;i<100;i++){
45+
for(leti=0;i<100;i++){
4646
require('http');
4747
}
4848

‎test/sequential/test-tcp-wrap-listen.js‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -38,8 +38,8 @@ server.onconnection = function(err, client){
3838

3939
varreq=newWriteWrap();
4040
req.async=false;
41-
varerr=client.writeBuffer(req,buffer);
42-
assert.equal(err,0);
41+
constreturnCode=client.writeBuffer(req,buffer);
42+
assert.equal(returnCode,0);
4343
client.pendingWrites.push(req);
4444

4545
console.log('client.writeQueueSize: '+client.writeQueueSize);

0 commit comments

Comments
(0)