Skip to content

Commit

Permalink
test: test-file-write-stream3.js refactor
Browse files Browse the repository at this point in the history
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>
  • Loading branch information
Richard Karmazin authored and jasnell committed Dec 5, 2016
1 parent ef74e03 commit 18cf668
Showing 1 changed file with 16 additions and 20 deletions.
36 changes: 16 additions & 20 deletions test/parallel/test-file-write-stream3.js
Original file line number Diff line number Diff line change
Expand Up @@ -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';

Expand All @@ -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);

Expand All @@ -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();
});
Expand All @@ -65,7 +63,7 @@ function run_test_1() {
throw err;
});

buffer = Buffer.from(fileDataInitial);
const buffer = Buffer.from(fileDataInitial);
file.write(buffer);
cb_occurred += 'write ';

Expand All @@ -74,13 +72,12 @@ function run_test_1() {


function run_test_2() {
var file, buffer, options;

buffer = Buffer.from('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);

Expand All @@ -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();
});
Expand All @@ -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);

Expand All @@ -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();
});
Expand Down

0 comments on commit 18cf668

Please sign in to comment.