diff options
author | Ruben Bridgewater <ruben@bridgewater.de> | 2018-02-10 02:33:08 +0100 |
---|---|---|
committer | Ruben Bridgewater <ruben@bridgewater.de> | 2018-02-16 16:54:07 +0100 |
commit | 644fdd60d4be49ffa66d0bda1702c4459f607635 (patch) | |
tree | 70da98bc22a5a6c5bd0063e25d6eb729125706bf /test/parallel/test-fs-buffer.js | |
parent | caee112e52b64f4bc1118c4a5fa5ad7b4211efea (diff) | |
download | android-node-v8-644fdd60d4be49ffa66d0bda1702c4459f607635.tar.gz android-node-v8-644fdd60d4be49ffa66d0bda1702c4459f607635.tar.bz2 android-node-v8-644fdd60d4be49ffa66d0bda1702c4459f607635.zip |
test: minor refactoring
Add punctuation and comments about code that should not throw.
Also remove a obsolete test and refactor some tests.
PR-URL: https://github.com/nodejs/node/pull/18669
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Diffstat (limited to 'test/parallel/test-fs-buffer.js')
-rw-r--r-- | test/parallel/test-fs-buffer.js | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/test/parallel/test-fs-buffer.js b/test/parallel/test-fs-buffer.js index 2d0bc2a557..c4ab80a429 100644 --- a/test/parallel/test-fs-buffer.js +++ b/test/parallel/test-fs-buffer.js @@ -9,17 +9,13 @@ const path = require('path'); const tmpdir = require('../common/tmpdir'); tmpdir.refresh(); -fs.access(Buffer.from(tmpdir.path), common.mustCall((err) => { - assert.ifError(err); -})); +fs.access(Buffer.from(tmpdir.path), common.mustCall(assert.ifError)); const buf = Buffer.from(path.join(tmpdir.path, 'a.txt')); fs.open(buf, 'w+', common.mustCall((err, fd) => { assert.ifError(err); assert(fd); - fs.close(fd, common.mustCall((err) => { - assert.ifError(err); - })); + fs.close(fd, common.mustCall(assert.ifError)); })); common.expectsError( |