test: refactor test-fs-read-stream-inherit.js

* convert assert.equal to assert.strictEqual * convert var to const/let PR-URL: #9894 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: James M Snell <[email protected]>

Author: Jonathan Darling

test/parallel/test-fs-read-stream-inherit.js

@@ -1,22 +1,22 @@
 'use strict';
-varcommon=require('../common');
-varassert=require('assert');
+constcommon=require('../common');
+constassert=require('assert');
 
-varpath=require('path');
-varfs=require('fs');
-varfn=path.join(common.fixturesDir,'elipses.txt');
-varrangeFile=path.join(common.fixturesDir,'x.txt');
+constpath=require('path');
+constfs=require('fs');
+constfn=path.join(common.fixturesDir,'elipses.txt');
+constrangeFile=path.join(common.fixturesDir,'x.txt');
 
-varcallbacks={open: 0,end: 0,close: 0};
+constcallbacks={open: 0,end: 0,close: 0};
 
-varpaused=false;
+letpaused=false;
 
-varfile=fs.ReadStream(fn);
+constfile=fs.ReadStream(fn);
 
 file.on('open',function(fd){
 file.length=0;
 callbacks.open++;
-assert.equal('number',typeoffd);
+assert.strictEqual(typeoffd,'number');
 assert.ok(file.readable);
 
 // GH-535
@@ -48,19 +48,17 @@ file.on('end', function(chunk){
 
 file.on('close',function(){
 callbacks.close++;
-
-//assert.equal(fs.readFileSync(fn), fileContent);
 });
 
-varfile3=fs.createReadStream(fn,Object.create({encoding: 'utf8'}));
+constfile3=fs.createReadStream(fn,Object.create({encoding: 'utf8'}));
 file3.length=0;
 file3.on('data',function(data){
-assert.equal('string',typeofdata);
+assert.strictEqual(typeofdata,'string');
 file3.length+=data.length;
 
-for(vari=0;i<data.length;i++){
+for(leti=0;i<data.length;i++){
 // http://www.fileformat.info/info/unicode/char/2026/index.htm
-assert.equal('\u2026',data[i]);
+assert.strictEqual(data[i],'\u2026');
 }
 });
 
@@ -69,74 +67,74 @@ file3.on('close', function(){
 });
 
 process.on('exit',function(){
-assert.equal(1,callbacks.open);
-assert.equal(1,callbacks.end);
-assert.equal(2,callbacks.close);
-assert.equal(30000,file.length);
-assert.equal(10000,file3.length);
+assert.strictEqual(callbacks.open,1);
+assert.strictEqual(callbacks.end,1);
+assert.strictEqual(callbacks.close,2);
+assert.strictEqual(file.length,30000);
+assert.strictEqual(file3.length,10000);
 console.error('ok');
 });
 
-varfile4=fs.createReadStream(rangeFile,Object.create({bufferSize: 1,
-start: 1,end: 2}));
-assert.equal(file4.start,1);
-assert.equal(file4.end,2);
-varcontentRead='';
+constfile4=fs.createReadStream(rangeFile,Object.create({bufferSize: 1,
+start: 1,end: 2}));
+assert.strictEqual(file4.start,1);
+assert.strictEqual(file4.end,2);
+letcontentRead='';
 file4.on('data',function(data){
 contentRead+=data.toString('utf-8');
 });
 file4.on('end',function(data){
-assert.equal(contentRead,'yz');
+assert.strictEqual(contentRead,'yz');
 });
 
-varfile5=fs.createReadStream(rangeFile,Object.create({bufferSize: 1,
-start: 1}));
-assert.equal(file5.start,1);
+constfile5=fs.createReadStream(rangeFile,Object.create({bufferSize: 1,
+start: 1}));
+assert.strictEqual(file5.start,1);
 file5.data='';
 file5.on('data',function(data){
 file5.data+=data.toString('utf-8');
 });
 file5.on('end',function(){
-assert.equal(file5.data,'yz\n');
+assert.strictEqual(file5.data,'yz\n');
 });
 
 // https://github.com/joyent/node/issues/2320
-varfile6=fs.createReadStream(rangeFile,Object.create({bufferSize: 1.23,
-start: 1}));
-assert.equal(file6.start,1);
+constfile6=fs.createReadStream(rangeFile,Object.create({bufferSize: 1.23,
+start: 1}));
+assert.strictEqual(file6.start,1);
 file6.data='';
 file6.on('data',function(data){
 file6.data+=data.toString('utf-8');
 });
 file6.on('end',function(){
-assert.equal(file6.data,'yz\n');
+assert.strictEqual(file6.data,'yz\n');
 });
 
 assert.throws(function(){
 fs.createReadStream(rangeFile,Object.create({start: 10,end: 2}));
 },/"start"optionmustbe<="end"option/);
 
-varstream=fs.createReadStream(rangeFile,Object.create({start: 0,
-end: 0}));
-assert.equal(stream.start,0);
-assert.equal(stream.end,0);
+conststream=fs.createReadStream(rangeFile,Object.create({start: 0,
+end: 0}));
+assert.strictEqual(stream.start,0);
+assert.strictEqual(stream.end,0);
 stream.data='';
 
 stream.on('data',function(chunk){
 stream.data+=chunk;
 });
 
 stream.on('end',function(){
-assert.equal('x',stream.data);
+assert.strictEqual(stream.data,'x');
 });
 
 // pause and then resume immediately.
-varpauseRes=fs.createReadStream(rangeFile);
+constpauseRes=fs.createReadStream(rangeFile);
 pauseRes.pause();
 pauseRes.resume();
 
-varfile7=fs.createReadStream(rangeFile,Object.create({autoClose: false}));
-assert.equal(file7.autoClose,false);
+letfile7=fs.createReadStream(rangeFile,Object.create({autoClose: false}));
+assert.strictEqual(file7.autoClose,false);
 file7.on('data',function(){});
 file7.on('end',function(){
 process.nextTick(function(){
@@ -154,18 +152,18 @@ function file7Next(){
 file7.data+=data;
 });
 file7.on('end',function(err){
-assert.equal(file7.data,'xyz\n');
+assert.strictEqual(file7.data,'xyz\n');
 });
 }
 
 // Just to make sure autoClose won't close the stream because of error.
-varfile8=fs.createReadStream(null,Object.create({fd: 13337,
-autoClose: false}));
+constfile8=fs.createReadStream(null,Object.create({fd: 13337,
+autoClose: false}));
 file8.on('data',function(){});
 file8.on('error',common.mustCall(function(){}));
 
 // Make sure stream is destroyed when file does not exist.
-varfile9=fs.createReadStream('/path/to/file/that/does/not/exist');
+constfile9=fs.createReadStream('/path/to/file/that/does/not/exist');
 file9.on('data',function(){});
 file9.on('error',common.mustCall(function(){}));