Skip to content

Commit

Permalink
test: refactor test-fs-write-file
Browse files Browse the repository at this point in the history
Replaced all error checks with assert.ifError(e).

PR-URL: nodejs#10030
Reviewed-By: Rich Trott <[email protected]>
  • Loading branch information
radelmann authored and Jonathan Darling committed Dec 8, 2016
1 parent 3cb0351 commit 2f30424
Showing 1 changed file with 10 additions and 10 deletions.
20 changes: 10 additions & 10 deletions test/parallel/test-fs-write-file.js
Original file line number Diff line number Diff line change
Expand Up @@ -18,10 +18,10 @@ const s = '南越国是前203年至前111年存在于岭南地区的一个国家
'它的建立保证了秦末乱世岭南地区社会秩序的稳定,有效的改善了岭南地区落后的政治、##济现状。\n';

fs.writeFile(filename, s, common.mustCall(function(e) {
if (e) throw e;
assert.ifError(e);

fs.readFile(filename, common.mustCall(function(e, buffer) {
if (e) throw e;
assert.ifError(e);
assert.strictEqual(Buffer.byteLength(s), buffer.length);
}));
}));
Expand All @@ -31,10 +31,10 @@ const filename2 = join(common.tmpDir, 'test2.txt');
const buf = Buffer.from(s, 'utf8');

fs.writeFile(filename2, buf, common.mustCall(function(e) {
if (e) throw e;
assert.ifError(e);

fs.readFile(filename2, common.mustCall(function(e, buffer) {
if (e) throw e;
assert.ifError(e);

assert.strictEqual(buf.length, buffer.length);
}));
Expand All @@ -45,7 +45,7 @@ const filename3 = join(common.tmpDir, 'test3.txt');

const m = 0o600;
fs.writeFile(filename3, n, { mode: m }, common.mustCall(function(e) {
if (e) throw e;
assert.ifError(e);

// windows permissions aren't unix
if (!common.isWindows) {
Expand All @@ -54,7 +54,7 @@ fs.writeFile(filename3, n, { mode: m }, common.mustCall(function(e) {
}

fs.readFile(filename3, common.mustCall(function(e, buffer) {
if (e) throw e;
assert.ifError(e);

assert.strictEqual(Buffer.byteLength('' + n), buffer.length);
}));
Expand All @@ -64,16 +64,16 @@ fs.writeFile(filename3, n, { mode: m }, common.mustCall(function(e) {
const filename4 = join(common.tmpDir, 'test4.txt');

fs.open(filename4, 'w+', common.mustCall(function(e, fd) {
if (e) throw e;
assert.ifError(e);

fs.writeFile(fd, s, common.mustCall(function(e) {
if (e) throw e;
assert.ifError(e);

fs.close(fd, common.mustCall(function(e) {
if (e) throw e;
assert.ifError(e);

fs.readFile(filename4, common.mustCall(function(e, buffer) {
if (e) throw e;
assert.ifError(e);

assert.strictEqual(Buffer.byteLength(s), buffer.length);
}));
Expand Down

0 comments on commit 2f30424

Please sign in to comment.