Skip to content

Commit

Permalink
test: improve test-fs-empty-readStream.js
Browse files Browse the repository at this point in the history
* use const instead of var
* use common.mustCall to control functions execution
* use assert.strictEqual instead of assert.equal
* use arrow functions

PR-URL: #10479
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Italo A. Casas <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Brian White <[email protected]>
  • Loading branch information
edsadr authored and MylesBorins committed Feb 1, 2017
1 parent 08e924e commit 52edebc
Showing 1 changed file with 16 additions and 16 deletions.
32 changes: 16 additions & 16 deletions test/parallel/test-fs-empty-readStream.js
Original file line number Diff line number Diff line change
@@ -1,38 +1,38 @@
'use strict';
var common = require('../common');
var assert = require('assert');
var path = require('path');
var fs = require('fs');
const common = require('../common');
const assert = require('assert');
const path = require('path');
const fs = require('fs');

var emptyFile = path.join(common.fixturesDir, 'empty.txt');
const emptyFile = path.join(common.fixturesDir, 'empty.txt');

fs.open(emptyFile, 'r', function(error, fd) {
fs.open(emptyFile, 'r', common.mustCall((error, fd) => {
assert.ifError(error);

var read = fs.createReadStream(emptyFile, { 'fd': fd });
const read = fs.createReadStream(emptyFile, { 'fd': fd });

read.once('data', function() {
read.once('data', () => {
throw new Error('data event should not emit');
});

read.once('end', common.mustCall(function endEvent1() {}));
});
}));

fs.open(emptyFile, 'r', function(error, fd) {
fs.open(emptyFile, 'r', common.mustCall((error, fd) => {
assert.ifError(error);

var read = fs.createReadStream(emptyFile, { 'fd': fd });
const read = fs.createReadStream(emptyFile, { 'fd': fd });
read.pause();

read.once('data', function() {
read.once('data', () => {
throw new Error('data event should not emit');
});

read.once('end', function endEvent2() {
throw new Error('end event should not emit');
});

setTimeout(function() {
assert.equal(read.isPaused(), true);
}, common.platformTimeout(50));
});
setTimeout(common.mustCall(() => {
assert.strictEqual(read.isPaused(), true);
}), common.platformTimeout(50));
}));

0 comments on commit 52edebc

Please sign in to comment.