test: replace assert.throws with common.expectsError (!17494) · 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_4 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-dgram-createSocket-type.js
test/parallel/test-dgram-custom-lookup.js
test/parallel/test-dgram-membership.js
test/parallel/test-dgram-multicast-setTTL.js
test/parallel/test-dgram-sendto.js
test/parallel/test-dgram-setTTL.js
test/parallel/test-dgram-socket-buffer-size.js
test/parallel/test-dns-lookup.js
test/parallel/test-dns-regress-7070.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