Skip to content

test: add missing assert.throws()

Rodrigo Muino Tomonari requested to merge github/fork/cjihrig/test into master

This commit replaces a try...catch in test/parallel/test-dgram-multicast-set-interface.js with assert.throws(). The existing implementation wasn't actually verifying that a synchronous exception is thrown.

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

Merge request reports