Browse Source

test: improve test-fs-write-file-sync

* use let and const instead of var
* use assert.strictEqual instead of assert.equal
* swap assertions arguments to match the standard

PR-URL: https://github.com/nodejs/node/pull/10624
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
v6.x
Adrian Estrada 8 years ago
committed by Myles Borins
parent
commit
04dc1cdfcb
No known key found for this signature in database GPG Key ID: 933B01F40B5CA946
  1. 36
      test/parallel/test-fs-write-file-sync.js

36
test/parallel/test-fs-write-file-sync.js

@ -1,11 +1,11 @@
'use strict';
var common = require('../common');
var assert = require('assert');
var path = require('path');
var fs = require('fs');
var openCount = 0;
var mode;
var content;
const common = require('../common');
const assert = require('assert');
const path = require('path');
const fs = require('fs');
let openCount = 0;
let mode;
let content;
// Need to hijack fs.open/close to make sure that things
// get closed once they're opened.
@ -28,39 +28,39 @@ if (common.isWindows) {
common.refreshTmpDir();
// Test writeFileSync
var file1 = path.join(common.tmpDir, 'testWriteFileSync.txt');
const file1 = path.join(common.tmpDir, 'testWriteFileSync.txt');
fs.writeFileSync(file1, '123', {mode: mode});
content = fs.readFileSync(file1, {encoding: 'utf8'});
assert.equal('123', content);
assert.strictEqual(content, '123');
assert.equal(mode, fs.statSync(file1).mode & 0o777);
assert.strictEqual(fs.statSync(file1).mode & 0o777, mode);
// Test appendFileSync
var file2 = path.join(common.tmpDir, 'testAppendFileSync.txt');
const file2 = path.join(common.tmpDir, 'testAppendFileSync.txt');
fs.appendFileSync(file2, 'abc', {mode: mode});
content = fs.readFileSync(file2, {encoding: 'utf8'});
assert.equal('abc', content);
assert.strictEqual(content, 'abc');
assert.equal(mode, fs.statSync(file2).mode & mode);
assert.strictEqual(fs.statSync(file2).mode & mode, mode);
// Test writeFileSync with file descriptor
var file3 = path.join(common.tmpDir, 'testWriteFileSyncFd.txt');
const file3 = path.join(common.tmpDir, 'testWriteFileSyncFd.txt');
var fd = fs.openSync(file3, 'w+', mode);
const fd = fs.openSync(file3, 'w+', mode);
fs.writeFileSync(fd, '123');
fs.closeSync(fd);
content = fs.readFileSync(file3, {encoding: 'utf8'});
assert.equal('123', content);
assert.strictEqual(content, '123');
assert.equal(mode, fs.statSync(file3).mode & 0o777);
assert.strictEqual(fs.statSync(file3).mode & 0o777, mode);
// Verify that all opened files were closed.
assert.equal(0, openCount);
assert.strictEqual(openCount, 0);
function openSync() {
openCount++;

Loading…
Cancel
Save