From: Ben Noordhuis <info@bnoordhuis.nl>
Date: Wed, 29 Aug 2012 00:30:15 +0000 (+0200)
Subject: doc: update assert.doesNotThrow() docs
X-Git-Tag: v0.9.2~52
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=e4cef1a0833e6d677298600e205a142d15639bf2;p=platform%2Fupstream%2Fnodejs.git

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.
---

diff --git a/doc/api/assert.markdown b/doc/api/assert.markdown
index c93c1e7..5849f6d 100644
--- a/doc/api/assert.markdown
+++ b/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.
 
diff --git a/lib/assert.js b/lib/assert.js
index 70dc0b0..f81c203 100644
--- a/lib/assert.js
+++ b/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)));
 };