diff options
| author | Artur Vieira <vieira.artur.g@gmail.com> | 2017-05-08 01:17:58 +0000 |
|---|---|---|
| committer | Refael Ackermann <refack@gmail.com> | 2017-05-10 11:30:37 -0400 |
| commit | cfe7b34058716d9c9dfb143ecc865a30f433db5c (patch) | |
| tree | 5d39787dec0eb36dd61572400134eab9f9cbf34a /test | |
| parent | 317180ffe55516771a0ffa9414ccb9892a68952f (diff) | |
| download | android-node-v8-cfe7b34058716d9c9dfb143ecc865a30f433db5c.tar.gz android-node-v8-cfe7b34058716d9c9dfb143ecc865a30f433db5c.tar.bz2 android-node-v8-cfe7b34058716d9c9dfb143ecc865a30f433db5c.zip | |
test: use assert regexp in tls no cert test
Replace the `assert.throws` second argument from a Type to a `RegExp`
matching the entire error message.
Error message changes are `semver-major`, so we assert their content.
PR-URL: https://github.com/nodejs/node/pull/12891
Refs: https://github.com/nodejs/node/pull/12603
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: David Cai <davidcai1993@yahoo.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Diffstat (limited to 'test')
| -rw-r--r-- | test/parallel/test-tls-no-cert-required.js | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/test/parallel/test-tls-no-cert-required.js b/test/parallel/test-tls-no-cert-required.js index 04a8b91cf8..06139307c4 100644 --- a/test/parallel/test-tls-no-cert-required.js +++ b/test/parallel/test-tls-no-cert-required.js @@ -41,7 +41,8 @@ tls.createServer(assert.fail) tls.createServer({}) .listen(0, common.mustCall(close)); -assert.throws(() => tls.createServer('this is not valid'), TypeError); +assert.throws(() => tls.createServer('this is not valid'), + /^TypeError: options must be an object$/); tls.createServer() .listen(0, common.mustCall(close)); |