Add assert.ifError

mikeal · ry · commit fe3d8f241120 · 2010-05-21T12:06:12.000-07:00
diff --git a/doc/api.markdown b/doc/api.markdown
@@ -2546,6 +2546,9 @@ Expects `block` to throw an error.
 
 Expects `block` not to throw an error.
 
+### assert.ifError(value)
+
+Tests if value is not a false value, throws if it is a true value. Useful when testing the first argument, `error` in callbacks.
 
 ## Path
 
diff --git a/lib/assert.js b/lib/assert.js
@@ -284,3 +284,4 @@ assert.doesNotThrow = function(block, /*optional*/error, /*optional*/message) {
   _throws.apply(this, [false].concat(pSlice.call(arguments)));
 };
 
+assert.ifError = function (err) { if (err) {throw err;}};
diff --git a/test/simple/test-assert.js b/test/simple/test-assert.js
@@ -154,3 +154,6 @@ try {
 }
 assert.equal(true,threw,'a.doesNotThrow is not catching type matching errors');
 
+assert.throws(function () {assert.ifError(new Error('test error'))});
+assert.doesNotThrow(function(){assert.ifError(null)});
+assert.doesNotThrow(function(){assert.ifError()});

Original file line number	Diff line number	Diff line change
`@@ -284,3 +284,4 @@ assert.doesNotThrow = function(block, /optional/error, /optional/message) {`
`284`	`284`	`_throws.apply(this, [false].concat(pSlice.call(arguments)));`
`285`	`285`	`};`
`286`	`286`
	`287`	`+assert.ifError = function (err) { if (err) {throw err;}};`
Original file line number	Diff line number	Diff line change
`@@ -154,3 +154,6 @@ try {`
`154`	`154`	`}`
`155`	`155`	`assert.equal(true,threw,'a.doesNotThrow is not catching type matching errors');`
`156`	`156`
	`157`	`+assert.throws(function () {assert.ifError(new Error('test error'))});`
	`158`	`+assert.doesNotThrow(function(){assert.ifError(null)});`
	`159`	`+assert.doesNotThrow(function(){assert.ifError()});`