remove error builder functions

stephenplusplus · stephenplusplus · commit 01112c4e6388 · 2016-11-03T11:27:47.000-04:00
diff --git a/packages/common/src/util.js b/packages/common/src/util.js
@@ -72,7 +72,7 @@ util.noop = noop;
  *
  * @param {object} errorBody - Error object.
  */
-var ApiError = createErrorClass('ApiError', function(errorBody) {
+util.ApiError = createErrorClass('ApiError', function(errorBody) {
   this.code = errorBody.code;
   this.errors = errorBody.errors;
   this.response = errorBody.response;
@@ -98,21 +98,12 @@ var ApiError = createErrorClass('ApiError', function(errorBody) {
   this.message = uniq(messages).join(' - ');
 });
 
-/**
- * Wrap the ApiError constructor so context isn't lost.
- *
- * @param {object} errorBody - Error object.
- */
-util.ApiError = function(errorBody) {
-  return new ApiError(errorBody);
-};
-
 /**
  * Custom error type for partial errors returned from the API.
  *
  * @param {object} b - Error object.
  */
-var PartialFailureError = createErrorClass('PartialFailureError', function(b) {
+util.PartialFailureError = createErrorClass('PartialFailureError', function(b) {
   var errorObject = b;
 
   this.errors = errorObject.errors;
@@ -122,15 +113,6 @@ var PartialFailureError = createErrorClass('PartialFailureError', function(b) {
   this.message = errorObject.message || defaultErrorMessage;
 });
 
-/**
- * Wrap the PartialFailureError constructor so context isn't lost.
- *
- * @param {object} errorBody - Error object.
- */
-util.PartialFailureError = function(errorBody) {
-  return new PartialFailureError(errorBody);
-};
-
 /**
  * Uniformly process an API response.
  *
