Skip to content

test: fix assert.throws error in test-http-parser

Rodrigo Muino Tomonari requested to merge github/fork/Trott/assert-throws-nope into master

The third argument of assert.throws() is a message that is used by the AssertionError, not the message to check in the thrown error. It appears that there is an assert.throws() in test-http-parser that expects the latter behavior. Rewrite the call to check the error message. Even if this wasn't a mistake, this change results in a more robust check.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines

Merge request reports