Permalink
Browse files

doc: update assert.doesNotThrow() docs

It takes an optional "expected exception" argument that is not used meaningfully
but is nevertheless documented. Undocument it, it confuses casual readers of the
documentation.

Fixes #3935.
  • Loading branch information...
1 parent 459717e commit e4cef1a0833e6d677298600e205a142d15639bf2 @bnoordhuis bnoordhuis committed Aug 29, 2012
Showing with 2 additions and 2 deletions.
  1. +1 −1 doc/api/assert.markdown
  2. +1 −1 lib/assert.js
View
2 doc/api/assert.markdown
@@ -74,7 +74,7 @@ Custom error validation:
"unexpected error"
);
-## assert.doesNotThrow(block, [error], [message])
+## assert.doesNotThrow(block, [message])
Expects `block` not to throw an error, see assert.throws for details.
View
2 lib/assert.js
@@ -327,7 +327,7 @@ assert.throws = function(block, /*optional*/error, /*optional*/message) {
};
// EXTENSION! This is annoying to write outside this module.
-assert.doesNotThrow = function(block, /*optional*/error, /*optional*/message) {
+assert.doesNotThrow = function(block, /*optional*/message) {
_throws.apply(this, [false].concat(pSlice.call(arguments)));
};

0 comments on commit e4cef1a

Please sign in to comment.