diff options
| author | Teddy Katz <teddy.katz@gmail.com> | 2017-01-06 19:11:07 -0500 |
|---|---|---|
| committer | James M Snell <jasnell@gmail.com> | 2017-01-09 09:20:38 -0800 |
| commit | 5d31448844963d1683417e4f3a5cbb671e3067c2 (patch) | |
| tree | f6c90bdf36dee851ad801c24a553489a3a20f2c0 /test/parallel/test-fs-readfile-pipe.js | |
| parent | 775de9cc96d6fe01c049cd4a3d5ed2df922b88b6 (diff) | |
| download | android-node-v8-5d31448844963d1683417e4f3a5cbb671e3067c2.tar.gz android-node-v8-5d31448844963d1683417e4f3a5cbb671e3067c2.tar.bz2 android-node-v8-5d31448844963d1683417e4f3a5cbb671e3067c2.zip | |
tools,test: enforce assert.ifError with lint rule
This adds an ESLint rule to enforce the use of `assert.ifError(err)`
instead of `if (err) throw err;` in tests.
PR-URL: https://github.com/nodejs/node/pull/10671
Ref: https://github.com/nodejs/node/pull/10543
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: Michal Zasso <targos@protonmail.com>
Diffstat (limited to 'test/parallel/test-fs-readfile-pipe.js')
| -rw-r--r-- | test/parallel/test-fs-readfile-pipe.js | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/test/parallel/test-fs-readfile-pipe.js b/test/parallel/test-fs-readfile-pipe.js index 91465a261c..566dbbe4c8 100644 --- a/test/parallel/test-fs-readfile-pipe.js +++ b/test/parallel/test-fs-readfile-pipe.js @@ -15,7 +15,7 @@ const dataExpected = fs.readFileSync(__filename, 'utf8'); if (process.argv[2] === 'child') { fs.readFile('/dev/stdin', function(er, data) { - if (er) throw er; + assert.ifError(er); process.stdout.write(data); }); return; |