test: refactor test-fs-append-file.js

- Replaced var with either const or let. - Replaced assert.equal() with assert.strictEqual(). - Replaced all error checks with assert.ifError(e). PR-URL: #10110 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Santiago Gimeno <[email protected]>

Author: radelmann

test/parallel/test-fs-append-file.js

@@ -1,129 +1,130 @@
 'use strict';
-varcommon=require('../common');
-varassert=require('assert');
-varfs=require('fs');
-varjoin=require('path').join;
+constcommon=require('../common');
+constassert=require('assert');
+constfs=require('fs');
+constjoin=require('path').join;
 
-varfilename=join(common.tmpDir,'append.txt');
+constfilename=join(common.tmpDir,'append.txt');
 
-varcurrentFileData='ABCD';
+constcurrentFileData='ABCD';
 
-varn=220;
-vars='南越国是前203年至前111年存在于岭南地区的一个国家，国都位于番禺，疆域包括今天中国的广东、'+
-'广西两省区的大部份地区，福建省、湖南、贵州、云南的一小部份地区和越南的北部。'+
-'南越国是秦朝灭亡后，由南海郡尉赵佗于前203年起兵兼并桂林郡和象郡后建立。'+
-'前196年和前179年，南越国曾先后两次名义上臣属于西汉，成为西汉的“外臣”。前112年，'+
-'南越国末代君主赵建德与西汉发生战争，被汉武帝于前111年所灭。南越国共存在93年，'+
-'历经五代君主。南越国是岭南地区的第一个有记载的政权国家，采用封建制和郡县制并存的制度，'+
-'它的建立保证了秦末乱世岭南地区社会秩序的稳定，有效的改善了岭南地区落后的政治、##济现状。\n';
+constn=220;
+consts='南越国是前203年至前111年存在于岭南地区的一个国家，国都位于番禺，疆域包括今天中国的广东、'+
+'广西两省区的大部份地区，福建省、湖南、贵州、云南的一小部份地区和越南的北部。'+
+'南越国是秦朝灭亡后，由南海郡尉赵佗于前203年起兵兼并桂林郡和象郡后建立。'+
+'前196年和前179年，南越国曾先后两次名义上臣属于西汉，成为西汉的“外臣”。前112年，'+
+'南越国末代君主赵建德与西汉发生战争，被汉武帝于前111年所灭。南越国共存在93年，'+
+'历经五代君主。南越国是岭南地区的第一个有记载的政权国家，采用封建制和郡县制并存的制度，'+
+'它的建立保证了秦末乱世岭南地区社会秩序的稳定，有效的改善了岭南地区落后的政治、##济现状。\n';
 
-varncallbacks=0;
+letncallbacks=0;
 
 common.refreshTmpDir();
 
 // test that empty file will be created and have content added
 fs.appendFile(filename,s,function(e){
-if(e)throwe;
+assert.ifError(e);
 
 ncallbacks++;
 
 fs.readFile(filename,function(e,buffer){
-if(e)throwe;
+assert.ifError(e);
 ncallbacks++;
-assert.equal(Buffer.byteLength(s),buffer.length);
+assert.strictEqual(Buffer.byteLength(s),buffer.length);
 });
 });
 
 // test that appends data to a non empty file
-varfilename2=join(common.tmpDir,'append2.txt');
+constfilename2=join(common.tmpDir,'append2.txt');
 fs.writeFileSync(filename2,currentFileData);
 
 fs.appendFile(filename2,s,function(e){
-if(e)throwe;
+assert.ifError(e);
 
 ncallbacks++;
 
 fs.readFile(filename2,function(e,buffer){
-if(e)throwe;
+assert.ifError(e);
 ncallbacks++;
-assert.equal(Buffer.byteLength(s)+currentFileData.length,buffer.length);
+assert.strictEqual(Buffer.byteLength(s)+currentFileData.length,
+buffer.length);
 });
 });
 
 // test that appendFile accepts buffers
-varfilename3=join(common.tmpDir,'append3.txt');
+constfilename3=join(common.tmpDir,'append3.txt');
 fs.writeFileSync(filename3,currentFileData);
 
-varbuf=Buffer.from(s,'utf8');
+constbuf=Buffer.from(s,'utf8');
 
 fs.appendFile(filename3,buf,function(e){
-if(e)throwe;
+assert.ifError(e);
 
 ncallbacks++;
 
 fs.readFile(filename3,function(e,buffer){
-if(e)throwe;
+assert.ifError(e);
 ncallbacks++;
-assert.equal(buf.length+currentFileData.length,buffer.length);
+assert.strictEqual(buf.length+currentFileData.length,buffer.length);
 });
 });
 
 // test that appendFile accepts numbers.
-varfilename4=join(common.tmpDir,'append4.txt');
+constfilename4=join(common.tmpDir,'append4.txt');
 fs.writeFileSync(filename4,currentFileData);
 
-varm=0o600;
+constm=0o600;
 fs.appendFile(filename4,n,{mode: m},function(e){
-if(e)throwe;
+assert.ifError(e);
 
 ncallbacks++;
 
 // windows permissions aren't unix
 if(!common.isWindows){
-varst=fs.statSync(filename4);
-assert.equal(st.mode&0o700,m);
+constst=fs.statSync(filename4);
+assert.strictEqual(st.mode&0o700,m);
 }
 
 fs.readFile(filename4,function(e,buffer){
-if(e)throwe;
+assert.ifError(e);
 ncallbacks++;
-assert.equal(Buffer.byteLength(''+n)+currentFileData.length,
-buffer.length);
+assert.strictEqual(Buffer.byteLength(''+n)+currentFileData.length,
+buffer.length);
 });
 });
 
 // test that appendFile accepts file descriptors
-varfilename5=join(common.tmpDir,'append5.txt');
+constfilename5=join(common.tmpDir,'append5.txt');
 fs.writeFileSync(filename5,currentFileData);
 
 fs.open(filename5,'a+',function(e,fd){
-if(e)throwe;
+assert.ifError(e);
 
 ncallbacks++;
 
 fs.appendFile(fd,s,function(e){
-if(e)throwe;
+assert.ifError(e);
 
 ncallbacks++;
 
 fs.close(fd,function(e){
-if(e)throwe;
+assert.ifError(e);
 
 ncallbacks++;
 
 fs.readFile(filename5,function(e,buffer){
-if(e)throwe;
+assert.ifError(e);
 
 ncallbacks++;
-assert.equal(Buffer.byteLength(s)+currentFileData.length,
-buffer.length);
+assert.strictEqual(Buffer.byteLength(s)+currentFileData.length,
+buffer.length);
 });
 });
 });
 });
 
 process.on('exit',function(){
-assert.equal(12,ncallbacks);
+assert.strictEqual(12,ncallbacks);
 
 fs.unlinkSync(filename);
 fs.unlinkSync(filename2);