Skip to content

Commit 5ca9077

Browse files
amriosaddaleax
amrios
authored andcommitted
test: refactor test-stdin-from-file.js
change var to const/let wrap common.mustCall on childProcess.exec callback remove unneeded fs.unlinkSync() refactor assert.equal to assert.strictEqual PR-URL: #10012 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Jeremiah Senkpiel <[email protected]>
1 parent 4d66578 commit 5ca9077

File tree

1 file changed

+27
-31
lines changed

1 file changed

+27
-31
lines changed

test/parallel/test-stdin-from-file.js

+27-31
Original file line numberDiff line numberDiff line change
@@ -1,44 +1,40 @@
11
'use strict';
2-
var common = require('../common');
3-
var assert = require('assert');
4-
var join = require('path').join;
5-
var childProcess = require('child_process');
6-
var fs = require('fs');
7-
8-
var stdoutScript = join(common.fixturesDir, 'echo-close-check.js');
9-
var tmpFile = join(common.tmpDir, 'stdin.txt');
10-
11-
var cmd = '"' + process.argv[0] + '" "' + stdoutScript + '" < "' +
12-
tmpFile + '"';
13-
14-
var string = 'abc\nümlaut.\nsomething else\n' +
15-
'南越国是前203年至前111年存在于岭南地区的一个国家,国都位于番禺,' +
16-
'疆域包括今天中国的广东、广西两省区的大部份地区,福建省、湖南、贵州、' +
17-
'云南的一小部份地区和越南的北部。南越国是秦朝灭亡后,' +
18-
'由南海郡尉赵佗于前203年起兵兼并桂林郡和象郡后建立。前196年和前179年,' +
19-
'南越国曾先后两次名义上臣属于西汉,成为西汉的“外臣”。前112年,' +
20-
'南越国末代君主赵建德与西汉发生战争,被汉武帝于前111年所灭。南越国共存在93年,' +
21-
'历经五代君主。南越国是岭南地区的第一个有记载的政权国家,' +
22-
'采用封建制和郡县制并存的制度,' +
23-
'它的建立保证了秦末乱世岭南地区社会秩序的稳定,' +
24-
'有效的改善了岭南地区落后的政治、##济现状。\n';
2+
const common = require('../common');
3+
const assert = require('assert');
4+
const join = require('path').join;
5+
const childProcess = require('child_process');
6+
const fs = require('fs');
7+
8+
const stdoutScript = join(common.fixturesDir, 'echo-close-check.js');
9+
const tmpFile = join(common.tmpDir, 'stdin.txt');
10+
11+
const cmd = '"' + process.argv[0] + '" "' + stdoutScript + '" < "' +
12+
tmpFile + '"';
13+
14+
const string = 'abc\nümlaut.\nsomething else\n' +
15+
'南越国是前203年至前111年存在于岭南地区的一个国家,国都位于番禺,' +
16+
'疆域包括今天中国的广东、广西两省区的大部份地区,福建省、湖南、贵州、' +
17+
'云南的一小部份地区和越南的北部。南越国是秦朝灭亡后,' +
18+
'由南海郡尉赵佗于前203年起兵兼并桂林郡和象郡后建立。前196年和前179年,' +
19+
'南越国曾先后两次名义上臣属于西汉,成为西汉的“外臣”。前112年,' +
20+
'南越国末代君主赵建德与西汉发生战争,被汉武帝于前111年所灭。南越国共存在93年,' +
21+
'历经五代君主。南越国是岭南地区的第一个有记载的政权国家,' +
22+
'采用封建制和郡县制并存的制度,' +
23+
'它的建立保证了秦末乱世岭南地区社会秩序的稳定,' +
24+
'有效的改善了岭南地区落后的政治、##济现状。\n';
2525

2626

2727
common.refreshTmpDir();
2828

2929
console.log(cmd + '\n\n');
3030

31-
try {
32-
fs.unlinkSync(tmpFile);
33-
} catch (e) {}
34-
3531
fs.writeFileSync(tmpFile, string);
3632

37-
childProcess.exec(cmd, function(err, stdout, stderr) {
33+
childProcess.exec(cmd, common.mustCall(function(err, stdout, stderr) {
3834
fs.unlinkSync(tmpFile);
3935

4036
if (err) throw err;
4137
console.log(stdout);
42-
assert.equal(stdout, 'hello world\r\n' + string);
43-
assert.equal('', stderr);
44-
});
38+
assert.strictEqual(stdout, 'hello world\r\n' + string);
39+
assert.strictEqual('', stderr);
40+
}));

0 commit comments

Comments
 (0)