BUG: assert.throws() throws an error without "message" property #2893

Closed
ghost opened this Issue Mar 7, 2012 · 4 comments

Comments

Projects
None yet
3 participants
@ghost

ghost commented Mar 7, 2012

assert.throws() throws an error without "message" property unlike the others:

assert.throws(function () {
  assert.ifError(null);
});

It should throw an error with a message telling what the error is about.

isaacs commented Mar 9, 2012

Yes, this is a bug.

@koichik koichik added a commit to koichik/node that referenced this issue Mar 24, 2012

@koichik koichik assert: fix throws() throws an error without message property
Fixes #2893.
deba244

koichik commented Mar 24, 2012

@isaacs - Can you review koichik/node@deba244?

Owner

bnoordhuis commented Jul 28, 2012

@koichik LGTM

koichik closed this in 72bc4dc Jul 29, 2012

koichik commented Jul 29, 2012

@bnoordhuis - Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment