Browse Source

test: improve test-fs-read-stream.js

var -> const
assert.equal() -> assert.strictEqual()

PR-URL: https://github.com/nodejs/node/pull/9629
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>
v7.x
Jenna Vuong 8 years ago
committed by Anna Henningsen
parent
commit
e695862531
No known key found for this signature in database GPG Key ID: D8B9F5AEAE84E4CF
  1. 38
      test/parallel/test-fs-read-stream.js

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

@ -1,11 +1,11 @@
'use strict'; 'use strict';
var common = require('../common'); const common = require('../common');
var assert = require('assert'); const assert = require('assert');
var path = require('path'); const path = require('path');
var fs = require('fs'); const fs = require('fs');
var fn = path.join(common.fixturesDir, 'elipses.txt'); const fn = path.join(common.fixturesDir, 'elipses.txt');
var rangeFile = path.join(common.fixturesDir, 'x.txt'); const rangeFile = path.join(common.fixturesDir, 'x.txt');
var callbacks = { open: 0, end: 0, close: 0 }; var callbacks = { open: 0, end: 0, close: 0 };
@ -20,7 +20,7 @@ assert.strictEqual(file.bytesRead, 0);
file.on('open', function(fd) { file.on('open', function(fd) {
file.length = 0; file.length = 0;
callbacks.open++; callbacks.open++;
assert.equal('number', typeof fd); assert.strictEqual('number', typeof fd);
assert.strictEqual(file.bytesRead, 0); assert.strictEqual(file.bytesRead, 0);
assert.ok(file.readable); assert.ok(file.readable);
@ -67,12 +67,12 @@ file.on('close', function() {
var file3 = fs.createReadStream(fn, {encoding: 'utf8'}); var file3 = fs.createReadStream(fn, {encoding: 'utf8'});
file3.length = 0; file3.length = 0;
file3.on('data', function(data) { file3.on('data', function(data) {
assert.equal('string', typeof data); assert.strictEqual('string', typeof data);
file3.length += data.length; file3.length += data.length;
for (var i = 0; i < data.length; i++) { for (var i = 0; i < data.length; i++) {
// http://www.fileformat.info/info/unicode/char/2026/index.htm // http://www.fileformat.info/info/unicode/char/2026/index.htm
assert.equal('\u2026', data[i]); assert.strictEqual('\u2026', data[i]);
} }
}); });
@ -81,11 +81,11 @@ file3.on('close', function() {
}); });
process.on('exit', function() { process.on('exit', function() {
assert.equal(1, callbacks.open); assert.strictEqual(1, callbacks.open);
assert.equal(1, callbacks.end); assert.strictEqual(1, callbacks.end);
assert.equal(2, callbacks.close); assert.strictEqual(2, callbacks.close);
assert.equal(30000, file.length); assert.strictEqual(30000, file.length);
assert.equal(10000, file3.length); assert.strictEqual(10000, file3.length);
console.error('ok'); console.error('ok');
}); });
@ -95,7 +95,7 @@ file4.on('data', function(data) {
contentRead += data.toString('utf-8'); contentRead += data.toString('utf-8');
}); });
file4.on('end', function(data) { file4.on('end', function(data) {
assert.equal(contentRead, 'yz'); assert.strictEqual(contentRead, 'yz');
}); });
var file5 = fs.createReadStream(rangeFile, {bufferSize: 1, start: 1}); var file5 = fs.createReadStream(rangeFile, {bufferSize: 1, start: 1});
@ -104,7 +104,7 @@ file5.on('data', function(data) {
file5.data += data.toString('utf-8'); file5.data += data.toString('utf-8');
}); });
file5.on('end', function() { file5.on('end', function() {
assert.equal(file5.data, 'yz\n'); assert.strictEqual(file5.data, 'yz\n');
}); });
// https://github.com/joyent/node/issues/2320 // https://github.com/joyent/node/issues/2320
@ -114,7 +114,7 @@ file6.on('data', function(data) {
file6.data += data.toString('utf-8'); file6.data += data.toString('utf-8');
}); });
file6.on('end', function() { file6.on('end', function() {
assert.equal(file6.data, 'yz\n'); assert.strictEqual(file6.data, 'yz\n');
}); });
assert.throws(function() { assert.throws(function() {
@ -129,7 +129,7 @@ stream.on('data', function(chunk) {
}); });
stream.on('end', function() { stream.on('end', function() {
assert.equal('x', stream.data); assert.strictEqual('x', stream.data);
}); });
// pause and then resume immediately. // pause and then resume immediately.
@ -155,7 +155,7 @@ function file7Next() {
file7.data += data; file7.data += data;
}); });
file7.on('end', function(err) { file7.on('end', function(err) {
assert.equal(file7.data, 'xyz\n'); assert.strictEqual(file7.data, 'xyz\n');
}); });
} }

Loading…
Cancel
Save