add error(res, url) helper

componentjs · Nov 2, 2012 · 89d5827 · 89d5827
1 parent ce98a8d
commit 89d5827
Showing 1 changed file with 17 additions and 4 deletions.
diff --git a/lib/Package.js b/lib/Package.js
@@ -154,10 +154,7 @@ Package.prototype.getJSON = function(fn){
       }
       fn(null, json);
     } else {
-      var name = http.STATUS_CODES[res.status];
-      var err = new Error('failed to fetch ' + url + ', got ' + res.status + ' "' + name + '"');
-      err.status = res.status;
-      fn(err);
+      fn(error(res, url));
     }
   });
 
@@ -323,3 +320,19 @@ Package.prototype.reallyInstall = function(){
     });
   });
 };
+
+/**
+ * Return an error for `res` / `url`.
+ *
+ * @param {Response} res
+ * @param {String} url
+ * @return {Error}
+ * @api private
+ */
+
+function error(res, url) {
+  var name = http.STATUS_CODES[res.status];
+  var err = new Error('failed to fetch ' + url + ', got ' + res.status + ' "' + name + '"');
+  err.status = res.status;
+  return err;
+}