Refactor: Use common.expectsError in tests (!17497) · Merge requests · Rodrigo Test / Test Group-nodejs / node · GitLab

Do not update/delete: Banner broadcast message test data

Do not update/delete: Notification broadcast message test data

Rodrigo Muino Tomonari requested to merge github/fork/mithunsasidharan/common_5 into master Dec 06, 2017

Refactored tests to replace assert.throws(fn, common.expectsError(err)); with common.expectsError(fn, err); in following test files :

test/parallel/test-dns.js
test/parallel/test-fs-realpath.js
test/parallel/test-fs-watchfile.js
test/parallel/test-http-client-check-http-token.js
test/parallel/test-http-client-reject-unexpected-agent.js
test/parallel/test-http-client-unescaped-path.js
test/parallel/test-http-hostname-typechecking.js
test/parallel/test-http-request-invalid-method-error.js
test/parallel/test-http-response-statuscode.js
test/parallel/test-http-server-de-chunked-trailer.js
test/parallel/test-http-write-head.js

Checklist

make -j4 test (UNIX), or vcbuild test (Windows) passes
tests and/or benchmarks are included
commit message follows commit guidelines

Affected core subsystem(s)

test