Merge pull request #7 from creynders/patch-2

Expose `isError`
bodenr · Aug 14, 2014 · 00c8f58 · 00c8f58
2 parents 482430d + 9652630
commit 00c8f58
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 2 deletions.
diff --git a/Makefile b/Makefile
@@ -1,4 +1,4 @@
-PROJECT := $(realpath $(dir $(lastword $(MAKEFILE_LIST))))
+PROJECT := "$(realpath $(dir $(lastword $(MAKEFILE_LIST))))"
 MD_PATH := $(PROJECT)/doc/md/
 HTML_PATH := $(PROJECT)/doc/html/
 NODE_BIN := $(PROJECT)/node_modules/.bin/

diff --git a/lib/errors.js b/lib/errors.js
@@ -109,7 +109,7 @@ exports.find = function(err) {
  * @api private
  */
 
-function isError(err) {
+var isError = exports.isError = function isError(err) {
     return err && err.hasOwnProperty('explanation') && err.hasOwnProperty('code');
 };
 

diff --git a/test/test.js b/test/test.js
@@ -247,4 +247,16 @@ describe('status code override', function() {
 	it('should allow overriding in constructor', function() {
 		new CustomHttpError({status:411}).status.should.equal(411);
 	});
+});
+
+describe('errors.isError()', function(){
+    var legitError = new errors.HttpError();
+
+    it('should return true for legit errors', function() {
+        errors.isError(legitError).should.be.ok;
+    });
+
+    it('should return false for other objects', function() {
+        errors.isError({}).should.not.be.ok;
+    });
 });