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 <cjihrig@gmail.com>
Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com>

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

@@ -1,129 +1,130 @@
 'use strict';
-var common = require('../common');
-var assert = require('assert');
-var fs = require('fs');
-var join = require('path').join;
+const common = require('../common');
+const assert = require('assert');
+const fs = require('fs');
+const join = require('path').join;
 
-var filename = join(common.tmpDir, 'append.txt');
+const filename = join(common.tmpDir, 'append.txt');
 
-var currentFileData = 'ABCD';
+const currentFileData = 'ABCD';
 
-var n = 220;
-var s = '南越国是前203年至前111年存在于岭南地区的一个国家，国都位于番禺，疆域包括今天中国的广东、' +
-        '广西两省区的大部份地区，福建省、湖南、贵州、云南的一小部份地区和越南的北部。' +
-        '南越国是秦朝灭亡后，由南海郡尉赵佗于前203年起兵兼并桂林郡和象郡后建立。' +
-        '前196年和前179年，南越国曾先后两次名义上臣属于西汉，成为西汉的“外臣”。前112年，' +
-        '南越国末代君主赵建德与西汉发生战争，被汉武帝于前111年所灭。南越国共存在93年，' +
-        '历经五代君主。南越国是岭南地区的第一个有记载的政权国家，采用封建制和郡县制并存的制度，' +
-        '它的建立保证了秦末乱世岭南地区社会秩序的稳定，有效的改善了岭南地区落后的政治、##济现状。\n';
+const n = 220;
+const s = '南越国是前203年至前111年存在于岭南地区的一个国家，国都位于番禺，疆域包括今天中国的广东、' +
+          '广西两省区的大部份地区，福建省、湖南、贵州、云南的一小部份地区和越南的北部。' +
+          '南越国是秦朝灭亡后，由南海郡尉赵佗于前203年起兵兼并桂林郡和象郡后建立。' +
+          '前196年和前179年，南越国曾先后两次名义上臣属于西汉，成为西汉的“外臣”。前112年，' +
+          '南越国末代君主赵建德与西汉发生战争，被汉武帝于前111年所灭。南越国共存在93年，' +
+          '历经五代君主。南越国是岭南地区的第一个有记载的政权国家，采用封建制和郡县制并存的制度，' +
+          '它的建立保证了秦末乱世岭南地区社会秩序的稳定，有效的改善了岭南地区落后的政治、##济现状。\n';
 
-var ncallbacks = 0;
+let ncallbacks = 0;
 
 common.refreshTmpDir();
 
 // test that empty file will be created and have content added
 fs.appendFile(filename, s, function(e) {
-  if (e) throw e;
+  assert.ifError(e);
 
   ncallbacks++;
 
   fs.readFile(filename, function(e, buffer) {
-    if (e) throw e;
+    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
-var filename2 = join(common.tmpDir, 'append2.txt');
+const filename2 = join(common.tmpDir, 'append2.txt');
 fs.writeFileSync(filename2, currentFileData);
 
 fs.appendFile(filename2, s, function(e) {
-  if (e) throw e;
+  assert.ifError(e);
 
   ncallbacks++;
 
   fs.readFile(filename2, function(e, buffer) {
-    if (e) throw e;
+    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
-var filename3 = join(common.tmpDir, 'append3.txt');
+const filename3 = join(common.tmpDir, 'append3.txt');
 fs.writeFileSync(filename3, currentFileData);
 
-var buf = Buffer.from(s, 'utf8');
+const buf = Buffer.from(s, 'utf8');
 
 fs.appendFile(filename3, buf, function(e) {
-  if (e) throw e;
+  assert.ifError(e);
 
   ncallbacks++;
 
   fs.readFile(filename3, function(e, buffer) {
-    if (e) throw e;
+    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.
-var filename4 = join(common.tmpDir, 'append4.txt');
+const filename4 = join(common.tmpDir, 'append4.txt');
 fs.writeFileSync(filename4, currentFileData);
 
-var m = 0o600;
+const m = 0o600;
 fs.appendFile(filename4, n, { mode: m }, function(e) {
-  if (e) throw e;
+  assert.ifError(e);
 
   ncallbacks++;
 
   // windows permissions aren't unix
   if (!common.isWindows) {
-    var st = fs.statSync(filename4);
-    assert.equal(st.mode & 0o700, m);
+    const st = fs.statSync(filename4);
+    assert.strictEqual(st.mode & 0o700, m);
   }
 
   fs.readFile(filename4, function(e, buffer) {
-    if (e) throw e;
+    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
-var filename5 = join(common.tmpDir, 'append5.txt');
+const filename5 = join(common.tmpDir, 'append5.txt');
 fs.writeFileSync(filename5, currentFileData);
 
 fs.open(filename5, 'a+', function(e, fd) {
-  if (e) throw e;
+  assert.ifError(e);
 
   ncallbacks++;
 
   fs.appendFile(fd, s, function(e) {
-    if (e) throw e;
+    assert.ifError(e);
 
     ncallbacks++;
 
     fs.close(fd, function(e) {
-      if (e) throw e;
+      assert.ifError(e);
 
       ncallbacks++;
 
       fs.readFile(filename5, function(e, buffer) {
-        if (e) throw e;
+        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);