From 04dc1cdfcbec23b1740bb88f2f393cbcdcf4f490 Mon Sep 17 00:00:00 2001
From: Adrian Estrada <edsadr@gmail.com>
Date: Sat, 7 Jan 2017 09:58:28 -0500
Subject: [PATCH] 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>
---
 test/parallel/test-fs-write-file-sync.js | 36 ++++++++++++------------
 1 file changed, 18 insertions(+), 18 deletions(-)

diff --git a/test/parallel/test-fs-write-file-sync.js b/test/parallel/test-fs-write-file-sync.js
index 7ae9e122ca..ca6324ac1f 100644
--- a/test/parallel/test-fs-write-file-sync.js
+++ b/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++;