test: test-file-write-stream3.js refactor

Richard Karmazin · MylesBorins · commit 16164b5b44e8 · 2016-12-20T17:08:49.000-05:00
PR-URL: #10035 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
diff --git a/test/parallel/test-file-write-stream3.js b/test/parallel/test-file-write-stream3.js
@@ -9,7 +9,7 @@ const filepath = path.join(common.tmpDir, 'write_pos.txt');
 
 
 const cb_expected = 'write open close write open close write open close ';
-var cb_occurred = '';
+let cb_occurred = '';
 
 const fileDataInitial = 'abcdefghijklmnopqrstuvwxyz';
 
@@ -34,10 +34,8 @@ common.refreshTmpDir();
 
 
 function run_test_1() {
-  var file, buffer, options;
-
-  options = {};
-  file = fs.createWriteStream(filepath, options);
+  const options = {};
+  const file = fs.createWriteStream(filepath, options);
   console.log('    (debug: start         ', file.start);
   console.log('    (debug: pos           ', file.pos);
 
@@ -51,10 +49,10 @@ function run_test_1() {
     console.log('    (debug: start         ', file.start);
     console.log('    (debug: pos           ', file.pos);
     assert.strictEqual(file.bytesWritten, buffer.length);
-    var fileData = fs.readFileSync(filepath, 'utf8');
+    const fileData = fs.readFileSync(filepath, 'utf8');
     console.log('    (debug: file data   ', fileData);
     console.log('    (debug: expected    ', fileDataExpected_1);
-    assert.equal(fileData, fileDataExpected_1);
+    assert.strictEqual(fileData, fileDataExpected_1);
 
     run_test_2();
   });
@@ -65,7 +63,7 @@ function run_test_1() {
     throw err;
   });
 
-  buffer = new Buffer(fileDataInitial);
+  const buffer = Buffer.from(fileDataInitial);
   file.write(buffer);
   cb_occurred += 'write ';
 
@@ -74,13 +72,12 @@ function run_test_1() {
 
 
 function run_test_2() {
-  var file, buffer, options;
 
-  buffer = new Buffer('123456');
+  const buffer = Buffer.from('123456');
 
-  options = { start: 10,
-              flags: 'r+' };
-  file = fs.createWriteStream(filepath, options);
+  const options = { start: 10,
+                    flags: 'r+' };
+  const file = fs.createWriteStream(filepath, options);
   console.log('    (debug: start         ', file.start);
   console.log('    (debug: pos           ', file.pos);
 
@@ -94,10 +91,10 @@ function run_test_2() {
     console.log('    (debug: start         ', file.start);
     console.log('    (debug: pos           ', file.pos);
     assert.strictEqual(file.bytesWritten, buffer.length);
-    var fileData = fs.readFileSync(filepath, 'utf8');
+    const fileData = fs.readFileSync(filepath, 'utf8');
     console.log('    (debug: file data   ', fileData);
     console.log('    (debug: expected    ', fileDataExpected_2);
-    assert.equal(fileData, fileDataExpected_2);
+    assert.strictEqual(fileData, fileDataExpected_2);
 
     run_test_3();
   });
@@ -116,13 +113,12 @@ function run_test_2() {
 
 
 function run_test_3() {
-  var file, options;
 
   const data = '\u2026\u2026';    // 3 bytes * 2 = 6 bytes in UTF-8
 
-  options = { start: 10,
-              flags: 'r+' };
-  file = fs.createWriteStream(filepath, options);
+  const options = { start: 10,
+                    flags: 'r+' };
+  const file = fs.createWriteStream(filepath, options);
   console.log('    (debug: start         ', file.start);
   console.log('    (debug: pos           ', file.pos);
 
@@ -139,7 +135,7 @@ function run_test_3() {
     const fileData = fs.readFileSync(filepath, 'utf8');
     console.log('    (debug: file data   ', fileData);
     console.log('    (debug: expected    ', fileDataExpected_3);
-    assert.equal(fileData, fileDataExpected_3);
+    assert.strictEqual(fileData, fileDataExpected_3);
 
     run_test_4();
   });
