From 8ac6a709b9a0d893a5296bf5e33f85111e093667 Mon Sep 17 00:00:00 2001 From: abouthiroppy Date: Fri, 20 Jan 2017 15:33:09 +0900 Subject: [PATCH] test: add fs-assert-encoding's test Check the error of `assertEncoding`. Confirmed in every place being used.(a place where `getoptions` is used) assetEncoding: https://github.com/nodejs/node/blob/521767c88605cb6481ea98f396924e55f9dd22f4/lib/internal/fs.js#L18 PR-URL: https://github.com/nodejs/node/pull/10913 Reviewed-By: Colin Ihrig Reviewed-By: James M Snell Reviewed-By: Luigi Pinca --- .../parallel/test-fs-assert-encoding-error.js | 76 +++++++++++++++++++ 1 file changed, 76 insertions(+) create mode 100644 test/parallel/test-fs-assert-encoding-error.js diff --git a/test/parallel/test-fs-assert-encoding-error.js b/test/parallel/test-fs-assert-encoding-error.js new file mode 100644 index 00000000000000..0b127d49974bb6 --- /dev/null +++ b/test/parallel/test-fs-assert-encoding-error.js @@ -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);