Skip to content

Commit fc42825

Browse files
llkatsFishrock123
authored andcommitted
test: modernize syntax, add strict checks
Changed `var` to `const`, strings to template literals, and assert.equal to assert.strictEqual where appropriate. PR-URL: #8841 Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
1 parent 72de859 commit fc42825

File tree

3 files changed

+28
-28
lines changed

3 files changed

+28
-28
lines changed

test/parallel/test-fs-readfile-pipe-large.js

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
'use strict';
2-
var common = require('../common');
3-
var assert = require('assert');
4-
var path = require('path');
2+
const common = require('../common');
3+
const assert = require('assert');
4+
const path = require('path');
55

66
// simulate `cat readfile.js | node readfile.js`
77

@@ -10,7 +10,7 @@ if (common.isWindows || common.isAix) {
1010
return;
1111
}
1212

13-
var fs = require('fs');
13+
const fs = require('fs');
1414

1515
if (process.argv[2] === 'child') {
1616
fs.readFile('/dev/stdin', function(er, data) {
@@ -20,15 +20,15 @@ if (process.argv[2] === 'child') {
2020
return;
2121
}
2222

23-
var filename = path.join(common.tmpDir, '/readfile_pipe_large_test.txt');
24-
var dataExpected = new Array(1000000).join('a');
23+
const filename = path.join(common.tmpDir, '/readfile_pipe_large_test.txt');
24+
const dataExpected = new Array(1000000).join('a');
2525
common.refreshTmpDir();
2626
fs.writeFileSync(filename, dataExpected);
2727

28-
var exec = require('child_process').exec;
29-
var f = JSON.stringify(__filename);
30-
var node = JSON.stringify(process.execPath);
31-
var cmd = 'cat ' + filename + ' | ' + node + ' ' + f + ' child';
28+
const exec = require('child_process').exec;
29+
const f = JSON.stringify(__filename);
30+
const node = JSON.stringify(process.execPath);
31+
const cmd = `cat ${filename} | ${node} ${f} child`;
3232
exec(cmd, { maxBuffer: 1000000 }, function(err, stdout, stderr) {
3333
if (err) console.error(err);
3434
assert(!err, 'it exits normally');

test/parallel/test-fs-readfile-pipe.js

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
'use strict';
2-
var common = require('../common');
3-
var assert = require('assert');
2+
const common = require('../common');
3+
const assert = require('assert');
44

55
// simulate `cat readfile.js | node readfile.js`
66

@@ -9,9 +9,9 @@ if (common.isWindows || common.isAix) {
99
return;
1010
}
1111

12-
var fs = require('fs');
12+
const fs = require('fs');
1313

14-
var dataExpected = fs.readFileSync(__filename, 'utf8');
14+
const dataExpected = fs.readFileSync(__filename, 'utf8');
1515

1616
if (process.argv[2] === 'child') {
1717
fs.readFile('/dev/stdin', function(er, data) {
@@ -21,10 +21,10 @@ if (process.argv[2] === 'child') {
2121
return;
2222
}
2323

24-
var exec = require('child_process').exec;
25-
var f = JSON.stringify(__filename);
26-
var node = JSON.stringify(process.execPath);
27-
var cmd = 'cat ' + f + ' | ' + node + ' ' + f + ' child';
24+
const exec = require('child_process').exec;
25+
const f = JSON.stringify(__filename);
26+
const node = JSON.stringify(process.execPath);
27+
const cmd = `cat ${f} | ${node} ${f} child`;
2828
exec(cmd, function(err, stdout, stderr) {
2929
if (err) console.error(err);
3030
assert(!err, 'it exits normally');

test/parallel/test-fs-readfilesync-pipe-large.js

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
'use strict';
2-
var common = require('../common');
3-
var assert = require('assert');
4-
var path = require('path');
2+
const common = require('../common');
3+
const assert = require('assert');
4+
const path = require('path');
55

66
// simulate `cat readfile.js | node readfile.js`
77

@@ -10,22 +10,22 @@ if (common.isWindows || common.isAix) {
1010
return;
1111
}
1212

13-
var fs = require('fs');
13+
const fs = require('fs');
1414

1515
if (process.argv[2] === 'child') {
1616
process.stdout.write(fs.readFileSync('/dev/stdin', 'utf8'));
1717
return;
1818
}
1919

20-
var filename = path.join(common.tmpDir, '/readfilesync_pipe_large_test.txt');
21-
var dataExpected = new Array(1000000).join('a');
20+
const filename = path.join(common.tmpDir, '/readfilesync_pipe_large_test.txt');
21+
const dataExpected = new Array(1000000).join('a');
2222
common.refreshTmpDir();
2323
fs.writeFileSync(filename, dataExpected);
2424

25-
var exec = require('child_process').exec;
26-
var f = JSON.stringify(__filename);
27-
var node = JSON.stringify(process.execPath);
28-
var cmd = 'cat ' + filename + ' | ' + node + ' ' + f + ' child';
25+
const exec = require('child_process').exec;
26+
const f = JSON.stringify(__filename);
27+
const node = JSON.stringify(process.execPath);
28+
const cmd = `cat ${filename} | ${node} ${f} child`;
2929
exec(cmd, { maxBuffer: 1000000 }, function(err, stdout, stderr) {
3030
if (err) console.error(err);
3131
assert(!err, 'it exits normally');

0 commit comments

Comments
 (0)