Skip to content

Commit

Permalink
test: improve code in test-fs-readfile-error
Browse files Browse the repository at this point in the history
* use const instead of var
* use common.mustCall to control the functions execution automatically
* use assert.strictEqual instead of assert.equal
* use assert.notStrictEqual instead of assert.notEqual
* use arrow functions

PR-URL: #10367
Reviewed-By: Sakthipriyan Vairamani <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: James M Snell <[email protected]>
  • Loading branch information
edsadr authored and evanlucas committed Jan 4, 2017
1 parent 98fcb22 commit 011bd46
Showing 1 changed file with 13 additions and 13 deletions.
26 changes: 13 additions & 13 deletions test/parallel/test-fs-readfile-error.js
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
'use strict';
var common = require('../common');
var assert = require('assert');
var exec = require('child_process').exec;
var path = require('path');
const common = require('../common');
const assert = require('assert');
const exec = require('child_process').exec;
const path = require('path');

// `fs.readFile('/')` does not fail on FreeBSD, because you can open and read
// the directory there.
Expand All @@ -12,23 +12,23 @@ if (process.platform === 'freebsd') {
}

function test(env, cb) {
var filename = path.join(common.fixturesDir, 'test-fs-readfile-error.js');
var execPath = '"' + process.execPath + '" "' + filename + '"';
var options = { env: Object.assign(process.env, env) };
exec(execPath, options, function(err, stdout, stderr) {
const filename = path.join(common.fixturesDir, 'test-fs-readfile-error.js');
const execPath = '"' + process.execPath + '" "' + filename + '"';
const options = { env: Object.assign(process.env, env) };
exec(execPath, options, common.mustCall((err, stdout, stderr) => {
assert(err);
assert.equal(stdout, '');
assert.notEqual(stderr, '');
assert.strictEqual(stdout, '');
assert.notStrictEqual(stderr, '');
cb('' + stderr);
});
}));
}

test({ NODE_DEBUG: '' }, common.mustCall(function(data) {
test({ NODE_DEBUG: '' }, common.mustCall((data) => {
assert(/EISDIR/.test(data));
assert(!/test-fs-readfile-error/.test(data));
}));

test({ NODE_DEBUG: 'fs' }, common.mustCall(function(data) {
test({ NODE_DEBUG: 'fs' }, common.mustCall((data) => {
assert(/EISDIR/.test(data));
assert(/test-fs-readfile-error/.test(data));
}));

0 comments on commit 011bd46

Please sign in to comment.