Skip to content

Commit

Permalink
test: improve test-fs-access
Browse files Browse the repository at this point in the history
* use const and let instead of var
* use common.mustCall to control functions execution
* use assert.ifError instead of assert.strictEqual for errors
* use arrow functions

PR-URL: #10542
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
  • Loading branch information
edsadr authored and italoacasas committed Jan 30, 2017
1 parent db7b27a commit b1c742e
Showing 1 changed file with 29 additions and 29 deletions.
58 changes: 29 additions & 29 deletions test/parallel/test-fs-access.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,19 +3,19 @@ const common = require('../common');
const assert = require('assert');
const fs = require('fs');
const path = require('path');
var doesNotExist = path.join(common.tmpDir, '__this_should_not_exist');
var readOnlyFile = path.join(common.tmpDir, 'read_only_file');
var readWriteFile = path.join(common.tmpDir, 'read_write_file');
const doesNotExist = path.join(common.tmpDir, '__this_should_not_exist');
const readOnlyFile = path.join(common.tmpDir, 'read_only_file');
const readWriteFile = path.join(common.tmpDir, 'read_write_file');

var removeFile = function(file) {
const removeFile = function(file) {
try {
fs.unlinkSync(file);
} catch (err) {
// Ignore error
}
};

var createFileWithPerms = function(file, mode) {
const createFileWithPerms = function(file, mode) {
removeFile(file);
fs.writeFileSync(file, '');
fs.chmodSync(file, mode);
Expand Down Expand Up @@ -48,7 +48,7 @@ createFileWithPerms(readWriteFile, 0o666);
* id, but that's fine. In this case, it is the responsability of the
* continuous integration platform to take care of that.
*/
var hasWriteAccessForReadonlyFile = false;
let hasWriteAccessForReadonlyFile = false;
if (!common.isWindows && process.getuid() === 0) {
hasWriteAccessForReadonlyFile = true;
try {
Expand All @@ -63,62 +63,62 @@ assert.strictEqual(typeof fs.R_OK, 'number');
assert.strictEqual(typeof fs.W_OK, 'number');
assert.strictEqual(typeof fs.X_OK, 'number');

fs.access(__filename, function(err) {
assert.strictEqual(err, null, 'error should not exist');
});
fs.access(__filename, common.mustCall((err) => {
assert.ifError(err);
}));

fs.access(__filename, fs.R_OK, function(err) {
assert.strictEqual(err, null, 'error should not exist');
});
fs.access(__filename, fs.R_OK, common.mustCall((err) => {
assert.ifError(err);
}));

fs.access(doesNotExist, function(err) {
fs.access(doesNotExist, common.mustCall((err) => {
assert.notEqual(err, null, 'error should exist');
assert.strictEqual(err.code, 'ENOENT');
assert.strictEqual(err.path, doesNotExist);
});
}));

fs.access(readOnlyFile, fs.F_OK | fs.R_OK, function(err) {
assert.strictEqual(err, null, 'error should not exist');
});
fs.access(readOnlyFile, fs.F_OK | fs.R_OK, common.mustCall((err) => {
assert.ifError(err);
}));

fs.access(readOnlyFile, fs.W_OK, function(err) {
fs.access(readOnlyFile, fs.W_OK, common.mustCall((err) => {
if (hasWriteAccessForReadonlyFile) {
assert.equal(err, null, 'error should not exist');
assert.ifError(err);
} else {
assert.notEqual(err, null, 'error should exist');
assert.strictEqual(err.path, readOnlyFile);
}
});
}));

assert.throws(function() {
fs.access(100, fs.F_OK, function(err) {});
assert.throws(() => {
fs.access(100, fs.F_OK, (err) => {});
}, /path must be a string or Buffer/);

assert.throws(function() {
assert.throws(() => {
fs.access(__filename, fs.F_OK);
}, /"callback" argument must be a function/);

assert.throws(function() {
assert.throws(() => {
fs.access(__filename, fs.F_OK, {});
}, /"callback" argument must be a function/);

assert.doesNotThrow(function() {
assert.doesNotThrow(() => {
fs.accessSync(__filename);
});

assert.doesNotThrow(function() {
assert.doesNotThrow(() => {
var mode = fs.F_OK | fs.R_OK | fs.W_OK;

fs.accessSync(readWriteFile, mode);
});

assert.throws(function() {
assert.throws(() => {
fs.accessSync(doesNotExist);
}, function(err) {
}, (err) => {
return err.code === 'ENOENT' && err.path === doesNotExist;
});

process.on('exit', function() {
process.on('exit', () => {
removeFile(readOnlyFile);
removeFile(readWriteFile);
});

0 comments on commit b1c742e

Please sign in to comment.