mirror of https://github.com/lukechilds/node.git
Browse Source
Check the error of `assertEncoding`.
Confirmed in every place being used.(a place where `getoptions` is used)
assetEncoding: 521767c886/lib/internal/fs.js (L18)
PR-URL: https://github.com/nodejs/node/pull/10913
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
v6
committed by
Yuta Hiroto
1 changed files with 76 additions and 0 deletions
@ -0,0 +1,76 @@ |
|||||
|
'use strict'; |
||||
|
require('../common'); |
||||
|
const assert = require('assert'); |
||||
|
const fs = require('fs'); |
||||
|
|
||||
|
const options = 'test'; |
||||
|
const noop = () => {}; |
||||
|
const unknownEncodingMessage = /^Error: Unknown encoding: test$/; |
||||
|
|
||||
|
assert.throws(() => { |
||||
|
fs.readFile('path', options, noop); |
||||
|
}, unknownEncodingMessage); |
||||
|
|
||||
|
assert.throws(() => { |
||||
|
fs.readFileSync('path', options); |
||||
|
}, unknownEncodingMessage); |
||||
|
|
||||
|
assert.throws(() => { |
||||
|
fs.readdir('path', options, noop); |
||||
|
}, unknownEncodingMessage); |
||||
|
|
||||
|
assert.throws(() => { |
||||
|
fs.readdirSync('path', options); |
||||
|
}, unknownEncodingMessage); |
||||
|
|
||||
|
assert.throws(() => { |
||||
|
fs.readlink('path', options, noop); |
||||
|
}, unknownEncodingMessage); |
||||
|
|
||||
|
assert.throws(() => { |
||||
|
fs.readlinkSync('path', options); |
||||
|
}, unknownEncodingMessage); |
||||
|
|
||||
|
assert.throws(() => { |
||||
|
fs.writeFile('path', 'data', options, noop); |
||||
|
}, unknownEncodingMessage); |
||||
|
|
||||
|
assert.throws(() => { |
||||
|
fs.writeFileSync('path', 'data', options); |
||||
|
}, unknownEncodingMessage); |
||||
|
|
||||
|
assert.throws(() => { |
||||
|
fs.appendFile('path', 'data', options, noop); |
||||
|
}, unknownEncodingMessage); |
||||
|
|
||||
|
assert.throws(() => { |
||||
|
fs.appendFileSync('path', 'data', options); |
||||
|
}, unknownEncodingMessage); |
||||
|
|
||||
|
assert.throws(() => { |
||||
|
fs.watch('path', options, noop); |
||||
|
}, unknownEncodingMessage); |
||||
|
|
||||
|
assert.throws(() => { |
||||
|
fs.realpath('path', options, noop); |
||||
|
}, unknownEncodingMessage); |
||||
|
|
||||
|
assert.throws(() => { |
||||
|
fs.realpathSync('path', options); |
||||
|
}, unknownEncodingMessage); |
||||
|
|
||||
|
assert.throws(() => { |
||||
|
fs.mkdtemp('path', options, noop); |
||||
|
}, unknownEncodingMessage); |
||||
|
|
||||
|
assert.throws(() => { |
||||
|
fs.mkdtempSync('path', options); |
||||
|
}, unknownEncodingMessage); |
||||
|
|
||||
|
assert.throws(() => { |
||||
|
fs.ReadStream('path', options); |
||||
|
}, unknownEncodingMessage); |
||||
|
|
||||
|
assert.throws(() => { |
||||
|
fs.WriteStream('path', options); |
||||
|
}, unknownEncodingMessage); |
Loading…
Reference in new issue