diff options
author | Gunar C. Gessner <gunar@gunargessner.com> | 2017-02-11 23:25:33 -0200 |
---|---|---|
committer | Refael Ackermann <refack@gmail.com> | 2017-07-18 23:55:20 -0400 |
commit | b61cab2234b3d9bf7265980f9286a136af20365b (patch) | |
tree | 44b4e6693c12e15c1f0be79ec05ef88704812d26 /test/parallel/test-fs-open-flags.js | |
parent | c5521fa6176377e0be958621102cad29be39a290 (diff) | |
download | android-node-v8-b61cab2234b3d9bf7265980f9286a136af20365b.tar.gz android-node-v8-b61cab2234b3d9bf7265980f9286a136af20365b.tar.bz2 android-node-v8-b61cab2234b3d9bf7265980f9286a136af20365b.zip |
errors: port internal/fs errors to internal/errors
* Assign codes to errors reported by internal/fs.js
PR-URL: https://github.com/nodejs/node/pull/11317
Refs: https://github.com/nodejs/node/issues/11273
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Diffstat (limited to 'test/parallel/test-fs-open-flags.js')
-rw-r--r-- | test/parallel/test-fs-open-flags.js | 17 |
1 files changed, 8 insertions, 9 deletions
diff --git a/test/parallel/test-fs-open-flags.js b/test/parallel/test-fs-open-flags.js index 742b997e60..76fd74736b 100644 --- a/test/parallel/test-fs-open-flags.js +++ b/test/parallel/test-fs-open-flags.js @@ -21,7 +21,7 @@ // Flags: --expose_internals 'use strict'; -require('../common'); +const common = require('../common'); const assert = require('assert'); const fs = require('fs'); @@ -55,30 +55,29 @@ assert.strictEqual(stringToFlags('xa'), O_APPEND | O_CREAT | O_WRONLY | O_EXCL); assert.strictEqual(stringToFlags('ax+'), O_APPEND | O_CREAT | O_RDWR | O_EXCL); assert.strictEqual(stringToFlags('xa+'), O_APPEND | O_CREAT | O_RDWR | O_EXCL); +const expectedError = + common.expectsError({code: 'ERR_INVALID_OPT_VALUE', type: TypeError}, 23); + ('+ +a +r +w rw wa war raw r++ a++ w++ x +x x+ rx rx+ wxx wax xwx xxx') .split(' ') .forEach(function(flags) { assert.throws( () => stringToFlags(flags), - new RegExp(`^Error: Unknown file open flag: ${escapeRegExp(flags)}`) + expectedError ); }); assert.throws( () => stringToFlags({}), - /^Error: Unknown file open flag: \[object Object\]$/ + expectedError ); assert.throws( () => stringToFlags(true), - /^Error: Unknown file open flag: true$/ + expectedError ); assert.throws( () => stringToFlags(null), - /^Error: Unknown file open flag: null$/ + expectedError ); - -function escapeRegExp(string) { - return string.replace(/[\\^$*+?.()|[\]{}]/g, '\\$&'); -} |